First off this is a wonderful update to jquery that has allowed me to do a lot of cool stuff, currently though I'm unable to push this to our production site because it breaks the old javascript in uc_quote.js ... I do not have the required javascript knowledge to fix this issue although I would appreciate some sort of compatibility update or help getting there. The javascript in uc_quote seems rather basic and it isn't a big file, my guess is that it's a fairly simple fix.

Hopefully someone with substantial jquery knowledge - in particular the changes that went on between these version upgrades can take a look.

Thanks in advance.

CommentFileSizeAuthor
#2 uc_quote.js_.txt8.72 KBjohnnyfuchs
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

johnnyfuchs’s picture

This is sorta an old thread, and this might not be the fix you're looking for, but I had some issues with uc_quote.js that I was able to get working after a few tweaks. There was an encoding error that was causing the json formatted rate quote to be unparsable. There is certianly a much cleaner fix available, but it's the best I could do in a few minutes.

johnnyfuchs’s picture

FileSize
8.72 KB

oops, here's the file

Anonymous’s picture

Thanks for this fix, johnnyfuchs. I used your file to replace modules/ubercart/shipping/uc_quote.js, and the checkout page now works with jquery_update enabled.

However, after trying this on a Drupal 6 site, I noticed that this fix is for Drupal 5 (and Ubercart 1). Is your uc_quote.js for Drupal 5 or Drupal 6? (Ubercart 1 or 2?) I'm guessing it's for Drupal 6 (which is good for me), since your file is much more similar to the Drupal 6 version than the Drupal 5 version.

Thanks again, this really seems to fix the issue of jquery_update breaking Ubercart.

Anonymous’s picture

UPDATE: Ah, my bad. Unfortunately, this fix doesn't work for me after all, with a Drupal 6.24 site and jquery_update-6.x-2.0-alpha1 enabled. When I try to check out, I get the dreaded "Javascript must be enabled to view the order total preview." I'd be curious to hear your arrangement that worked.

heddn’s picture

Issue summary: View changes
Status: Active » Closed (won't fix)

Marking this as won't fixed. 5.x is no longer supported/maintained. If this is still an issue in 7.x, then please re-open.