[jira] Issue Comment Edited: (OFBIZ-2012) On change quantity to zero randomly no itemes have been deleted from cart in One Page Check out process.

["Sumit Pandit (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/OFBIZ-2012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12641752#action_12641752 ] 

sumitp edited comment on OFBIZ-2012 at 10/22/08 5:58 AM:
---------------------------------------------------------------

Jyotsna, 

I saw your patch. Here you replaced the name of "update_${cartLineIndex}" by "update${cartLineIndex}". 

And inside JS file in query string, you explicitly created it as follows : 

+  qtyParam = "update_" + itemIndex +"="+qtyElement.value;
+  var formValues = $('cartForm').serialize() + '&' + qtyParam;

So here what message i got from here is that, you want to send only that item which need to update, ignore remaining.

If the purpose is same which i described above then our code can like this :

+        var formValues =   "update_" + itemIndex +"="+qtyElement.value;

After applying your patch I have an UI issue. 
when i delete some item the screen is not showing proper. 
You can find it in attachment.

      was (Author: sumitp):
    Jyotsna, 

I sow your patch. Here you replaced the name of "update_${cartLineIndex}" by "update${cartLineIndex}". 

And inside  JS file in querry string, you explicitly create it as follows : 

+  qtyParam = "update_" + itemIndex +"="+qtyElement.value;
+  var formValues = $('cartForm').serialize() + '&' + qtyParam;

So here what message i got from here is that, you want to send only that item which need to update, ignore remaining.

If the purpose is same which i described above then our can write like :

+        var formValues =   "update_" + itemIndex +"="+qtyElement.value;

After applying your patch I have an UI issue. 
when i delete some item the screen is not showing proper. 
You can find it in attachment.
  
> On change quantity to zero randomly no itemes have been deleted from cart in One Page Check out process.
> --------------------------------------------------------------------------------------------------------
>
>                 Key: OFBIZ-2012
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2012
>             Project: OFBiz
>          Issue Type: Bug
>          Components: ecommerce
>            Reporter: Rishi Solanki
>            Priority: Minor
>         Attachments: Picture 1.png, RemoveItems.patch, UpdteQtyFix_js.patch
>
>
> To regenerate the error :
> Add more than 3 product to cart then change individual qty to 0. Not in serial First delete the 2nd product and then 3rd.
> Here you will get the problem of not deleting the product from cart from the 2nd time. You may check it by refreshing the page.
> Solution :
> When you click on remove item link it will call the function removeItem of chekoutProcess.js. It will work fine for it. Now you need to one thing call the same function when you get the zero as quantity.
> Finally you need some customization in the removeItem as well in cartItemQtyChanged.
> Regards
> --
> Rishi Solanki

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.