Got Ubercart, uc_option_image, and uc_aac working together under firefox (woo-hoo!).

Unfortunately, they don't quite work under Internet Explorer 6 or 7, at least for the two computers I've tested.

If there are fewer than two attributes with price-changing options, both add-ons work correctly.

But if there is more than one price-changing attribute, neither add-on works.

Does anybody have some pointers on how I can troubleshoot this on a Windows / IE platform?

Test pages at http://www.kbsystem.com/tripod/kb1003-giant and http://www.kbsystem.com/tripod/kb1a-field

(KB is "Kay and Bill"; Kay is my grandma. She's a super-cool lady -- in her younger days she was one of the first woman aerospace engineers to work on the Space Shuttle.)

Comments

cyu’s picture

cyu commented
Assigned: Unassigned » cyu

Can you try grabbing the latest dev snapshot, there are some IE related fixes in there? I need to address a compatibility issue with Ubercart Beta2, so I'll take a look at this as well.

cyu’s picture

cyu commented
Status: Active » Postponed (maintainer needs more info)

I took out some js that had been flaky on IE, so let me know if you still have problems after installing this latest release combined with UC Beta 2.

pillarsdotnet’s picture

pillarsdotnet commented

doublechecked the latest CVS -- is the snapshot newer than CVS checkout?

cyu’s picture

cyu commented
Version: 6.x-1.x-dev » 6.x-1.0

I'm looking at your demo site and see $(this).unbind("click.uc_aac"); which should now be $(this).unbind("click"); in both the latest snapshot and the 6.x-1.0 release.

cyu’s picture

cyu commented
pillarsdotnet’s picture

pillarsdotnet commented
Status: Postponed (maintainer needs more info) » Closed (fixed)

okay, CVS lies.

Running "cvs update" doesn't change $(this).unbind("click.uc_aac"); to $(this).unbind("click");

Running "cvs diff" doesn't show that discrepancy between my copy and the CVS repository.

Checking out a fresh copy in a new directory and comparing shows that the freshly-checked-out copy has "click" where my copy has "click.uc_aac"

Manually making the changes and then running "cvs diff" again shows this:

root@pillars:~/modules/uc_aac# cvs diff
cvs diff: Diffing .
Index: uc_aac.module
===================================================================
RCS file: /cvs/drupal-contrib/contributions/modules/uc_aac/uc_aac.module,v
retrieving revision 1.5.2.3
diff -u -p -r1.5.2.3 uc_aac.module
--- uc_aac.module       2 Dec 2008 22:07:10 -0000       1.5.2.3
+++ uc_aac.module       17 Jan 2009 16:35:04 -0000
@@ -123,8 +123,8 @@ function theme_uc_aac_js($nid, $price, $
         if (element.name && element.name.match("attributes")) {
           // Needed because off odd click() behavior in IE with Jquery Update
           // See http://drupal.org/node/336350
-          $(this).unbind("click.uc_aac");
-          $(this).bind("click.uc_aac", function () {
+          $(this).unbind("click");
+          $(this).bind("click", function () {
             perform_ajax_calculation_'. $nid .'();
           });
         }
@@ -212,14 +212,14 @@ function theme_uc_aac_js($nid, $price, $
                 }
               }
             }
-            var currentsellHTML= $("#node-'. $nid .' .sell_price").html();
-            var currentdisplayHTML = $("#node-'. $nid .' .display_price").html();
+            var currentsellHTML= $("#node-'. $nid .' .sell-price").html();
+            var currentdisplayHTML = $("#node-'. $nid .' .display-price").html();
             updated_price = json_results.updated_price;
             if (currentdisplayHTML) {
-              $("#node-'. $nid .' .display_price").html(currentdisplayHTML.replace(replace_price_'. $nid .', updated_price));
+              $("#node-'. $nid .' .display-price").html(currentdisplayHTML.replace(replace_price_'. $nid .', updated_price));
             }
             if (currentsellHTML) {
-              $("#node-'. $nid .' .sell_price").html(currentsellHTML.replace(replace_price_'. $nid .', updated_price));
+              $("#node-'. $nid .' .sell-price").html(currentsellHTML.replace(replace_price_'. $nid .', updated_price));
             }
             replace_price_'. $nid .' = updated_price;
           }

So the lesson I've learned today, kiddies, is to never trust CVS for my updates.

(grumble, grumble...)

(testing...)

Yup, works fine.