Posted by ZoeN on November 30, 2010 at 11:02pm
2 followers
Jump to:
| Project: | Ubercart Fee |
| Version: | 6.x-1.0-beta2 |
| Component: | Code |
| Category: | bug report |
| Priority: | major |
| Assigned: | Unassigned |
| Status: | needs review |
Issue Summary
In Internet Explorer only, we were getting an infinite loop situation on the checkout page, that resulted in fees and updated totals never being calculated, and the grey "loading" bar under Payment Options never going away.
Our JS guy says: "After a lot of debugging I found out that when using the Javascript For ... In statement the stupid IE browser goes inside the loop even if the array is empty.
Unfortunately the file uc_fee.js has conditionals inside these loops to continue the calculation process of the cart. The variables continued being set as true causing an endless loop."
Here's our fix. It just wraps a "For...In" loop in an If that verifies there is something to loop through.
| Attachment | Size |
|---|---|
| fee_IE_compatible.patch | 1.3 KB |
Comments
#1
Bumping up the priority. Ran into the same issue and ended up creating the same patch as what is posted.
This bug pretty much rendered the checkout page useless on IE and was difficult to track down (no Firebug in IE) since there are so many different AJAX requests on the checkout page in conjunction with other modules.