[ic] Any problems with this small hunk of code?
Spencer Christensen
schristensen at backcountry.com
Thu Sep 29 11:51:09 EDT 2005
Brian J. Miller wrote:
> Spencer Christensen wrote:
>
>>Brian J. Miller wrote:
>>
>>
>>>DB wrote:
>>>
>>>
>>>>The following code seems to work ok within a loop on a page I named
>>>>items_list.html, the goal being to add a part to the cart and return to
>>>>the same page. I'm just wondering if there are any glaring problems with
>>>>doing things this way. Any thoughts welcome.
>>>>
>>>> <form action="[process-target]" method="POST">
>>>> <input type="hidden" name="mv_session_id" value="[data session id]">
>>>> <input type="hidden" name="mv_todo" value="refresh">
>>>> <input type="hidden" name="mv_nextpage" value="ites_list">
>>>
>>>
>>>
>>>I am assuming the above should read "items_list" instead of "ites_list"
>>>based on the name in your text. An easy way to avoid this issue if you
>>>are going to submit back to the same page is to use an IC "built-in"
>>>variable.
>>>
>>><input type="hidden" name="mv_nextpage" value="@@MV_PAGE@@">
>>>
>>>Should give you the same thing and is more portable.
>>
>>
>>Hello DB,
>> Besides what Brian mentioned, I would say that this method is fine.
>>One thing that crosses my mind is your quantities. Is there a reason
>>for hard coding the quantities to 1? Are you restricting customers from
>>ordering more than one of the same item? If so, then you will need to
>>stop them from adding the same item to their cart more than once. They
>> add it to their cart, the page refreshes, and then they click on the
>>item again. Something to think about. But if this isn't a problem,
>>then don't worry about it.
>>
>
>
>>From his original code:
>
> <input type="text" name="mv_order_quantity" value="1" size="3">
>
> The "1" is just the default in an open text box. How was it hard coded?
> There isn't even a maxlength not that that means much anyways.
Oops, my mistake. I misread his code and assumed type=hidden. Sorry
about that. :-) Please ignore my previous comments about quantity.
Spencer Christensen
Web Engineer
(801)746-7580 x101
www.Backcountry.com
"We use the gear we sell"
More information about the interchange-users
mailing list