Maniphest T201210

VisualEditor not switching to edit conflict interface
Closed, ResolvedPublic
Actions

Description

This bug was reproduced on the English Wikipedia.

Steps to reproduce:

Open an existing page for edition using the Visual Editor (ie: a test page in your user space) in browser 1.
Using an anonymous user on browser 2 (an incognito window for example) make edits to that page so as to create an edit conflict.
Save the page on browser 2.
With browser 1, save the page. The VisualEditor will inform you that a conflict has been detected.
Press the button for conflict resolution.

Actual result:

The user gets taken to a wikicode editor for a previous version of the page.
An error message informing that "Some parts of the edit form did not reach the server; double-check that your edits are intact and try again." is displayed
When "Publish changes" is pressed, the user is then taken to the conflict resolution interface.

Expected result:

That the user be shown the conflict resolution interface right away.

Related Objects
Search...

		Status	Subtype	Assigned	Task
		Resolved		DLynch	T201210 VisualEditor not switching to edit conflict interface
		Resolved		DLynch	T199554 Switching between visual editor and source mode gives error message

Event Timeline

Tinss created this task.Aug 3 2018, 10:08 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 3 2018, 10:08 PM

Legoktm edited projects, added MediaWiki-Page-editing, VisualEditor; removed MediaWiki-General.Aug 4 2018, 12:06 AM

I can reproduce this. The "Some parts of the edit form did not reach the server; double-check that your edits are intact and try again." message is troubling. This is bad if it is affecting a lot of edit summaries. Needs further investigation.

Is this related to T199554? I note that this error message (incompleteForm) can be triggered by wpUltimateParam (EditPage.php line 966), so perhaps they are related? Pinging @DLynch, @matmarex

The incompleteForm message part will be fixed when my patch on T199554 is merged. I'm not sure if that's the sole part that's holding up going to the conflict resolution interface, though -- I'll double-check it.

This works correctly for me now (testing on https://en.wikipedia.beta.wmflabs.org/wiki/T201210). After causing an edit conflict, then trying to save in VE, I was redirected directly to this screen: