Re: [ic] Any problems with this small hunk of code?

[Spencer Christensen <suppressed>]

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"
_______________________________________________
interchange-users mailing list
suppressed
http://www.icdevgroup.org/mailman/listinfo/interchange-users