Text Box Dissapears

[welshop.com]

Just discovered something rather worrying with regard to all the text box attributes on my site which have "Attribute Required for Text"

Basically when the product page is first opened the text box appears however if it is left blank and the item is added to the cart the order goes through without coming up with the standard alert.

If there are two attributes on a page say a drop down which is required as well the error comes up for that however this also causes the test box to dissapear.

Any ideas?

Url: http://www.celtmyth.co.uk/pages-prod...lour-wrap.html

Cheers
Brinley

[Ajeh]

There is a setting on the Attributes specifically for Text Attributes to make them required ...

Attribute Required for Text:
No Yes

[welshop.com]

Hi Linda - thanks for responding

I have the Attribute required for text set to yes - see screenshot



Any advice would be appreciated

Cheers
Brinley

[Ajeh]

We fondly call this feature a bug ...

Edit the /includes/functions/functions_lookup.php and around line 419 change the code to read:

// text required validation
    if (ereg('^txt_', $option)) {
//      $check_attributes = $db->Execute("select attributes_display_only, attributes_required from " . TABLE_PRODUCTS_ATTRIBUTES . " where products_id='" . (int)$product_id . "' and options_id='" . ereg_replace('txt_', '', (int)$option) . "' and options_values_id='0'");
      $check_attributes = $db->Execute("select attributes_display_only, attributes_required from " . TABLE_PRODUCTS_ATTRIBUTES . " where products_id='" . (int)$product_id . "' and options_id='" . ereg_replace('txt_', '', $option) . "' and options_values_id='0'");
// text cannot be blank
      if ($check_attributes->fields['attributes_required'] == '1' and empty($value)) {
        $check_valid = false;
      }
    } 

See if that works for you ...

[welshop.com]

Hi Linda

Thanks for having a pop at a solution.

I made the amendments you suggested but still no change unfortunately - the text fields still dissapear if alongside dropdowns and also the text fields dont cause an alert if left blank.

[Ajeh]

odd ... I have it working in my version that is about to be released ...

I will have to see if there is more changes that might effect this ...

Meanwhile ... switch to Classic ... does it work then?