在hook_menu的访问回调中重定向用户会产生不良影响吗？

这个答案引发了关于如果用户无权访问他们要访问的页面的最佳方式将用户重定向到特定页面的讨论。

一种选择是将访问回调设置为true，然后在页面回调中重定向用户。尽管这似乎是正确的，但我认为它在页面回调中混合了访问功能和页面构建功能。

例如

function hook_menu() {
    $items['player/my_page'] = array(
        'title' => t('My Page'), // note this is a required parameter
        'access callback' => TRUE,
        'page callback' => 'some_function',
    );
    return $items;
}

function some_function() {
    global $user;   
    if(!$user->uid) { // here checking if the user is logged in but could be checking for a specific permission or field value
        $dest = drupal_get_destination();
        drupal_goto('user/login', $dest); // this remembers where the user is coming from
    }
    // carry on building rest of page
}

另一种选择是将访问回调函数设置为调用一个检查用户是否具有访问权限的函数，但是不是返回false而是将用户重定向到另一个页面。这很好，因为它将访问逻辑和页面构建逻辑分开。但是，访问回调的目的是返回一个布尔值，因此这通过重定向用户来破坏该逻辑。

例如

function hook_menu() {
    $items['player/my_page'] = array(
        'title' => t('My Page'), // note this is a required parameter
        'access callback' => 'check_access',
        'page callback' => 'some_function',
    );
    return $items;
}

function check_access() {
    global $user;
    // here checking if the user is logged in but could be checking for a specific permission or field value
    if(!$user->uid) {
        $dest = drupal_get_destination();
        drupal_goto('user/login', $dest);
    }
    return TRUE;
}

在我不知道的访问回调中重定向用户是否会带来不良影响？

您认为这里的最佳做法是什么？

我认为您可以通过更改传递回调函数来实现。如果访问回调返回FALSE，则传递给传递回调。如果仅在特定页面上需要这种重定向行为，则可以通过hook_menu()或仅为这些页面修改传递回调hook_menu_alter()。如果您希望行为是全局的，则可以使用对其hook_page_delivery_callback_alter()进行更改。

这是一个传递回调示例。

function custom_deliver_html_page($page_callback_result) {
  if ($page_callback_result === MENU_ACCESS_DENIED) {
    drupal_goto('<front>');
  }
  drupal_deliver_html_page($page_callback_result);
 }

顺便说一下，这是未经测试的，我以前从未真正更改过传递回调。

我不会在访问回调中这样做。如果另一位开发人员想page_callback在将来的某个时间更改，他们最终将scratch之以鼻，为什么您在访问回调中重定向时，回调无法正常工作。