You'll want to use

http://example.com/admin/settings/featured-comments/feature/10840

as url.

But also:

function feature_comment($cid = NULL) {
  if (!isset($cid)) {
    // Do something else.
    drupal_goto(...);
  }
 $comment = _comment_load($cid);
 db_query("UPDATE {featured_comments} SET featured=1 WHERE cid=%d", $cid);
 drupal_goto("node/$comment->nid");
}


On Tue, Feb 3, 2009 at 11:34 AM, cooper Quintin <cooperq@cooperq.com> wrote:
Hello all,
I am writing a module and I am trying to use a url as a callback
argument, but it does not work.  Here is the code:

$items[] = array(
   'path' => 'admin/settings/featured-comments/feature',
   'title' => t('featured comment'),
   'callback' => 'feature_comment',
   'callback_args' => $_GET['cid'],
   'access' => user_access('administer comments'),
   'type' => MENU_CALLBACK
 );

and the callback function:
function feature_comment($cid) {
 $comment = _comment_load($cid);
 db_query("UPDATE {featured_comments} SET featured=1 WHERE cid=%d", $cid);
 drupal_goto("node/$comment->nid");
}

going to an example url:
http://example.com/admin/settings/featured-comments/feature?cid=10840
gives the following error: warning: Missing argument 1 for
feature_comment() in
/var/www/html/sites/all/modules/featured_comments/featured_comments.module
on line 149.


So my question is, what is the best way to use parts of the linked url
as a callback argument for a function, because obviously it is not what
I am doing!
Thanks,

--
Cooper Quintin
Freelance Programmer, Indymedia Journalist
http://CooperQ.com
(510) 827-5382