Whitespace in <nowiki>…</nowiki> was normalized (or something) in this edit: https://fr.wikipedia.org/w/?oldid=175073844&diff=prev&diffmode=source
Description
Details
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Open | None | T262408 [TRACKING] Selser issues on talk pages | |||
Resolved | Arlolra | T264028 Whitespace in <nowiki>…</nowiki> normalized or something |
Event Timeline
Also https://fr.wikipedia.org/?diff=178528344 and maybe https://fr.wikipedia.org/?diff=178513936 (a self-closing nowiki removed)
Change 680402 had a related patch set uploaded (by Arlolra; author: Arlolra):
[mediawiki/services/parsoid@master] [WIP] Disable single line context when serializing nowikis
Change 680402 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Disable single line context when serializing nowikis
Change 682690 had a related patch set uploaded (by Arlolra; author: Arlolra):
[mediawiki/vendor@master] Bump wikimedia/parsoid to v0.14.0-a1
Change 682690 merged by jenkins-bot:
[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.14.0-a1 for MW 1.37
Change 693970 had a related patch set uploaded (by C. Scott Ananian; author: Jforrester):
[mediawiki/core@REL1_36] composer: Lock Parsoid version to specific 0.13.0 release
Change 693972 had a related patch set uploaded (by C. Scott Ananian; author: C. Scott Ananian):
[mediawiki/vendor@REL1_36] Bump wikimedia/parsoid to "release version" 0.13.0
Change 693971 had a related patch set uploaded (by C. Scott Ananian; author: C. Scott Ananian):
[mediawiki/vendor@REL1_36] Bump wikimedia/parsoid to "release version" 0.13.0
Change 693972 abandoned by C. Scott Ananian:
[mediawiki/vendor@REL1_36] Bump wikimedia/parsoid to "release version" 0.13.0
Reason:
in favor of I2bfdd568ba24489830fb0d6799524058b3b6d65f
Change 693971 merged by Jforrester:
[mediawiki/vendor@REL1_36] Bump wikimedia/parsoid to "release version" 0.13.0
Change 693970 merged by jenkins-bot:
[mediawiki/core@REL1_36] composer: Lock Parsoid version to specific 0.13.0 release
Hmm,
> echo -e ": test '''<nowiki>\n123</nowiki>''' asdf" > t > git co 925136a > cat t | php bin/parse.php --wt2wt :test '''<nowiki> 123</nowiki>''' asdf > git co master > cat t | php bin/parse.php --wt2wt :test '''<nowiki> 123</nowiki>''' asdf
However, continuing,
> cat t | php bin/parse.php > t.html > sed 's/asdf/asdfe/' < t.html > t2.html > cat t2.html | php bin/parse.php --selser --oldtextfile t --oldhtmlfile t.html : test '''<nowiki> 123</nowiki>''' asdfe
But note that if we started with : test <nowiki>\n123</nowiki> asdf, it would have been fine. The patch does account for some cases of selser reuse.
This is the limitation of https://github.com/wikimedia/parsoid/blob/master/src/Html2Wt/WikitextSerializer.php#L1172-L1185
The nowiki is nested in the idiomatic text element, which doesn't suppress the single line context.
Presumably this isn't too common since it's true of all the other cases there, not just nowikis.
Also, I'm sure it's discussed elsewhere, but it would really help to not be normalizing the entire list when an item is added.
Change 704591 had a related patch set uploaded (by Arlolra; author: Arlolra):
[mediawiki/services/parsoid@master] Add a note about the limitations of suppressing slc
Change 704591 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Add a note about the limitations of suppressing slc