<div dir="ltr"><div class="gmail_default" style="font-family:&#39;courier new&#39;,monospace">Unbelievable, night_pharmacy_form_submit doesn&#39;t even get hit :S . See <a href="http://git.webciniz.im/?p=project/night_pharmacy.git;a=blob;f=night_pharmacy.module;h=b0fecaef686cc458f946e4ab69bbfd81d5f8be25;hb=refs/heads/2013120501#l57">http://git.webciniz.im/?p=project/night_pharmacy.git;a=blob;f=night_pharmacy.module;h=b0fecaef686cc458f946e4ab69bbfd81d5f8be25;hb=refs/heads/2013120501#l57</a> please</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 4 December 2013 16:55, Paolo Mainardi <span dir="ltr">&lt;<a href="mailto:paolo@twinbit.it" target="_blank">paolo@twinbit.it</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">Hi Muzaffer,<div><br></div><div>You should consider to refactor using drupal coding standard, example from comment module:</div><div><br></div><div><div>  $form[&#39;actions&#39;] = array(&#39;#type&#39; =&gt; &#39;actions&#39;);</div>
<div class="im">

<div>  $form[&#39;actions&#39;][&#39;submit&#39;] = array(</div></div><div>    &#39;#type&#39; =&gt; &#39;submit&#39;,</div><div>    &#39;#value&#39; =&gt; t(&#39;Save&#39;),</div><div>    &#39;#access&#39; =&gt; ($comment-&gt;cid &amp;&amp; user_access(&#39;administer comments&#39;)) || variable_get(&#39;comment_preview_&#39; . $node-&gt;type, DRUPAL_OPTIONAL) != DRUPAL_REQUIRED || isset($form_state[&#39;comment_preview&#39;]),</div>


<div>    &#39;#weight&#39; =&gt; 19,</div><div>  );</div><div>  $form[&#39;actions&#39;][&#39;preview&#39;] = array(</div><div>    &#39;#type&#39; =&gt; &#39;submit&#39;,</div><div>    &#39;#value&#39; =&gt; t(&#39;Preview&#39;),</div>


<div>    &#39;#access&#39; =&gt; (variable_get(&#39;comment_preview_&#39; . $node-&gt;type, DRUPAL_OPTIONAL) != DRUPAL_DISABLED),</div><div>    &#39;#weight&#39; =&gt; 20,</div><div>    &#39;#submit&#39; =&gt; array(&#39;comment_form_build_preview&#39;),</div>


<div>  );</div></div><div><br></div><div><a href="https://api.drupal.org/api/drupal/developer!topics!forms_api_reference.html/7#actions" target="_blank">https://api.drupal.org/api/drupal/developer!topics!forms_api_reference.html/7#actions</a><br>


</div><div><br></div><div>Why you need to call a specific submit function, as you&#39;ll have for free &quot;night_pharmacy_form_submit($form, &amp;$form_state)&quot; already executed by Drupal ?</div><div><br></div><div>


<br></div></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">On Wed, Dec 4, 2013 at 3:46 PM, Muzaffer Tolga Ozses <span dir="ltr">&lt;<a href="mailto:tolga@ozses.net" target="_blank">tolga@ozses.net</a>&gt;</span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-family:courier new,monospace">So, what else can I check?</div></div>


<div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On 4 December 2013 15:24, Murat Günana <span dir="ltr">&lt;<a href="mailto:mgunana@gmail.com" target="_blank">mgunana@gmail.com</a>&gt;</span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">According to the form api it should be able to do that.<br></div><div><div><div class="gmail_extra">
<br><br><div class="gmail_quote">On 4 December 2013 13:17, John Morahan <span dir="ltr">&lt;<a href="mailto:john.morahan@gmail.com" target="_blank">john.morahan@gmail.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">&#39;#type&#39; =&gt; &#39;button&#39; won&#39;t execute a submit callback, try &#39;#type&#39; =&gt; &#39;submit&#39;</div>




<div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On 4 December 2013 13:16, Murat Günana <span dir="ltr">&lt;<a href="mailto:mgunana@gmail.com" target="_blank">mgunana@gmail.com</a>&gt;</span> wrote:<br>






<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>You shouldn&#39;t return anything in the callback function. you need to remove &quot;return $source&quot; line from there.<br>






</div>Just for test purposes put this line in callback function and see if you can get any form error <span style="margin:0px;padding:0px;border:0px none;font-family:&#39;Bitstream Vera Sans Mono&#39;,Monaco,&#39;Lucida Console&#39;,monospace;font-style:normal;font-variant:normal;font-weight:normal;line-height:17.9844px;font-size:12px;vertical-align:baseline;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:pre-wrap;word-spacing:0px;background-color:rgb(246,246,242);color:rgb(0,0,187)">form_set_error</span><span style="margin:0px;padding:0px;border:0px none;font-family:&#39;Bitstream Vera Sans Mono&#39;,Monaco,&#39;Lucida Console&#39;,monospace;font-style:normal;font-variant:normal;font-weight:normal;line-height:17.9844px;font-size:12px;vertical-align:baseline;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:pre-wrap;word-spacing:0px;background-color:rgb(246,246,242);color:rgb(0,119,0)">(</span><span style="margin:0px;padding:0px;border:0px none;font-family:&#39;Bitstream Vera Sans Mono&#39;,Monaco,&#39;Lucida Console&#39;,monospace;font-style:normal;font-variant:normal;font-weight:normal;line-height:17.9844px;font-size:12px;vertical-align:baseline;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:pre-wrap;word-spacing:0px;background-color:rgb(246,246,242);color:rgb(221,0,0)">&#39;iller_select&#39;</span><span style="margin:0px;padding:0px;border:0px none;font-family:&#39;Bitstream Vera Sans Mono&#39;,Monaco,&#39;Lucida Console&#39;,monospace;font-style:normal;font-variant:normal;font-weight:normal;line-height:17.9844px;font-size:12px;vertical-align:baseline;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:pre-wrap;word-spacing:0px;background-color:rgb(246,246,242);color:rgb(0,119,0)">, </span><span style="margin:0px;padding:0px;border:0px none;font-family:&#39;Bitstream Vera Sans Mono&#39;,Monaco,&#39;Lucida Console&#39;,monospace;font-style:normal;font-variant:normal;font-weight:normal;line-height:17.9844px;font-size:12px;vertical-align:baseline;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:pre-wrap;word-spacing:0px;background-color:rgb(246,246,242);color:rgb(221,0,0)">&#39;Error.&#39;</span><span style="margin:0px;padding:0px;border:0px none;font-family:&#39;Bitstream Vera Sans Mono&#39;,Monaco,&#39;Lucida Console&#39;,monospace;font-style:normal;font-variant:normal;font-weight:normal;line-height:17.9844px;font-size:12px;vertical-align:baseline;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:pre-wrap;word-spacing:0px;background-color:rgb(246,246,242);color:rgb(0,119,0)">);</span></div>






<div><div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On 4 December 2013 12:59, Muzaffer Tolga Ozses <span dir="ltr">&lt;<a href="mailto:tolga@ozses.net" target="_blank">tolga@ozses.net</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">







<div dir="ltr"><div class="gmail_default"><font face="courier new, monospace">Look at <a href="http://git.webciniz.im/?p=project/night_pharmacy.git;a=blob;f=night_pharmacy.module;h=49adff05adea3549c4566c2cb2ec2ce2f6d5db08;hb=16ee60869975ca8794abc193e0713376945c0c02#l54" target="_blank">http://git.webciniz.im/?p=project/night_pharmacy.git;a=blob;f=night_pharmacy.module;h=49adff05adea3549c4566c2cb2ec2ce2f6d5db08;hb=16ee60869975ca8794abc193e0713376945c0c02#l54</a> . I made the changes you suggested and no dice :S</font><br>








</div></div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On 4 December 2013 14:54, Murat Günana <span dir="ltr">&lt;<a href="mailto:mgunana@gmail.com" target="_blank">mgunana@gmail.com</a>&gt;</span> wrote:<br>








<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>implement your form submit element like;<br>$form[&#39;actions&#39;][&#39;submit&#39;] = array(<br>








     &#39;#type&#39; =&gt; &#39;button&#39;,<br>     &#39;#value&#39; =&gt; t(&#39;Submit&#39;),<br>     &#39;#submit&#39; =&gt; array(&#39;night_pharmacy_table&#39;),<br>
   );<br></div>and make sure you clear the cache.<br></div><div class="gmail_extra"><div><div><br><br><div class="gmail_quote">On 4 December 2013 12:49, Muzaffer Tolga Ozses <span dir="ltr">&lt;<a href="mailto:tolga@ozses.net" target="_blank">tolga@ozses.net</a>&gt;</span> wrote:<br>









<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-family:courier new,monospace">Thanks Patrick,</div><div class="gmail_default" style="font-family:courier new,monospace">









<br></div><div class="gmail_default" style="font-family:courier new,monospace">

Unfortunately, night_pharmacy_table never gets called, and I have no clue why. Do you happen to have an idea?</div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On 4 December 2013 14:35, Patrick Dawkins <span dir="ltr">&lt;<a href="mailto:pjcdawkins@gmail.com" target="_blank">pjcdawkins@gmail.com</a>&gt;</span> wrote:<br>











<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">night_pharmacy_table is a form submit callback. It already gets passed the $form parameter; you have got the function signature wrong.<div>











<br></div><div><div>So you need to change<div><div>function night_pharmacy_table($form_state) {</div>

<div>to</div><div>function night_pharmacy_table($form, &amp;$form_state) {</div></div></div><div><br></div><div><br></div><div>Documentation here: <a href="https://api.drupal.org/api/drupal/developer!topics!forms_api_reference.html/7#submit_property" target="_blank">https://api.drupal.org/api/drupal/developer!topics!forms_api_reference.html/7#submit_property</a><div>













<br></div></div><div><br></div></div></div><div><div><div class="gmail_extra"><br><br><div class="gmail_quote">On 4 December 2013 12:14, Muzaffer Tolga Ozses <span dir="ltr">&lt;<a href="mailto:tolga@ozses.net" target="_blank">tolga@ozses.net</a>&gt;</span> wrote:<br>













<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-family:&#39;courier new&#39;,monospace">Hi,</div><div class="gmail_default" style="font-family:&#39;courier new&#39;,monospace">













<br></div><div class="gmail_default" style="font-family:&#39;courier new&#39;,monospace">
This is really a PHP question, but it involves $form_state, so bear with me please.</div><div class="gmail_default" style="font-family:&#39;courier new&#39;,monospace"><br></div><div class="gmail_default" style="font-family:&#39;courier new&#39;,monospace">














In my code at <a href="http://git.webciniz.im/?p=project/night_pharmacy.git;a=blob;f=night_pharmacy.module;h=132b5d4a90692acfadfe9e11af128f712df1c24d;hb=e89e63bb07f064898ca21be1c97aba221b4dbdaf#l57" target="_blank">http://git.webciniz.im/?p=project/night_pharmacy.git;a=blob;f=night_pharmacy.module;h=132b5d4a90692acfadfe9e11af128f712df1c24d;hb=e89e63bb07f064898ca21be1c97aba221b4dbdaf#l57</a> I want to be able to pass $form parameters to night_pharmacy_table, to no avail. Can you help me guys?</div>














<div class="gmail_default" style="font-family:&#39;courier new&#39;,monospace"><br></div><div class="gmail_default" style="font-family:&#39;courier new&#39;,monospace">Thanks in advance :)</div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div></div></div></div>
</blockquote></div><br><br clear="all"><br></div></div><div>-- <br>Kind regards,<br><br>Murat Gunana<br>Tel: <a href="tel:07927964976" value="+3907927964976" target="_blank">07927964976</a><br>Email: <a href="mailto:mgunana@gmail.com" target="_blank">mgunana@gmail.com</a><br>



</div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Kind regards,<br><br>Murat Gunana<br>Tel: <a href="tel:07927964976" value="+3907927964976" target="_blank">07927964976</a><br>Email: <a href="mailto:mgunana@gmail.com" target="_blank">mgunana@gmail.com</a><br>



</div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Kind regards,<br><br>Murat Gunana<br>Tel: <a href="tel:07927964976" value="+3907927964976" target="_blank">07927964976</a><br>Email: <a href="mailto:mgunana@gmail.com" target="_blank">mgunana@gmail.com</a><br>



</div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div><a href="http://twinbit.it/" style="color:rgb(17,85,204);font-family:Consolas,Menlo,Monaco,&#39;Andale Mono WT&#39;,&#39;Andale Mono&#39;,&#39;Lucida Console&#39;,&#39;Lucida Sans Typewriter&#39;,&#39;DejaVu Sans Mono&#39;,&#39;Bitstream Vera Sans Mono&#39;,&#39;Liberation Mono&#39;,&#39;Nimbus Mono L&#39;,&#39;Courier New&#39;,Courier,monospace;font-size:x-small" target="_blank"><img border="0" alt="TWINBIT logo" width="200" height="46"></a><span style="font-family:Consolas,Menlo,Monaco,&#39;Andale Mono WT&#39;,&#39;Andale Mono&#39;,&#39;Lucida Console&#39;,&#39;Lucida Sans Typewriter&#39;,&#39;DejaVu Sans Mono&#39;,&#39;Bitstream Vera Sans Mono&#39;,&#39;Liberation Mono&#39;,&#39;Nimbus Mono L&#39;,&#39;Courier New&#39;,Courier,monospace;font-size:x-small"> </span><br style="font-family:Consolas,Menlo,Monaco,&#39;Andale Mono WT&#39;,&#39;Andale Mono&#39;,&#39;Lucida Console&#39;,&#39;Lucida Sans Typewriter&#39;,&#39;DejaVu Sans Mono&#39;,&#39;Bitstream Vera Sans Mono&#39;,&#39;Liberation Mono&#39;,&#39;Nimbus Mono L&#39;,&#39;Courier New&#39;,Courier,monospace;font-size:x-small">


<br style="font-family:Consolas,Menlo,Monaco,&#39;Andale Mono WT&#39;,&#39;Andale Mono&#39;,&#39;Lucida Console&#39;,&#39;Lucida Sans Typewriter&#39;,&#39;DejaVu Sans Mono&#39;,&#39;Bitstream Vera Sans Mono&#39;,&#39;Liberation Mono&#39;,&#39;Nimbus Mono L&#39;,&#39;Courier New&#39;,Courier,monospace;font-size:x-small">


<font style="font-family:arial,helvetica,sans-serif;color:rgb(51,51,51)"><font size="4">Paolo Mainardi</font> <br></font><i>Twinbit Founder - CTO<br style="font-family:Consolas,Menlo,Monaco,&#39;Andale Mono WT&#39;,&#39;Andale Mono&#39;,&#39;Lucida Console&#39;,&#39;Lucida Sans Typewriter&#39;,&#39;DejaVu Sans Mono&#39;,&#39;Bitstream Vera Sans Mono&#39;,&#39;Liberation Mono&#39;,&#39;Nimbus Mono L&#39;,&#39;Courier New&#39;,Courier,monospace;font-size:x-small">


</i><br style="font-family:Consolas,Menlo,Monaco,&#39;Andale Mono WT&#39;,&#39;Andale Mono&#39;,&#39;Lucida Console&#39;,&#39;Lucida Sans Typewriter&#39;,&#39;DejaVu Sans Mono&#39;,&#39;Bitstream Vera Sans Mono&#39;,&#39;Liberation Mono&#39;,&#39;Nimbus Mono L&#39;,&#39;Courier New&#39;,Courier,monospace;font-size:x-small">


<font face="Consolas, Menlo, Monaco, Courier New, Courier, monospace" color="#666666" style="font-family:Consolas,Menlo,Monaco,&#39;Andale Mono WT&#39;,&#39;Andale Mono&#39;,&#39;Lucida Console&#39;,&#39;Lucida Sans Typewriter&#39;,&#39;DejaVu Sans Mono&#39;,&#39;Bitstream Vera Sans Mono&#39;,&#39;Liberation Mono&#39;,&#39;Nimbus Mono L&#39;,&#39;Courier New&#39;,Courier,monospace"><font face="Consolas, Menlo, Monaco, Courier New, Courier, monospace"><b>www</b>:      <a href="http://twinbit.it/" style="color:rgb(241,126,14)" target="_blank">twinbit.it</a></font> <br>


<font face="Consolas, Menlo, Monaco, Courier New, Courier, monospace"><b>mobile</b>:   <a value="+393895122611" style="color:rgb(17,85,204)">(+39) 3401678089</a></font> <br><font face="Consolas, Menlo, Monaco, Courier New, Courier, monospace"><b>skype</b>:    paolo_mainardi</font> <br>


<font face="Consolas, Menlo, Monaco, Courier New, Courier, monospace"><b>linkedin</b>: <a href="http://www.linkedin.com/in/paolomainardi" style="color:rgb(241,126,14)" target="_blank">paolomainardi</a></font> <br><font face="Consolas, Menlo, Monaco, Courier New, Courier, monospace"><b>twitter</b>:  <a href="http://twitter.com/paolomainardi" style="color:rgb(241,126,14)" target="_blank">@paolomainardi</a></font> <br>


<font face="Consolas, Menlo, Monaco, Courier New, Courier, monospace"><b>blog</b>:     <a href="http://paolomainardi.com/" style="color:rgb(241,126,14)" target="_blank">paolomainardi.com</a></font> <br></font></div>
</font></span></div>
</blockquote></div><br></div>