Closed Bug 779838 Opened 12 years ago Closed 12 years ago

Attr.firstChild returning NULL

Categories

(Core :: DOM: Core & HTML, defect)

14 Branch
defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: merkinman, Unassigned)

References

Details

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:14.0) Gecko/20100101 Firefox/14.0.1 Build ID: 20120713134347 Steps to reproduce: Visited http://www.power1051fm.com/pages/onair/breakfast-club/ that worked in previous versions of Firefox as well as other browsers Actual results: Images did not load and the Error Console displayed "Error: TypeError: items[i].getElementsByTagNameNS("*", "thumbnail")[0].attributes.getNamedItem("url").firstChild is null Source File: http://www.power1051fm.com/pages/onair/breakfast-club/ Line: 470" Expected results: The thumbnails display like they do in other browsers as well as previous versions of Firefox
OS: Windows 7 → All
Hardware: x86_64 → All
Regression window Good: http://hg.mozilla.org/mozilla-central/rev/9894cd999781 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:14.0) Gecko/20120403 Firefox/14.0a1 ID:20120403002322 Bad: http://hg.mozilla.org/mozilla-central/rev/c410b2d6d570 Mozilla/5.0 (Windows NT 6.1; WOW64; rv:14.0) Gecko/20120403 Firefox/14.0a1 ID:20120403002822 Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=9894cd999781&tochange=c410b2d6d570 Triggered by: af7362ead6e5 Ms2ger Bug 737122 - Attr nodes shouldn't have children; r=sicking
Blocks: 737122
Status: UNCONFIRMED → NEW
Component: Untriaged → DOM: Core & HTML
Ever confirmed: true
Product: Firefox → Core
Yep, this is intentional. Instead of var rss_thumb = items[i].getElementsByTagNameNS('*','thumbnail')[0].attributes.getNamedItem('url').firstChild.nodeValue; the site should use var rss_thumb = items[i].getElementsByTagNameNS('*','thumbnail')[0].getAttribute('url');
Summary: getElementsByTagNameNS returning NULL → Attr.firstChild returning NULL
Webpage is updated with new code now. Working in FIrefox 13 and 14, as well as other browsers tested.
(In reply to :Ms2ger (Back and backlogged) from comment #2) > Yep, this is intentional. Marked as WONTFIX
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.