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#l57 please


On 4 December 2013 16:55, Paolo Mainardi <paolo@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@ozses.net> wrote:
So, what else can I check?


On 4 December 2013 15:24, Murat Günana <mgunana@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@gmail.com> wrote:
'#type' => 'button' won't execute a submit callback, try '#type' => 'submit'


On 4 December 2013 13:16, Murat Günana <mgunana@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@ozses.net> wrote:


On 4 December 2013 14:54, Murat Günana <mgunana@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@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@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) {





On 4 December 2013 12:14, Muzaffer Tolga Ozses <tolga@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#l57 I 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@gmail.com




--
Kind regards,

Murat Gunana
Tel: 07927964976
Email: mgunana@gmail.com




--
Kind regards,

Murat Gunana
Tel: 07927964976
Email: mgunana@gmail.com




--
TWINBIT logo 

Paolo Mainardi 
Twinbit Founder - CTO

www:      twinbit.it 
mobile:   (+39) 3401678089 
skype:    paolo_mainardi 
linkedinpaolomainardi 
twitter:  @paolomainardi 
blog:     paolomainardi.com