You need to change submit type from "button" to "submit". For some reason it's not getting triggered.
See discussion https://drupal.org/node/72855. The working one for you should be like $form['actions']['submit'] = array( '#type' => 'submit', '#value' => t('Submit'), '#submit' => array('night_pharmacy_form_submit'), ); On 4 December 2013 22:48, Muzaffer Tolga Ozses <to...@ozses.net> wrote: > Unbelievable, night_pharmacy_form_submit doesn't even get hit :S . See > http://git.webciniz.im/?p=project/night_pharmacy.git;a=blob;f=night_pharmacy.module;h=b0fecaef686cc458f946e4ab69bbfd81d5f8be25;hb=refs/heads/2013120501#l57please > > > On 4 December 2013 16:55, Paolo Mainardi <pa...@twinbit.it> wrote: > >> Hi Muzaffer, >> >> You should consider to refactor using drupal coding standard, example >> from comment module: >> >> $form['actions'] = array('#type' => 'actions'); >> $form['actions']['submit'] = array( >> '#type' => 'submit', >> '#value' => t('Save'), >> '#access' => ($comment->cid && user_access('administer comments')) || >> variable_get('comment_preview_' . $node->type, DRUPAL_OPTIONAL) != >> DRUPAL_REQUIRED || isset($form_state['comment_preview']), >> '#weight' => 19, >> ); >> $form['actions']['preview'] = array( >> '#type' => 'submit', >> '#value' => t('Preview'), >> '#access' => (variable_get('comment_preview_' . $node->type, >> DRUPAL_OPTIONAL) != DRUPAL_DISABLED), >> '#weight' => 20, >> '#submit' => array('comment_form_build_preview'), >> ); >> >> >> https://api.drupal.org/api/drupal/developer!topics!forms_api_reference.html/7#actions >> >> Why you need to call a specific submit function, as you'll have for free >> "night_pharmacy_form_submit($form, &$form_state)" already executed by >> Drupal ? >> >> >> >> >> On Wed, Dec 4, 2013 at 3:46 PM, Muzaffer Tolga Ozses <to...@ozses.net>wrote: >> >>> So, what else can I check? >>> >>> >>> On 4 December 2013 15:24, Murat Günana <mgun...@gmail.com> wrote: >>> >>>> According to the form api it should be able to do that. >>>> >>>> >>>> On 4 December 2013 13:17, John Morahan <john.mora...@gmail.com> wrote: >>>> >>>>> '#type' => 'button' won't execute a submit callback, try '#type' => >>>>> 'submit' >>>>> >>>>> >>>>> On 4 December 2013 13:16, Murat Günana <mgun...@gmail.com> wrote: >>>>> >>>>>> You shouldn't return anything in the callback function. you need to >>>>>> remove "return $source" line from there. >>>>>> Just for test purposes put this line in callback function and see if >>>>>> you can get any form error form_set_error('iller_select', 'Error.'); >>>>>> >>>>>> >>>>>> On 4 December 2013 12:59, Muzaffer Tolga Ozses <to...@ozses.net>wrote: >>>>>> >>>>>>> Look at >>>>>>> http://git.webciniz.im/?p=project/night_pharmacy.git;a=blob;f=night_pharmacy.module;h=49adff05adea3549c4566c2cb2ec2ce2f6d5db08;hb=16ee60869975ca8794abc193e0713376945c0c02#l54. >>>>>>> I made the changes you suggested and no dice :S >>>>>>> >>>>>>> >>>>>>> On 4 December 2013 14:54, Murat Günana <mgun...@gmail.com> wrote: >>>>>>> >>>>>>>> implement your form submit element like; >>>>>>>> $form['actions']['submit'] = array( >>>>>>>> '#type' => 'button', >>>>>>>> '#value' => t('Submit'), >>>>>>>> '#submit' => array('night_pharmacy_table'), >>>>>>>> ); >>>>>>>> and make sure you clear the cache. >>>>>>>> >>>>>>>> >>>>>>>> On 4 December 2013 12:49, Muzaffer Tolga Ozses <to...@ozses.net>wrote: >>>>>>>> >>>>>>>>> Thanks Patrick, >>>>>>>>> >>>>>>>>> Unfortunately, night_pharmacy_table never gets called, and I have >>>>>>>>> no clue why. Do you happen to have an idea? >>>>>>>>> >>>>>>>>> >>>>>>>>> On 4 December 2013 14:35, Patrick Dawkins <pjcdawk...@gmail.com>wrote: >>>>>>>>> >>>>>>>>>> night_pharmacy_table is a form submit callback. It already gets >>>>>>>>>> passed the $form parameter; you have got the function signature >>>>>>>>>> wrong. >>>>>>>>>> >>>>>>>>>> So you need to change >>>>>>>>>> function night_pharmacy_table($form_state) { >>>>>>>>>> to >>>>>>>>>> function night_pharmacy_table($form, &$form_state) { >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> Documentation here: >>>>>>>>>> https://api.drupal.org/api/drupal/developer!topics!forms_api_reference.html/7#submit_property >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>>> On 4 December 2013 12:14, Muzaffer Tolga Ozses >>>>>>>>>> <to...@ozses.net>wrote: >>>>>>>>>> >>>>>>>>>>> Hi, >>>>>>>>>>> >>>>>>>>>>> This is really a PHP question, but it involves $form_state, so >>>>>>>>>>> bear with me please. >>>>>>>>>>> >>>>>>>>>>> In my code at >>>>>>>>>>> http://git.webciniz.im/?p=project/night_pharmacy.git;a=blob;f=night_pharmacy.module;h=132b5d4a90692acfadfe9e11af128f712df1c24d;hb=e89e63bb07f064898ca21be1c97aba221b4dbdaf#l57I >>>>>>>>>>> want to be able to pass $form parameters to night_pharmacy_table, >>>>>>>>>>> to no >>>>>>>>>>> avail. Can you help me guys? >>>>>>>>>>> >>>>>>>>>>> Thanks in advance :) >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> -- >>>>>>>> Kind regards, >>>>>>>> >>>>>>>> Murat Gunana >>>>>>>> Tel: 07927964976 >>>>>>>> Email: mgun...@gmail.com >>>>>>>> >>>>>>> >>>>>>> >>>>>> >>>>>> >>>>>> -- >>>>>> Kind regards, >>>>>> >>>>>> Murat Gunana >>>>>> Tel: 07927964976 >>>>>> Email: mgun...@gmail.com >>>>>> >>>>> >>>>> >>>> >>>> >>>> -- >>>> Kind regards, >>>> >>>> Murat Gunana >>>> Tel: 07927964976 >>>> Email: mgun...@gmail.com >>>> >>> >>> >> >> >> -- >> [image: TWINBIT logo] <http://twinbit.it/> >> >> Paolo Mainardi >> >> *Twinbit Founder - CTO * >> *www*: twinbit.it >> *mobile*: (+39) 3401678089 >> *skype*: paolo_mainardi >> *linkedin*: paolomainardi <http://www.linkedin.com/in/paolomainardi> >> *twitter*: @paolomainardi <http://twitter.com/paolomainardi> >> *blog*: paolomainardi.com >> > > -- Kind regards, Murat Gunana Tel: 07927964976 Email: mgun...@gmail.com
