Last modified: 2014-08-21 21:25:26 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 T71865, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 69865 - Unclosed nested template adds nowiki on |, but not on the opening {{
Unclosed nested template adds nowiki on |, but not on the opening {{
Status: NEW
Product: Parsoid
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: High normal
: ---
Assigned To: Parsoid Team
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-08-21 18:21 UTC by Derk-Jan Hartman
Modified: 2014-08-21 21:25 UTC (History)
4 users (show)

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


Attachments

Description Derk-Jan Hartman 2014-08-21 18:21:50 UTC
https://fr.wikipedia.org/w/index.php?title=Eliaquim_Mangala&diff=106681571&oldid=106681205

One, can this be improved to detect the illogical unmatched transaction and warn about it ?

2nd, perhaps it can nowiki the {{ as well (I suspect this might be difficult...).

3rd, Perhaps the template editor UI itself can check for unmatched {{ }} in the field ?
Comment 1 James Forrester 2014-08-21 20:37:46 UTC
That's very odd – I can't reproduce the | getting converted to <nowiki>|</nowiki> with my own edits… Unclear if this is a Parsoid issue, possibly?

(When we have DOM-editing of templates' parameters, bug 50355, this will be moot.)
Comment 2 ssastry 2014-08-21 21:25:26 UTC
Yes, this is a bug in Parsoid's nowiki code for template args ... it should have added a nowiki around the whole string or also on the opening "{{".

Parsoid could possibly raise warnings about this to the client (VE in this case) ... but, not all clients might handle a non-standard response and do a back-and-forth negotiation. So, a little less unsure about it.

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


Navigation
Links