Extra Credit!: why is this code magically changing the #1 to the #10

leadman · Sep 6, 2001

Hi all - here is the puzzle:

My "add to cart button" points to this custom tag code:

<cfparam name="formVal" default="Add to Cart">
<cfif isdefined("form.submit&quot

>
<cfif form.submit eq FormVal>
<cf_addItem
strProdID="#form.addID#"
strProdName="#form.addName#"
curPrice="#form.addPrice#"
quantity="#form.addQty#">
</cfif>
</cfif>
<cfoutput>
<center>
<FORM action="#getfilefrompath(cgi.script_name)#?first=#FirstRow#&searchcatagory=#searchcatagory#&textsearch=#textsearch#&querycount=#querycount#&strProdID=#strProdID#" method="post">
<input type="hidden" name="addName" value="#strprodName#">
<input type="hidden" name="addID" value="#strProdID#">
<input type="hidden" name="addPrice" value="#curPrice#">
<input type="hidden" name="addQty" value="1">
<input type="submit" name="submit" value="#formVal#">
</form>
</center>
</cfoutput>

Notice that the value for the "addQty" field is "1" (the rest of the values are being filled in correctly by the variables). Now - if you submit the above form, it submits to itself and the code at the top (in the cfif tag) goes into effect and the values are passed to the cf_addItem custom tag (addQty should still be 1, right?). That tag contains the following (sorry about the long post but i cant zero in any further):


<cfif isdefined("attributes.strProdID&quot

and isdefined("attributes.strProdName&quot

and isdefined("attributes.curPrice&quot

and isdefined("attributes.quantity&quot

>
<cfscript>
if (not(isdefined("session.cart&quot

)) {
// Check to make sure that the Shopping cart structure exists.
session.cart = structnew();
}

// The item structure we are going to use to store the items in the cart
// is going to have four parts...
// 1. The item id
// 2. The item name
// 3. The price per unit
// 4. The quantity

tempvalue = listtoarray('#attributes.strProdID#,#attributes.strProdName#,#attributes.curPrice#,#attributes.quantity#');

// if the item is not yet in the cart, simply add it to the cart
if (not(structKeyExists(session.cart, attributes.strProdName))) {
StructInsert(session.cart,attributes.strProdname,tempvalue);
}

// if the item is already in the cart, update the item quantity
else {
tempvalue[4]=session.cart[attributes.strProdname][4]+attributes.quantity;
StructUpdate(session.cart,attributes.strProdName,tempvalue);
}
</cfscript>
</cfif>

WHEW! Okay now, the only part of that i dont really get is the "listtoarray" thing so maybe that is wheer the problem is but when you go to view the contents of the shopping cart, the quantity is 10 not 1! Can anyone figure this out?
(just for fun, here is the relevant code for viewing the cart):


<cfif isdefined("session.cart&quot

>
<cfif IsStruct(session.cart)>


<cfset subtotal = 0>

<cfoutput>
<cfloop collection="#session.cart#" item="i">

<cfset subtotal = subtotal + (session.cart[4] * session.cart[3])>
<tr>
<td valign=top><font face="arial" size=2><a href="productpage.cfm?strProdID=#session.cart[1]#">#session.cart[2]#</a></font></td>
<td valign=top>
<table border=0 width=100%>
<tr>
<td><font face="arial" size=3>#session.cart[4]#</font></td>
<form method="post">
<input type="hidden" name="name" value="#session.cart[2]#">

<input type="hidden" name="id" value="#session.cart[1]#">
<input type="hidden" name="price" value="#session.cart[3]#">
<input type="hidden" name="quantity" value="1">
<td align=right>
<input type="submit" name="action" value=" + "> <input type="submit" name="action" value=" - "></td>
</form>
</tr> </table>
</td>

mogilla · Sep 7, 2001

Be sure this code is not in a loop:

<cfparam name="formVal" default="Add to Cart">
<cfif isdefined("form.submit&quot

>
<cfif form.submit eq FormVal>
<cf_addItem
strProdID="#form.addID#"
strProdName="#form.addName#"
curPrice="#form.addPrice#"
quantity="#form.addQty#">
</cfif>
</cfif>

You have 10 items/page so it is adding 10 items of each to cart? Hope that helps

tleish · Sep 7, 2001

Try this:

[COLOR=008080]<table border=1>[/color]

<cfif isdefined("form.action")>
<cf_addItem
strProdID="#form.ID#"
strProdName="#form.Name#"
curPrice="#form.Price#"
quantity="#Trim(FORM.action)##form.quantity#">
</cfif>

[COLOR=666666][/color]
<cfif isdefined("session.cart") AND IsStruct(session.cart)>

[COLOR=666666][/color]
<CFLOCK SCOPE="SESSION" TYPE="READONLY" TIMEOUT="10">
<CFSET REQUEST.cart = SESSION.cart>
</CFLOCK>

[COLOR=666666][/color]
<cfset subtotal = 0>

<cfoutput>
<cfloop collection="#request.cart#" item="i">
<cfset subtotal = subtotal + (request.cart

Code:

i

Code:

4

Code:

* request.cart

Code:

i

Code:

3

Code:

)>
[COLOR=008080]<tr>[/color]
[COLOR=008080]<td valign=top>[/color]
[COLOR=000080]<font face="arial" size=2>[/color]
<a href="productpage.cfm?strProdID=#request.cart

Code:

i

Code:

1

Code:

#">
#request.cart

Code:

i

Code:

2

Code:

#
</a>
[COLOR=000080]</font>[/color]
[COLOR=008080]</td>[/color]
[COLOR=008080]<td valign=top>[/color]
[COLOR=008080]<table border=0 width=100%>[/color]
[COLOR=008080]<tr>[/color]
[COLOR=008080]<td>[/color]
[COLOR=000080]<font face="arial" size=3>[/color]
#request.cart

Code:

i

Code:

4

Code:

#
[COLOR=000080]</font>[/color]
[COLOR=008080]</td>[/color]
[COLOR=000080][COLOR=FF8000]<form method="post">[/color][/color]
[COLOR=000080][COLOR=FF8000]<input type="hidden" name="name"
value="#request.cart

Code:

i

Code:

2

Code:

#">[/color][/color]
[COLOR=000080][COLOR=FF8000]<input type="hidden" name="id"
value="#request.cart

Code:

i

Code:

1

Code:

#">[/color][/color]
[COLOR=000080][COLOR=FF8000]<input type="hidden" name="price"
value="#request.cart

Code:

i

Code:

3

Code:

#">[/color][/color]
[COLOR=000080][COLOR=FF8000]<input type="hidden" name="quantity" value="1">[/color][/color]
[COLOR=008080]<td align=right>[/color]
[COLOR=000080][COLOR=FF8000]<input type="submit" name="action" value=" + ">[/color][/color]
[COLOR=000080][COLOR=FF8000]<input type="submit" name="action" value=" - ">[/color][/color]
[COLOR=008080]</td>[/color]
[COLOR=000080][COLOR=FF8000]</form>[/color][/color]
[COLOR=008080]</tr>[/color]
[COLOR=008080]</table>[/color]
[COLOR=008080]</td>[/color]
[COLOR=008080]</tr>[/color]
</cfloop>
</cfoutput>
</cfif>

[COLOR=008080]</table>[/color] - tleish

Tek-Tips is the largest IT community on the Internet today!

Members share and learn making Tek-Tips Forums the best source of peer-reviewed technical information on the Internet!

Extra Credit!: why is this code magically changing the #1 to the #10

leadman

Programmer

mogilla

IS-IT--Management

tleish

Programmer

Similar threads

Part and Inventory Search

Sponsor