Page MenuHomePhabricator

<references/> / <ref> bugginh <nowiki> when used in subpages
Closed, ResolvedPublic

Description

Author: koneko

Description:
The problem happened today in fr:'s village pump

everything like <nowiki>{{a_template}}</nowiki> in the pump's daily subpages
suddenly started to appear
UNIQ254abebc3bc90b69-nowiki7d65265d39e5d81700000001a_templateUNIQ254abebc3bc90b69-nowiki7d65265d39e5d81700000002
when called from the main pump page (to be exact, any {{ or }} inside a <nowiki>
balise was changed into these strange things)

After nearly half a hour of various tests, the reason appear to be the
<references/> and <ref></ref> balises that where used in one of the recent
messages. (the page work fine again since I deleted theses things from today's
pump subpage)

One told be there was a known probleme when a </ref> is forgot, but that's not
the case here (see the diff from the original faulty version to the "safe"
version :
http://fr.wikipedia.org/w/index.php?title=Wikip%C3%A9dia%3ALe_Bistro%2F7_avril_2006&diff=6570918&oldid=6569942
)


Version: unspecified
Severity: major
URL: http://fr.wikipedia.org/wiki/Wikip%C3%A9dia:Le_bistro

Details

Reference
bz5499

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 9:10 PM
bzimport added a project: MediaWiki-Parser.
bzimport set Reference to bz5499.

koneko wrote:

hmm correction : it's when the daily subpage itself has a template containing a
<nowiki> +}}/{{ something.

I've not tested when the nowiki is directly on the subpage.

folengo wrote:

"Balise", a French word, means "tag".

See a reproduction of the bug at [[:fr:Utilisateur:Teofilo-
Folengo/test2]] http://tinyurl.com/gsgd3

koneko wrote:

oops, I've spoken frenglish again -_- sorry

foenyx wrote:

probably same cause than the bug 2172

Fixed on trunk, r13553.

When Cite.php does its rendering through a callback to the parser, it
requests not to clear state; however the tag-strip state was getting cleared
regardless. I've removed the offending line and it now appears to work
properly. No parser test cases are broken, so hopefully this won't break
anything else. :)