Last modified: 2014-05-05 16:30:44 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T66747, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 64747 - More foster-parenting scenario induced template encapsulation failures
More foster-parenting scenario induced template encapsulation failures
Status: RESOLVED FIXED
Product: Parsoid
Classification: Unclassified
DOM (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: ssastry
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-05-02 16:04 UTC by ssastry
Modified: 2014-05-05 16:30 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description ssastry 2014-05-02 16:04:15 UTC
Copied over from bug 64022. Following errors logged in production after y'day's deploy:

[fatal][commonswiki:Commons:Village_pump/Archive/2012/04][oldid:89940097] Cannot read property 'nodeName' of null
[fatal][enwiki:User:The_Obento_Musubi/Sandbox/Archive_3][oldid:605399257] Cannot read property 'nodeName' of null
[fatal][fawiki:الگو:Cmbox][oldid:8301284] Cannot read property 'nodeName' of null
[fatal][frwiki:Discussion:Batna][oldid:85943356] Cannot read property 'nodeName' of null
[fatal][frwiki:Discussion:Vague_belge_d'ovnis][oldid:103426136] Cannot read property 'nodeName' of null
[fatal][itwiki:Discussioni_progetto:Aviazione/Archivio22][oldid:49763882] Cannot read property 'nodeName' of null
[fatal][itwiki:Discussioni_progetto:Aviazione/Template/Archivio2][oldid:61182245] Cannot read property 'nodeName' of null
Comment 1 ssastry 2014-05-02 16:11:06 UTC
The nodeName null errors are actually template encapsulation crashers. Reduced test case to reproduce:
----
<table>
{{#if:|
<td>foo</td>
}}
</table>
----

Discovered that this testcase by itself is not sufficient to deal with failures in those pages. But, a good start.
Comment 2 Gerrit Notification Bot 2014-05-02 18:09:15 UTC
Change 131093 had a related patch set uploaded by Subramanya Sastry:
WIP: (Bug 64747): Bug fixes computing transclusion dom-range

https://gerrit.wikimedia.org/r/131093
Comment 3 Gerrit Notification Bot 2014-05-02 23:59:22 UTC
Change 131093 merged by jenkins-bot:
(Bug 64747): Bug fixes computing transclusion dom-range

https://gerrit.wikimedia.org/r/131093

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links