On Wed, Feb 4, 2009 at 12:32 AM, Jakob Petsovits
<jpetso@gmx.at> wrote:
On Tuesday 03 February 2009, Iņaki Lopez wrote:
> $items[] = array(
> 'path' => 'admin/settings/featured-comments/feature',
> 'title' => t('featured comment'),
> 'callback' => 'feature_comment',
> 'callback_args' => $_GET['cid'],
>
> should be..
>
> $items[] = array(
> 'path' => 'admin/settings/featured-comments/feature/%',
> 'title' => t('featured comment'),
> 'callback' => 'feature_comment',
> 'callback_args' =>array(4),
> ...
Bad advice:
1. There is no '%' wildcard in Drupal 5, and Drupal 6 puts the path as array
key instead of having a separate 'path' element.
2. There is no 'callback_args' property. array(...) is correct, but it also
needs to be renamed to 'callback arguments' for Drupal to do anything with it.
In Drupal 5, that would be "'callback arguments' => array($_GET['cid'])".
(Or 'page callback' and 'page arguments' in Drupal 6, which I understand the
original poster is not using at this time.)
Maybe the thread is better off dead -
http://api.drupal.org/api/function/hook_menu/5 has been posted already -
but it's hard to leave the above example uncommented as is.
Cheers,
j