Getting session variable inside submit fonction

Posted by stomerfull on December 17, 2012 at 4:26pm

Hello ,

I'm not able to get session variable inside my submit handler function
i post my module form in home page and i would like to obtain value setting in submit handler

<?php
function my_module_form($form, &$form_state) {
global $base_path
$form['#action'] = $base_url;
    $form['cp'] = array(
        '#type' => 'textfield',
        '#size' => 25,
        '#required' => false,
     );
    $form['submit'] = array(
        '#type' => 'submit',
        '#value' => t('Next'),
    );
    return $form;
}
?>

<?php
function my_module_form_validate($form, &$form_state) {
   $mess = "";
    if( $form_state['values']['cp'] == '' || $form_state['values']['cp'] == '0' ){
        $mess = form_set_error('cp', "error 1");
    }elseif( $form_state['values']['cp'] != '' ||  $form_state['values']['cp'] != '0'){
          if (!preg_match('#^[0-9]{5,5}$#',$form_state['values']['cp'])){
                $mess = form_set_error('cp', "error 2");
          }elseif( !is_numeric($form_state['values']['cp'])){
                $mess = form_set_error('cp', "error "");
          } 
    }
    return $mess;
}
?>

<?php
function my_module_form_submit($form, &$form_state) {

    $cp = $form_state['values']['cp'];
    
       $_SESSION['myvar']= doStuff($cp);
}
?>

im not able to get $_SESSION['myvar'] inside my front page

Thank you very much for your help

Comments

Two things that I

Posted by Drave Robber on December 17, 2012 at 5:31pm

Two things that I notice:

function naming - if the form constructor is my_module_form($form, &$form_state), default validator and submit handler are my_module_form_validate($form, &$form_state) and my_module_form_submit($form, &$form_state), respectively;
form validators act on references and should not attempt to return; stuff; form_set_error() is also used in a weird manner. should be something like:

<?php
function my_module_form_validate($form, &$form_state) {
  if( $form_state['values']['cp'] == '' || $form_state['values']['cp'] == '0' ) {
    form_set_error('cp', "error 1");
  }
  elseif( $form_state['values']['cp'] != '' ||  $form_state['values']['cp'] != '0') {
    if (!preg_match('#^[0-9]{5,5}$#',$form_state['values']['cp'])) {
      form_set_error('cp', "error 2");
    }
    elseif (!is_numeric($form_state['values']['cp'])) {
      form_set_error('cp', "error "");
    } 
  }
}
?>