[Bug 39836] Patching after edit conflicts (5)
https://bugzilla.wikimedia.org/show_bug.cgi?id=39836 abraham.taheriv...@wikimedia.de changed: What|Removed |Added Status|RESOLVED|VERIFIED --- Comment #8 from abraham.taheriv...@wikimedia.de 2012-11-29 15:34:28 UTC --- Veriefied in Wikidata demo for sprint 24 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39836] Patching after edit conflicts (5)
https://bugzilla.wikimedia.org/show_bug.cgi?id=39836 abraham.taheriv...@wikimedia.de changed: What|Removed |Added Status|ASSIGNED|RESOLVED CC||abraham.taherivand@wikimedi ||a.de Resolution||FIXED -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39836] Patching after edit conflicts (5)
https://bugzilla.wikimedia.org/show_bug.cgi?id=39836 --- Comment #7 from jeb...@gmail.com 2012-11-16 09:56:02 UTC --- Another case that fails involves removal of sitelinks or aliases, where the user has an edit conflict with himself. Load an item in a single browser window. Add a new sitelink that does not exist already in the item, and save the sitelink. Now click on edit for the same sitelink and remove it, and then save the sitelink. You will now get an adit conflict. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39836] Patching after edit conflicts (5)
https://bugzilla.wikimedia.org/show_bug.cgi?id=39836 --- Comment #6 from jeb...@gmail.com 2012-11-15 17:38:39 UTC --- It seems to be one important case missing handling. This is where there is detected an edit conflict with the user itself, but there are some intermediate edits by another user. The only way to handle this seems to be to create an applicable diff for each revision from the base revision up until the last revision, and if the only none-empty diffs are owned by the user itself there is no edit conflict. Otherwise if non-empty applicable diffs are owned by other users, then there is an edit conflict. The proposed solution is heavy and should only be used if all other methods indicates an edit conflict, and if there is indications that user was not the only one to edit since the base revision. To reproduce this failure open two different browsers, with to different users or one non-logged in user. Set both browsers to operate in the same language. Edit the description with browser A, then edit the label two times with browser B. There should now be a warning about an edit conflict in browser B even if the user in that browser only have a conflict with himself. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39836] Patching after edit conflicts (5)
https://bugzilla.wikimedia.org/show_bug.cgi?id=39836 --- Comment #5 from Daniel Kinzler 2012-11-14 20:00:07 UTC --- I344d7681 is merged, story is pending demo. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39836] Patching after edit conflicts (5)
https://bugzilla.wikimedia.org/show_bug.cgi?id=39836 --- Comment #4 from Daniel Kinzler 2012-11-14 19:59:31 UTC --- Fix for a nasty edge case when adding the first alias for a given language: I53d046c1 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39836] Patching after edit conflicts (5)
https://bugzilla.wikimedia.org/show_bug.cgi?id=39836 Daniel Kinzler changed: What|Removed |Added Status|NEW |ASSIGNED CC||daniel.kinz...@wikimedia.de --- Comment #3 from Daniel Kinzler 2012-11-10 11:59:55 UTC --- Implemented in I344d7681 as follows: * modifications must always be based on the revision that is indicated as the base revision. I.e. if ApiModifyEntity gets a baserevid parameter, it must loat that version of the entity, and then modify that, not the current revision. * If the current revision is the base revsision, or there is no base revision, then just save the new content as is (no conflict). * Otherwise, EditPage creates a patch by generating a diff from the base content to the new content (the content provided by the caller). * Then, a "clean" patch is generated by removing all changes from the patch that conflict with (are not applicable to) the current revision. * If the clean patch is different from the original patch, there is a conflict. * If there is a conflict but the current user was the only one to edit since the base revision, the conflict is ignored. Otherwise, saving is aborted. * If saving was not aborted, we now have either a clean patch, or a patch with conflicts against the user's own edits. * Apply that path to the current revision to get a fresh version of the new content which has all the intended changes performed on top of the current revision instead of the base revision. (in git terms, this is a rebase). This should get us a clean result. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39836] Patching after edit conflicts (5)
https://bugzilla.wikimedia.org/show_bug.cgi?id=39836 jeb...@gmail.com changed: What|Removed |Added CC||rainerril...@hotmail.com --- Comment #2 from jeb...@gmail.com 2012-09-10 16:59:02 UTC --- *** Bug 40130 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39836] Patching after edit conflicts (5)
https://bugzilla.wikimedia.org/show_bug.cgi?id=39836 Daniel Kinzler changed: What|Removed |Added Blocks||40130 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 39836] Patching after edit conflicts (5)
https://bugzilla.wikimedia.org/show_bug.cgi?id=39836 denny vrandecic changed: What|Removed |Added Priority|Unprioritized |Normal CC||denny.vrandecic@wikimedia.d ||e Summary|Patching after edit |Patching after edit |conflicts |conflicts (5) -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l