It took me a while to figure out where this was coming from, but found it was here. I was getting a massive amount of hits on undefined/blank.gif after the launch of one of our websites. At first I thought it was some kind of mistake. But then found out it was mostly IE6 users that were causing the hit on blank.gif

After some digging found out it was blank.gif from pngbehave. Now Ive checked the code and everything is placed correctly in the html.

var BlankImgPath = "/sites/all/modules/pngbehave"; is properly defined, but I believe it has something to do with the browser of the client taking too long to process that bit of javascript or the problem of not even having JS enabled.

This is not a major problem, and for the time being I have solved it by hardcoding the url into iepngfix.htc, but it might be interesting to figure out a solution as it unnecessarily causes a load of entries in the logs.

Maybe a possible solution could be to check if blankimg is defined and if not to go for the most logical solution by giving the path /sites/all/modules/...

CommentFileSizeAuthor
#7 pngbehave-450484.patch415 bytesjoelstein

Comments

SeanBannister’s picture

I'm having a similar problem with the Drupal 5 version, I'm having to manually specify IEPNGFix.blankImg if I don't set this the transparency still works but I get a broken image displayed on top of each image that was touched by PNG Behave.

If anyone else is having this problem a temporary fix is to edit iepngfix.htc and change :
IEPNGFix.blankImg = BlankImgPath + '/blank.gif';
to
IEPNGFix.blankImg = '/sites/all/modules/pngbehave/blank.gif';

Keep in mind that /sites/all/modules/pngbehave could be different if you place your modules in another subdirectory such as /sites/all/modules/contrib/

OpenChimp’s picture

I had this same problem in a D6 installation and used the same solution as Sean in order to fix this problem.

rhache’s picture

Used same solution with success as well.

Rene

KerriO’s picture

Same problem, same fix.

halefx’s picture

Same problem, same fix.

mcdruid’s picture

this will also happen if you don't enter anything into the module's settings, but try and reference the iepngfix.htc file in your CSS (as per instructions on module page).

The module checks whether there are any selectors to include or exclude before outputting the JS to set BlankImgPath inside conditional comments. If there are none, it doesn't output the JS and the var is undefined when the .htc code tries to use it.

So a fix for that (without hacking any of the code) is to put something in one of the settings boxes (e.g. "img" in the "CSS Classes/Selectors to Include:" box).

joelstein’s picture

Status: Active » Needs review
StatusFileSize
new415 bytes

Attached is a patch which checks if the BlankImgPath is undefined, and if so, uses the hard-coded "/sites/all/modules/pngbehave" value. Works well for me in IE6 & IE7. Haven't tested any other versions.

mrjeeves’s picture

Assigned: Unassigned » mrjeeves

Thank you. i'll roll that into the 6 version before i branch for 7

mrjeeves’s picture

Version: 6.x-1.7 » 6.x-1.x-dev

committed the patch above, minus the addition of the extra '/blank.gif' to the end of the path.

mrjeeves’s picture

Version: 6.x-1.x-dev » 6.x-1.8
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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