X-Cart forums

[balinor]

Hey thanks Yes, I'm a big fan of clean and simple.

[mffowler]

In 4.1.0, the "{if $o.selected eq 'Y'} selected="selected"{/if}" forces the first option to be selected and ignores my added "option". You simply have to move "{if $o.selected eq 'Y'} selected="selected"{/if}" from the looped option to the title option that you added.

I didn't notice this being forced prior to 4.1, but the code was there prior. There must be something that prevented the forced selection in previous versions that is fixed now.

- Mike

[nfc5382]

can we add the javascript code anywhere in a global spot to alert, with a generic message, if the first item is selected? Instead of having to add the validation code in the product options for each product and changing the 'poNNNN', etc....

[mffowler]

You can import the javascript validation in 4.1.x if that helps. Otherwise there are probably some options about the forum.

- Mike

[HEK]

Hi Mike and everybody,

thats amazing mod, thanks Mike.
I have a question if customer choose ONLY one option
so they can only choose color option or size option.
What can i do?

--HEK

[mffowler]

If you create the validation for each (and all) options, then it isn't possible to select only one option.

- Mike

[Steve-C]

Thanks Mike. Great Mod. Just what I needed.

[carpeperdiem]

Wow. Lovely mod. Very useful. Thank you.

Post #12 is required reading to make this work for 4.1.x


Here are the changes I made to my customer_options.tpl, as a reference (version 4.1.8 )

FIND:

{foreach from=$v.options item=o}
    <option value="{$o.optionid}"{if $o.selected eq 'Y'} selected="selected"{/if}>{$o.option_name}{if $v.is_modifier eq 'Y' && $o.price_modifier ne 0} ({if $o.modifier_type ne '%'}{include file="currency.tpl" value=$o.price_modifier display_sign=1 plain_text_message=1}{else}{$o.price_modifier}%{/if}){/if}</option>
{/foreach}

REPLACE WITH:

{* I have 2 products that have a single-color option -- they will someday have more than one so I need to exclude these 2 products from this mod...
if you don't need to exclude any products, make the next line go away *}
{if $product.productid ne '9' && $product.productid ne '3' }
{* BEGIN "PLEASE SELECT" MOD *}
<option>Please Select Your {$v.classtext|default:$v.class}...{if $o.selected eq 'Y'} selected="selected"{/if}</option> {/if}
{foreach from=$v.options item=o}
    <option value="{$o.optionid}">{$o.option_name}{if $v.is_modifier eq 'Y' && $o.price_modifier ne 0} ({if $o.modifier_type ne '%'}{include file="currency.tpl" value=$o.price_modifier display_sign=1 plain_text_message=1}{else}{$o.price_modifier}%{/if}){/if}</option>
{/foreach}

Mike -- this a GREAT addition. Thank you!

[carpeperdiem]

Mike (or any javascript expert):

In Firefox/Mac (only) the warning popup box hugs the top of the window. IN every other browser (including Firefox/win, Safari/mac, MSIE 6/7) the box either centers on the display or is at a position to be visible to the user.

Is this a default of the error popup box, or do you know if it's possible to position this error box somehwere -- force it to the middle of the screen, or x=300 y=300 or something besides huging the top of the window (as it does in Firefox/mac). This may be a Firefox bug -- but it may be solved with a line of code that forces the error ox where we want it.

Any ideas?

Thanks!

Jeremy

[inebriate]

you should use document.getElementsByTagName("select") rather than using product option id's since some products may have multiple options

that will return an array of all the elements that have the <select> tags

you would then loop through each array element's value and if it equals "noneselected" (replace with the value you used for the default none selected option, if it doesnt have a value, assign one) or you can compare with the index equaling 0

if the loop stumbles upon just 1 thats not selected, break the loop and throw an error message