[development] url as callback argument

Robrecht Jacques robrechtj+drupal at gmail.com
Tue Feb 3 10:46:21 UTC 2009


You'll want to use

http://example.com/admin/settings/featured-comments/feature/10840<http://example.com/admin/settings/featured-comments/feature?cid=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 at 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.drupal.org/pipermail/development/attachments/20090203/541db3fd/attachment.htm 


More information about the development mailing list