[development] Passing parametre

Muzaffer Tolga Ozses tolga at ozses.net
Thu Dec 5 06:56:48 UTC 2013


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 <mgunana at 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 <tolga at 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 <paolo at 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 <tolga at ozses.net>wrote:
>>>
>>>> So, what else can I check?
>>>>
>>>>
>>>> On 4 December 2013 15:24, Murat Günana <mgunana at gmail.com> wrote:
>>>>
>>>>> According to the form api it should be able to do that.
>>>>>
>>>>>
>>>>> On 4 December 2013 13:17, John Morahan <john.morahan at gmail.com> wrote:
>>>>>
>>>>>> '#type' => 'button' won't execute a submit callback, try '#type' =>
>>>>>> 'submit'
>>>>>>
>>>>>>
>>>>>> On 4 December 2013 13:16, Murat Günana <mgunana at 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 <tolga at 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 <mgunana at 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 <tolga at 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 <pjcdawkins at 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 <tolga at 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: mgunana at gmail.com
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Kind regards,
>>>>>>>
>>>>>>> Murat Gunana
>>>>>>> Tel: 07927964976
>>>>>>> Email: mgunana at gmail.com
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> Kind regards,
>>>>>
>>>>> Murat Gunana
>>>>> Tel: 07927964976
>>>>> Email: mgunana at 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: mgunana at gmail.com
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.drupal.org/pipermail/development/attachments/20131205/b522e9e4/attachment.html 


More information about the development mailing list