Undefined offset: 0 in scheduler_list_access_callback() [#1124276]

Comment #1

jonathan1055 commented 12 April 2011 at 17:40

Title:

Undefined offset: 0 in scheduler_list_access_callback() (line 99...sites/all/modules/scheduler/scheduler.module)

» Undefined offset: 0 in scheduler_list_access_callback()

Hi,
Yes you are right, this is an error. It must be related to our recent change in #1071976: Authenticated user to view list of own scheduled nodes. I'll check it now.

Jonathan

ps. I moved the 'line 99...sites/all/modules/scheduler/scheduler.module' out of the rather long title.

Log in or register to post comments

Comment #2

jonathan1055 commented 12 April 2011 at 17:56

Issue tags:

-scheduler

Eric,
The problem is that we did not add parameters to the page_callback and access_callback in hook_menu(). One solution would be to add array(NULL, NULL) into the definition as below.

  $items['admin/content/scheduler'] = array(
    'type' => MENU_LOCAL_TASK,
    'title' => 'Scheduled',
    'page callback' => 'scheduler_list',
    'page arguments' => array(NULL, NULL), // <<<<< this was missing
    'access callback' => 'scheduler_list_access_callback',
    'access arguments' => array(NULL, NULL),  // <<<<< this was missing
    'description' => 'Display a list of scheduled nodes',
    'file' => NULL,
  );
  $items['user/%/scheduler'] = array(
    'type' => MENU_LOCAL_TASK,
    'title' => 'Scheduled',
    'page callback' => 'scheduler_list',
    'page arguments' => array('user_only', 1), // This will pass the uid of the user account being viewed.
    'access callback' => 'scheduler_list_access_callback',
    'access arguments' => array('user_only', 1),
    'description' => 'Display a list of scheduled nodes',
    'file' => NULL,
  );

An alternative would be to test for isset($args[0]) in both the scheduler_list_access_callback() and scheduler_list() functions.

What's your preference?

Jonathan