This is an automated email from the ASF dual-hosted git repository.

jleroux pushed a commit to branch release22.01
in repository https://gitbox.apache.org/repos/asf/ofbiz-plugins.git


The following commit(s) were added to refs/heads/release22.01 by this push:
     new 736da3e5b Fixed: Unable to add product to cart for specific product 
(OFBIZ-12105)
736da3e5b is described below

commit 736da3e5bb9aec581efcabf69d0b9efed79bad4f
Author: Jacques Le Roux <jacques.le.r...@les7arts.com>
AuthorDate: Sat Dec 31 10:21:36 2022 +0100

    Fixed: Unable to add product to cart for specific product (OFBIZ-12105)
    
    Issue exist for Specfic product only as 
giant-widget-with-variant-explosion-WG-9943-p
    
    Step to reproduce
    
    1. Open - https://demo-trunk.ofbiz.apache.org/ecomseo
    2. Click on the giant-widget-with-variant-explosion product link.
    3. Select colour and size.
    4. Select unit of measure.
    5. Click on add to cart link.
    
    Actual Result: Unable to product to cart
    Expected Result : Product should be added to cart.
    
    jleroux: error in js console log:
    The use of "getAttributeNode()" is deprecated. Use "getAttribute()" instead.
    This was only a problem in Firefox since a very old version:
    https://bugzilla.mozilla.org/show_bug.cgi?id=690120
    
    Tested without fix on last Edge (ie Chromium) on Win7, works w/ fix too
    
    Thanks: Ashish Sharma for report
---
 ecommerce/webapp/ecommerce/images/productAdditionalView.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ecommerce/webapp/ecommerce/images/productAdditionalView.js 
b/ecommerce/webapp/ecommerce/images/productAdditionalView.js
index a028a436c..ad746d39b 100644
--- a/ecommerce/webapp/ecommerce/images/productAdditionalView.js
+++ b/ecommerce/webapp/ecommerce/images/productAdditionalView.js
@@ -23,7 +23,7 @@ imgView = {
             allAnchors = document.getElementsByTagName('a');
             if (allAnchors.length) {
                 for (var i = 0; i < allAnchors.length; i++) {
-                    if (allAnchors[i].getAttributeNode('swapDetail') && 
allAnchors[i].getAttributeNode('swapDetail').value != '') {
+                    if (allAnchors[i].getAttribute('swapDetail') && 
allAnchors[i].getAttribute('swapDetail').value != '') {
                         allAnchors[i].onmouseover = imgView.showImage;
                         allAnchors[i].onmouseout = imgView.showDetailImage;
                     }
@@ -31,7 +31,7 @@ imgView = {
             }
         }
     },
-    showDetailImage: function() { 
+    showDetailImage: function() {
         var mainImage = document.getElementById('detailImage');
         mainImage.src = document.getElementById('originalImage').value;
         return false;

Reply via email to