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.

CommentFileSizeAuthor
fee_IE_compatible.patch1.3 KBzoen

Comments

omniverse’s picture

omniverse commented
Priority: Normal » Major

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.

deggertsen’s picture

deggertsen commented
Status: Needs review » Reviewed & tested by the community

Works for me! Thanks!

scott859’s picture

scott859 commented
Issue summary: View changes

I am getting the same issue using the 6.x-1.x-dev version - but it appears in all browsers that I've tested:

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.

Although I am not sure of the cause, can this patch be applied to the dev version as well?