Add to Cart produces more than 1 item... Help!

[mrdellano]

Here's some background.

 

I first encountered this problem when I tried to add a quantity box to product_info.php. No matter what number I put in the box, it would add different multiples, depending on the product.

 

It was driving me nuts, so I removed the quantity box and it was working normal again. When I clicked on Add to Cart, it would just add 1 product to the cart.

 

Then I installed Ultimate SEO URLs and the problem returned. When I click on Add to Cart, it adds 16 or 23 at a time, depending on the product.

 

I did not alter product_info.php.

 

I am not sure what is causing this to happen. Is it something in MySQL? I didn't alter anything when installing USEOU. It just had me run the following:

 

ALTER TABLE `categories_description` ADD `categories_seo_url` VARCHAR( 100 ) NOT NULL ;

ALTER TABLE `products_description` ADD `products_seo_url` VARCHAR( 100 ) NOT NULL;

 

Does anyone know how to fix this problem?

 

Thanks!

 

- Rich

[mrdellano]

I should also note that this only happens when I add to cart from product_info.php.

 

If I click on Buy Now from the Category list, or any list (like Search results), it works just fine.

[mrdellano]

Well, I don't know if this will screw anything up, but I conjured up a quick fix.

 

In includes/application_top.php, I changed :

 

case 'add_product' :	if (isset($HTTP_POST_VARS['products_id']) && is_numeric($HTTP_POST_VARS['products_id'])) {
							$cart->add_cart($_POST['products_id'], $cart->get_quantity(tep_get_uprid($_POST['products_id'], $_POST['id']))+$_POST['products_id'], $_POST['id']);
						  }
						  tep_redirect(tep_href_link($goto, tep_get_all_get_params($parameters)));
						  break;

 

to the following:

 

case 'add_product' :	if (isset($HTTP_POST_VARS['products_id']) && is_numeric($HTTP_POST_VARS['products_id'])) {
							$cart->add_cart($HTTP_GET_VARS['products_id'], $cart->get_quantity($HTTP_GET_VARS['products_id'])+1);
						  }
						  tep_redirect(tep_href_link($goto, tep_get_all_get_params($parameters)));
						  break;

 

Basically, it does the same thing as "case 'buy_now'" which, as I mentioned before, worked just fine.

 

If anyone can find a reason why this fix is a bad idea, then let me know. Otherwise, I guess I'll just stick with it for now.

 

Hope this helps anyone experiencing the same problem!