Can't paste wikitext references
Closed, ResolvedPublic8 Estimated Story Points
Actions

Description

Steps to reproduce:

Copy this:

<ref name="Example">Burci, Gian Luca; Vignes, Claude-Henri (2004-01-01). [https://books.google.com/books?id=Xou_nD9jJF0C ''World Health Organization'']. Kluwer Law International. ISBN 9789041122735. Pages 15–20.</ref>

Open a page in the visual editor and paste it in. Make no other edits.

Wait for the wikitext to convert. See spurious claim that it doesn't exist when you select the ref number:

Screen Shot 2016-05-02 at 6.55.38 PM.png (202×639 px, 43 KB)

See error when you try to review your changes or save the page: "Could not start the review because your revision matches the latest version of this page."

Details

	Subject	Repo	Branch	Lines +/-
	Wikitext paste: Discard autoGenerated items after conversion	mediawiki/extensions/VisualEditor	wmf/1.27.0-wmf.23	+61 -11
	Wikitext paste: Discard autoGenerated items after conversion	mediawiki/extensions/VisualEditor	master	+60 -10

Customize query in gerrit

Related Objects

Mentioned In: T375306: Visual Editor sometimes adds "2" to reference names, creating reference errors
T125034: Do not change existing ref names, not even to add a gratuitous "2" to the end of each ref name.
Mentioned Here: T125034: Do not change existing ref names, not even to add a gratuitous "2" to the end of each ref name.
T88290: Update data-mw encoding for <ref> tags to point to the HTML content in the <references /> output rather than duplicating it

Event Timeline

• Whatamidoing-WMF created this task.May 3 2016, 2:10 AM

Restricted Application added a project: VisualEditor. · View Herald TranscriptMay 3 2016, 2:10 AM

Restricted Application added subscribers: Zppix, Aklapper. · View Herald Transcript

• Whatamidoing-WMF mentioned this in T125034: Do not change existing ref names, not even to add a gratuitous "2" to the end of each ref name..May 3 2016, 2:11 AM

• Elitre subscribed.May 3 2016, 10:10 AM

Parsoid have stopped inlineing the HTML for references, and we deliberately throw away the auto-generated reference list...

In T134228#2259824, @Esanders wrote:

Parsoid have stopped inlineing the HTML for references, and we deliberately throw away the auto-generated reference list...

I haven't looked more at what this is about, but do you know if this this new?

Esanders renamed this task from Can't paste wikitext for refs into Firefox to Can't paste wikitext references.May 3 2016, 2:12 PM

Esanders triaged this task as High priority.

Change 286656 had a related patch set uploaded (by Esanders):
Wikitext paste: Discard autoGenerated items after conversion

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

gerritbot added a project: Patch-For-Review.May 3 2016, 2:12 PM

We used to be able to get the references contents from body.html, but not we only get body.id - which means our approach of filtering 'autoGenerated' items from the imported HTML throws away the only copy of the reference body.

The above patch moves the 'autoGenerated' filtering to after the HTML has been converted, solving the issue.

In T134228#2260129, @Esanders wrote:

We used to be able to get the references contents from body.html, but not we only get body.id - which means our approach of filtering 'autoGenerated' items from the imported HTML throws away the only copy of the reference body.

Yes, providing only body.id instead of body.html for <ref> tags has been in place for a long time now .. since T88290: Update data-mw encoding for <ref> tags to point to the HTML content in the <references /> output rather than duplicating it.

This was reported in Google Chrome 49.0.2623.112 and I reproduced it in Firefox 45 but not Safari 9. For the problems in Safari, see T125034 instead.

Esanders merged a task: T125034: Do not change existing ref names, not even to add a gratuitous "2" to the end of each ref name..May 4 2016, 12:40 PM

Esanders added subscribers: Jdforrester-WMF, TerraCodes, StudiesWorld.

Jdforrester-WMF moved this task from To Triage to TR0: Interrupt on the VisualEditor board.May 4 2016, 1:26 PM

Jdforrester-WMF set the point value for this task to 8.

Jdforrester-WMF added a subtask: T134382: Design and provide a way for arbitrary special characters/entities to be shown to users in a special way denoting their status.May 4 2016, 1:35 PM

Change 286863 had a related patch set uploaded (by Jforrester):
Wikitext paste: Discard autoGenerated items after conversion

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

Jdforrester-WMF removed a subtask: T134382: Design and provide a way for arbitrary special characters/entities to be shown to users in a special way denoting their status.May 4 2016, 3:09 PM

Change 286656 merged by jenkins-bot:
Wikitext paste: Discard autoGenerated items after conversion

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

ReleaseTaggerBot added a project: MW-1.28-release (WMF-deploy-2016-05-10_(1.28.0-wmf.1)).May 4 2016, 4:01 PM

Jdforrester-WMF closed this task as Resolved.May 4 2016, 4:03 PM

Jdforrester-WMF assigned this task to Esanders.

Jdforrester-WMF edited projects, added MW-1.27-release (WMF-deploy-2016-05-03_(1.27.0-wmf.23)); removed Patch-For-Review.

Change 286863 merged by jenkins-bot:
Wikitext paste: Discard autoGenerated items after conversion