sorry, I don&#39;t know how did I realize the drupal version was six, and my reply was a &quot;finish-porting-this-to-d6&quot;.. <br>Now I see, only half of the menu entry is &#39;ported&#39; as well.. <br><br>Indeed, a very bad advice at all.. where the hell is my head?.. brbr<br>
<br><div class="gmail_quote">On Wed, Feb 4, 2009 at 12:32 AM, Jakob Petsovits <span dir="ltr">&lt;<a href="mailto:jpetso@gmx.at">jpetso@gmx.at</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;">
<div class="Ih2E3d">On Tuesday 03 February 2009, Iņaki Lopez wrote:<br>
&gt; $items[] = array(<br>
&gt; &nbsp; &nbsp;&#39;path&#39; =&gt; &#39;admin/settings/featured-comments/feature&#39;,<br>
&gt; &nbsp; &nbsp;&#39;title&#39; =&gt; t(&#39;featured comment&#39;),<br>
&gt; &nbsp; &nbsp;&#39;callback&#39; =&gt; &#39;feature_comment&#39;,<br>
&gt; &nbsp; &nbsp;&#39;callback_args&#39; =&gt; $_GET[&#39;cid&#39;],<br>
&gt;<br>
&gt; should be..<br>
&gt;<br>
&gt; $items[] = array(<br>
&gt; &nbsp; &nbsp;&#39;path&#39; =&gt; &#39;admin/settings/featured-comments/feature/%&#39;,<br>
&gt; &nbsp; &nbsp;&#39;title&#39; =&gt; t(&#39;featured comment&#39;),<br>
&gt; &nbsp; &nbsp;&#39;callback&#39; =&gt; &#39;feature_comment&#39;,<br>
&gt; &nbsp; &nbsp;&#39;callback_args&#39; =&gt;array(4),<br>
&gt; ...<br>
<br>
</div>Bad advice:<br>
<br>
1. There is no &#39;%&#39; wildcard in Drupal 5, and Drupal 6 puts the path as array<br>
key instead of having a separate &#39;path&#39; element.<br>
<br>
2. There is no &#39;callback_args&#39; property. array(...) is correct, but it also<br>
needs to be renamed to &#39;callback arguments&#39; for Drupal to do anything with it.<br>
In Drupal 5, that would be &quot;&#39;callback arguments&#39; =&gt; array($_GET[&#39;cid&#39;])&quot;.<br>
(Or &#39;page callback&#39; and &#39;page arguments&#39; in Drupal 6, which I understand the<br>
original poster is not using at this time.)<br>
<br>
Maybe the thread is better off dead -<br>
<a href="http://api.drupal.org/api/function/hook_menu/5" target="_blank">http://api.drupal.org/api/function/hook_menu/5</a> has been posted already -<br>
but it&#39;s hard to leave the above example uncommented as is.<br>
<br>
Cheers,<br>
<font color="#888888"> &nbsp;j<br>
<br>
</font></blockquote></div><br>