You were right, I did. It's just that I have been told 'button' and 'submit' both could submit forms.
On 5 December 2013 01:21, Murat Günana <mgun...@gmail.com> wrote: > 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 >
