Page MenuHomePhabricator

text was not displayed, but removed due edit conflict
Closed, ResolvedPublic5 Estimated Story PointsBUG REPORT

Description

Steps to Reproduce:

  1. Made an edit in a section
  2. while I made this one, another user edit this section
  3. I got an edit conflict, and merged both texts
  4. while I was merging, another sectionw as edited
  5. edit conflict detected again, but since I've only pressed "edit section", only my section was displayed and I was not able to see a difference

Actual Results:

when I saved the edit, this happend: https://de.wikipedia.org/w/index.php?title=Wikipedia:Administratoren/Anfragen&diff=prev&oldid=191361038&diffmode=source (the second edit was not included, hence removed)

Expected Results:
The template from "Funkruf" should be automatically merged, since this was a different section.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript
thiemowmde subscribed.

This sounds like a serious bug that leads to data loss, hence important. As a first step we will check if this is the TwoColConflict code, or core.

This sounds like a serious bug that leads to data loss, hence important. As a first step we will check if this is the TwoColConflict code, or core.

I looked into that just now. It seems that this is only happening with the TwoColConflict solution screen. The old conflict solving interface deals with the situation as you would expect.

So first of all it's strange that a second edit conflict is triggered although the edit by the third person is happening in a completely different paragraph. It might be related to the way how we merge the results of our solved edit conflict back into the normal workflow.

I also wanted to note that this seems to be independent of the fact that a section was edited. This happens with edits in the same paragraph or more precises probably line.

Change 569242 had a related patch set uploaded (by Andrew-WMDE; owner: Andrew-WMDE):
[mediawiki/extensions/TwoColConflict@master] [WIP] Added carriage returns cause auto-merging to fail

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

thiemowmde added a subscriber: Andrew-WMDE.
thiemowmde set the point value for this task to 5.Feb 4 2020, 2:11 PM

Change 569242 merged by jenkins-bot:
[mediawiki/extensions/TwoColConflict@master] Added carriage returns cause auto-merging to fail

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

awight assigned this task to Andrew-WMDE.
awight moved this task from Demo to Done on the WMDE-QWERTY-Sprint-2020-02-04 board.
awight subscribed.

Smoke tests are happy!