Running update.php after enabling Get Clicky generated the following message -

warning: Missing argument 1 for getclicky_menu() in /home/xxxxxxxxxx.com/dev/public_html/sites/all/modules/getclicky/getclicky.module on line 20.

Can't find any settings or othet Get Clicky page, probably related to the above ;¬)

Comments

John Bryan’s picture

hook_menu() in Drupal 6 no longer has the $may_cache argument. Removing this argument from the module stops the error message but the admin/settings get-clicky menu item still does not get created.

Looks like you need to check for differences in hook & function behaviour in Drupal 6 for this module. There are many but the changes are minor, e.g. arguments that were "nid" (node id) are now $node (the whole node object).

regards

John Bryan

John Bryan’s picture

Title: Missing argument 1 for getclicky_menu() » Missing argument 1 for getclicky_menu() - code fix supplied

Comparing with a working hook_menu from another module I have got the menu link working:-
Original

function getclicky_menu($maycache) {
  $items = array();
  if ($maycache) {
    $items[] = array(
      'path' => 'admin/settings/getclicky',
      'title' => t('Get Clicky'),
      'description' => t('Configure the settings used to generate your Get Clicky tracking code.'),
      'callback' => 'drupal_get_form',
      'callback arguments' => 'getclicky_admin_settings',
      'access' => user_access('administer site configuration'),
      'type' => MENU_NORMAL_ITEM,
    );
  }
  return $items;
}

Updated

function getclicky_menu() {
  $items['admin/settings/getclicky'] = array(
      'access arguments' => array('administer site configuration'),
      'page callback' => 'drupal_get_form',
      'page arguments' => array('getclicky_admin_settings'),
      'title' => t('Get Clicky'),
      'description' => t('Configure the settings used to generate your Get Clicky tracking code.'),
      'type' => MENU_NORMAL_ITEM,
    );

  return $items;
}

Presumably though there could easily be other hooks of functions not upgraded for Drupal 6?

gloscon’s picture

Get Clicky is updated to Drupal-6 now.

http://drupal.org/project/getclicky

gloscon’s picture

Version: 6.x-1.x-dev » 6.x-1.0
Status: Active » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.