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

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

StatusFileSize
new8.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.