There is a minor bug in the Cloud Zoom JQuery plugin used to zoom into product images on the product page.
The plugin uses a div to trap mouse movement. So that it works in IE, the code puts a background image on the div. But it points to a dummy URL url(.) which results in 404 missing page errors filling up the log.
You can simply replace the code for the mousetrap div:

$mouseTrap = jWin.parent().append(format("<div class='mousetrap' style='background-image:url(\".\");
z-index:999;position:absolute;width:%0px;height:%1px;left:%2px;top:%3px;\'></div>", 
sImg.outerWidth(), sImg.outerHeight(), 0, 0)).find(':last');

with

$mouseTrap = jWin.parent().append(format("<div class='mousetrap' style='background-image:url(\"%0\");
z-index:999;position:absolute;width:%1px;height:%2px;left:%3px;top:%4px;\'></div>",
'data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7',
sImg.outerWidth(),sImg.outerHeight(),0,0)).find(':last');

This simply puts an inline base64 encoded 1x1 transpatent gif as the background and resolves the issue.

I have written some more about this here: http://technology.blue-bag.com/commerce-detective-cloud-zoom-jquery-plug...

Hope that helps.

Comments

guy_schneerson’s picture

guy_schneerson CreditAttribution: guy_schneerson commented

would be good to get this into Kickstart as this can easily fill the error log over time and makes it harder to spot real issues.

bojanz’s picture

bojanz CreditAttribution: bojanz commented
Status: Needs review » Postponed (maintainer needs more info)

I think this got fixed with the Cloud Zoom update.
Please test the latest -dev (or 2.0, we're tagging it in a few hours).

DYdave’s picture

DYdave CreditAttribution: DYdave commented

bojanz,

I don't understand exactly what you meant by:

Please test the latest -dev (or 2.0, we're tagging it in a few hours).

because it seems obviously there are going to be some licensing issues, since 2.x versions of cloud_zoom are paid.
I have tried the trial version, which indeed fixes the problem, but displays a very ugly Unlicensed version message on the image while moving the mouse over zoom area.

I think this should probably be followed up with in #1473594: Mouse trap bakground image produces 404.

I might have missed something with the file download, so I would appreciate if you could correct me if I was wrong, but obviously, purchasing the Cloud Zoom license to fix this issue is probably not going to work for everyone.

Thanks again very much for your comments and help on this.
Cheers!

iAugur’s picture

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

The latest dev release of Kickstart (7.x-2.x-dev) uses the latest dev version (7.x-1.1+1-dev) of the CloudZoom module which uses the latest free version of the JS library (v1.0.3) - available here http://www.professorcloud.com/downloads/cloud-zoom.1.0.3.zip
Both this and the latest v2 releases have been fixed to use a blank image which solves the 404 issue.
I'll mark as fixed.

DYdave’s picture

DYdave CreditAttribution: DYdave commented
Status: Closed (fixed) » Fixed

Hi iAugur,

Thanks again very much for following up with this.

I am running the exact same modules and versions of cloud zoom JS that you mentioned, with caching and JS aggregation enabled.
Unfortunately I'm still getting this issue, could that come from caching and JS aggregation enabled.

Am I the only one to have this problem?
Any help on this would be greatly appreciated.

Thanks again very much in advance.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

David Stosik’s picture

David Stosik CreditAttribution: David Stosik commented
Title: Cloud Zoom JQuery Plugin causes 404 missing page errors » Cloud Zoom JQuery Plugin causes 404 missing page errors when JS is aggregated
Version: 7.x-2.0-rc2 » 7.x-2.x-dev
Component: User interface » Code
Priority: Minor » Normal
Status: Closed (fixed) » Active

Hi,
This bug is not completely fixed. Please have a look to DYdave's comment #9 in #1473594: Mouse trap bakground image produces 404.
Basically, turning JS aggregation produces 404 errors again.

Regards,
David

nonstopunter’s picture

nonstopunter CreditAttribution: nonstopunter commented

We are getting /scanimageurl appended at the end of URL randomly and redirecting to 404 page error and it seems its happening for IE 8 and 9, has anyone encountered similar problem ? cant seem to find the cause of the problem .

vasike’s picture

vasike
Location Ramnicu Valcea
CreditAttribution: vasike commented
Status: Active » Fixed

i can't reproduce it (#7 & #8) using the latest Commerce Kickstart. Maybe the Drupal upgrades helped with the JS aggregation.

DYdave’s picture

DYdave CreditAttribution: DYdave commented
Status: Fixed » Active

Hi vasike,

Just a quick follow up on this issue:
Right after your update, I popped out a simplytest.me demo of the commerce_kickstart-2.x latest dev branch.

After installing, I went straight to: admin/config/development/performance
to turn on JS and CSS aggregation (as well as cache for anon).

I then browsed a product page: /tops/drupal-commerce-hoodie
where I found with no surprise the 404 for blank.png was still there, requested at:
http://sfa5b4b9debf23b7.s3.simplytest.me/sites/default/files/js/blank.png
(URL is temporary, just to show you it is still requesting the same file from the same problematic location)

The problem is still exactly the same and if you took a closer look at #1473594: Mouse trap bakground image produces 404 you might find as well it has very little (nothing actually) to do with Commerce Kickstart itself. This is more an issue related with the Cloud Zoom module, which I am afraid would be difficult to solve entirely/properly (more details in related issue).

In other words, this problem will exist in Commerce Kickstart as long as it hasn't been fixed in Cloud Zoom (or perhaps someone could come up with a workaround?).

Feel free to let me know if I missed or overlooked anything while testing, I would surely be glad to try testing again in more details.
Thanks again very much for your testing, comments, feedbacks and reviews.
Cheers!

vasike’s picture

vasike
Location Ramnicu Valcea
CreditAttribution: vasike commented

@DYdave
Hi, you're so right. i totally misunderstood the issue. thank you
Cheers

vasike’s picture

vasike
Location Ramnicu Valcea
CreditAttribution: vasike commented
Issue summary: View changes

minor typo fix

mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented
Issue summary: View changes
Status: Active » Postponed
Related issues: +#1473594: Mouse trap bakground image produces 404, +#2193489: Change library source to github fork

Postponing. Blocked on

  1. #1473594: Mouse trap bakground image produces 404
  2. #2193489: Change library source to github fork
mglaman’s picture

mglaman
Primary language English
Location WI, USA
CreditAttribution: mglaman at Centarro commented
Status: Postponed » Closed (won't fix)

There is nothing we can do here. The Cloud Zoom does not have active development and the license is no proprietary.