Since I started this thread I feel it is my duty to put it to rest. The problem was indeed that I had callback_args instead of callback arguments, sorry to bother this list with my stupid typo. Additionally the below poster is correct, there is no wildcard in paths in drupal 5 which was also giving me some interesting problems. I thought that the development list was intended for all development of drupal and its modules but I guess its only for drupal core development, sorry about that! Anyway, I declare this thread closed, thanks everyone for your help. Cooper Iñaki Lopez wrote:
sorry, I don't know how did I realize the drupal version was six, and my reply was a "finish-porting-this-to-d6".. Now I see, only half of the menu entry is 'ported' as well..
Indeed, a very bad advice at all.. where the hell is my head?.. brbr
On Wed, Feb 4, 2009 at 12:32 AM, Jakob Petsovits <jpetso@gmx.at <mailto: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
-- Cooper Quintin Freelance Programmer, Indymedia Journalist http://CooperQ.com (510) 827-5382