<br><br><div class="gmail_quote">On Tue, Feb 3, 2009 at 11:34 AM, cooper Quintin <span dir="ltr">&lt;<a href="mailto:cooperq@cooperq.com">cooperq@cooperq.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hello all,<br>
I am writing a module and I am trying to use a url as a callback<br>
argument, but it does not work. &nbsp;Here is the code:<br>
<br>
$items[] = array(<br>
 &nbsp; &nbsp;&#39;path&#39; =&gt; &#39;admin/settings/featured-comments/feature&#39;,<br>
 &nbsp; &nbsp;&#39;title&#39; =&gt; t(&#39;featured comment&#39;),<br>
 &nbsp; &nbsp;&#39;callback&#39; =&gt; &#39;feature_comment&#39;,<br>
 &nbsp; &nbsp;&#39;callback_args&#39; =&gt; $_GET[&#39;cid&#39;],<br>
 &nbsp; &nbsp;&#39;access&#39; =&gt; user_access(&#39;administer comments&#39;),<br>
 &nbsp; &nbsp;&#39;type&#39; =&gt; MENU_CALLBACK<br>
 &nbsp;);<br>
<br>
and the callback function:<br>
function feature_comment($cid) {<br>
 &nbsp;$comment = _comment_load($cid);<br>
 &nbsp;db_query(&quot;UPDATE {featured_comments} SET featured=1 WHERE cid=%d&quot;, $cid);<br>
 &nbsp;drupal_goto(&quot;node/$comment-&gt;nid&quot;);<br>
}<br>
</blockquote></div><br><br>I think you can bypass the problem loading the $_GET in feature_comment function and obviously setting $cid = NULL, so:<br><br>function feature_comment($cid = NULL) {<br>&nbsp; if (!isset($cid)) {<br>
&nbsp;&nbsp;&nbsp;&nbsp; if (!$cid = $_GET[&#39;cid&#39;]) {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; drupal_goto();<br>&nbsp;&nbsp;&nbsp;&nbsp; }<br>&nbsp; }<br>
 &nbsp;$comment = _comment_load($cid);<br>
 &nbsp;db_query(&quot;UPDATE {featured_comments} SET featured=1 WHERE cid=%d&quot;, $cid);<br>
 &nbsp;drupal_goto(&quot;node/$comment-&gt;<div class="Ih2E3d">nid&quot;);<br>}</div><br><br clear="all"><br>-- <br>Paolo Mainardi<br><br>Vice Presidente Assoc.ILDN (<a href="http://www.ildn.net">http://www.ildn.net</a>)<br>Blog: <a href="http://www.paolomainardi.com">http://www.paolomainardi.com</a><br>