The menu callback for 'node//reviews' is defined like this:

        $items[] = array(
          'path' => 'node/' . arg(1) . '/reviews',
          'callback' => 'views_page',
          'callback arguments' => array('review list', arg(1)),
          'type' => MENU_LOCAL_TASK,
          'title' => t('Reviews'),
          'weight' => 2,
          'access' => user_access('read reviews'),
        );

This code obviously doesn't work ('views_page() does not exist, 'callback arguments' are also wrong). However, the URL 'node//reviews' does work.. This happens since the default view 'review_list' is defined like this:

  $view->url = 'node/$arg/reviews';

So the 'views_menu' hook actually fixes the wrong information from nodereview_menu, but leaving some of the info intact..
In short - this is a mess, that should be resolved, although it looks like working, it took me a while to find out what is going on here..

Comments

danielhonrade’s picture

Status: Active » Closed (won't fix)

This won't be fixed anymore or replicate to verify this issue, we just have to move forward and making version 6 stable and create version 7 dev.