The way this module adds the js to the page is a bad idea IMO, and likely not to work in many situations. Consider this common occurrence:

<?php
  drupal_add_js($file);
  $vars['scripts'] = drupal_get_js();
?>

Which would essentially wipe out what the ie6update module has added to the scripts variable.

I would recommend using hook_footer instead. Patch attached.

CommentFileSizeAuthor
ie6update_footer.patch1.06 KBq0rban

Comments

rjbrown99’s picture

rjbrown99 commented

+1

The initial module would not work for me based on the interaction with my theme and other modules. Your hook_footer patch does indeed fix the problem. I agree with this approach and suggest that the module author consider it.

Thank you.

jrglasgow’s picture

jrglasgow commented
Status: Needs review » Fixed

this has been committed

Status: Fixed » Closed (fixed)

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

benone’s picture

benone commented

Has been committed to what version ?
I downloaded last for D6 and it not works.
I need jquery_update and cant disable it.

benone’s picture

benone commented

I added the patch it works now. Just need to know what to do to show it only once ? And it shows vertical scrollbar in ie6 , it doesnt looks good.
Anyway , I woul like to show it to my user only once.
Any ideas ?