Last modified: 2014-07-15 20:33:48 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 T55571, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 53571 - Parsoid adds <nowiki>...</nowiki> when a single apostrophe follows a link in a bold or italic sentence
Parsoid adds <nowiki>...</nowiki> when a single apostrophe follows a link in ...
Status: RESOLVED FIXED
Product: Parsoid
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Normal normal
: ---
Assigned To: Gabriel Wicke
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-08-29 23:26 UTC by Chris McKenna
Modified: 2014-07-15 20:33 UTC (History)
3 users (show)

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


Attachments

Description Chris McKenna 2013-08-29 23:26:22 UTC
When a single apostrophe follows a link in the middle of a section of bold or italic text, Parsoid unnecessarily nowikis everything between the end of the link and the end of the bold or italic text. i.e.
''Italic [[link]]'s text'' → ''Italic [[link]]<nowiki>'s text</nowiki>''

In my sandbox at least this only seems to happen when the sequence is created in VE though, in the example diff the sentence above the addition, which contains the same markup, but was preexisting, was not touched. https://en.wikipedia.org/w/index.php?title=User:Thryduulf/sandbox3&oldid=570377353&veaction=edit

The change at line 89 of this article-space diff makes me wonder whether it sometimes happens to existing text too https://en.wikipedia.org/w/index.php?title=42_%28number%29&diff=570700339&oldid=570125224

It only occurs when the apostrophe follows a link, i.e. it only occurs when ]]' occurs in a bold or italicised section.
Comment 1 Arlo Breault 2014-07-15 20:33:48 UTC
This seems fixed now,

> echo "<i>Italic <a href='./Link'>link</a>'s text</i>" | node tests/parse --html2wt
> ''Italic [./Link link]'s text''

Also, tested with VE:

https://www.mediawiki.org/w/index.php?title=User:Arlolra/Sandbox/apos&oldid=1068500

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


Navigation
Links