Hm, I commented the lines in template.php and built a new module with the hook_form_alter and additional logic but this does not make any difference at all. Exactly the same behaviour...

Thanasis

On 20/02/12 09:25, Mukesh Agarwal wrote:

Yes, Drupal is quite structured and has defined ways to deal with
stuff.. template.php and hook_form_alter are for 2 different purposes
and in your case writing a custom module which implements the hook is
the right way.

On Mon, Feb 20, 2012 at 2:53 PM, Athanasios Velios <a.velios@gmail.com

<mailto:a.velios@gmail.com>> wrote:

   Do you reckon that the problem is putting the code in template.php
   instead of a custom module?

   Thanks again,

   Thanasis


   On 20/02/12 08:27, Mukesh Agarwal wrote:

       The place to implement logic based form_set_error should be in the
       validate function and not template.php .. I would recommend you use
       hook_form_alter

       <http://api.drupal.org/api/__drupal/modules%21system%__21system.api.php/function/__hook_form_alter/7

       <http://api.drupal.org/api/drupal/modules%21system%21system.api.php/function/hook_form_alter/7>>

       -> $form_id will indicate the form you wish to alter, and there
       you can
       add a custom validate function in $form['#validate'] -- the custom
       function should then take care of form_set_error

       http://api.drupal.org/api/__drupal/includes%21form.inc/__function/form_set_error/7

       <http://api.drupal.org/api/drupal/includes%21form.inc/function/form_set_error/7>
       -- you need to set the element name as first parameter to make
       sure that
       the particular field is highlighted.

       On Mon, Feb 20, 2012 at 1:51 PM, Athanasios Velios
       <a.velios@gmail.com <mailto:a.velios@gmail.com>

       <mailto:a.velios@gmail.com <mailto:a.velios@gmail.com>>> wrote:

           Hello,

           I am trying to alter a form (uc_cart_view_form) with various
       input
           fields (Qty for products). In template.php I am adding some
       extra
           logic which applies across a selection of these fields. Once the
           logic is checked I need to display an error message. I am using
           form_set_error to show the message. The problem is that the user
           then updates the input values and updates the form, but the new
           figures are not taken into account. They are shown in the
       form, but
           not used in my template.php extra logic.

           drupal_set_message allows the new values to work but the message
           appears twice and it does not, obviously, link to specific
       fields in
           the form which are problematic.

           Setting form_state rebuild to true before form_set_error
       does not
           have an effect.

           Any ideas of what needs to be done for these values to be
       taken into
           account with form_set_error?

           Thanks

           Thanasis




       --
       Cheers,
       Mukesh Agarwal

       __________________________________

       Innoraft Solutions <http://www.innoraft.com>  || +91 8017220799




--
Cheers,
Mukesh Agarwal
________________________________

Innoraft Solutions <http://www.innoraft.com>  || +91 8017220799