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 <[email protected]
<mailto:[email protected]>> 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
        <[email protected] <mailto:[email protected]>
        <mailto:[email protected] <mailto:[email protected]>>> 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

Reply via email to