Posted by mcurry on September 7, 2006 at 10:10pm
Jump to:
| Project: | URLify |
| Version: | master |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
it appears that the javascript insertion (in urlify_footer()) is being triggered prematurely, on the /node/add page which gives a list of content types. The inserted script generates a script error because document.getElementById("edit-path") has no properties (it returns null, so dereferencing it to access the .onchange property triggers a script error.)
This is a major problem in IE, as some people get a script error popup on every visit to that page:
---------------------------
Error
---------------------------
A Runtime Error has occurred.
Do you wish to Debug?
Line: 169
Error: 'document.getElementById(...)' is null or not an object
---------------------------
Yes No
---------------------------Less of an issue in firefox, since ff doesn't pop up an error message.
So, it appears that the function _urlify_insert_js() is returning true when it shouldn't.
Comments
#1
inactivist,
Thanks for the report. It should now be fixed in all versions.
#2
Hey, I'm using the latest and greatest Drupal mod you have listed for 4.7, and I am still getting a document.getElementById error in IE when I go to any node/add page:
Error: 'document.getElementById(...)' is null or not an objectIt goes away when I turn off URLify.
Any ideas?
Thanks in advance,
Jeff