[Bug 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 Andre Klapper changed: What|Removed |Added Keywords|patch-need-review |patch-reviewed Status|ASSIGNED|NEW Assignee|j...@compwhizii.net |wikibugs-l@lists.wikimedia. ||org --- Comment #8 from Andre Klapper --- (In reply to John Du Hart from comment #2) > Created attachment 8910 [details] > Race condition check Result from uploading patch: patch -p0 < patch patching file includes/specials/SpecialMovepage.php Hunk #1 FAILED at 34. Hunk #2 succeeded at 93 with fuzz 1 (offset 7 lines). Hunk #3 succeeded at 136 with fuzz 2 (offset -57 lines). Hunk #4 FAILED at 336. Hunk #5 succeeded at 529 (offset 127 lines). Hunk #6 FAILED at 472. 3 out of 6 hunks FAILED -- saving rejects to file includes/specials/SpecialMovepage.php.rej patching file languages/messages/MessagesEn.php Hunk #1 succeeded at 3529 with fuzz 2 (offset 257 lines). patching file languages/messages/MessagesQqq.php Hunk #1 FAILED at 2889. 1 out of 1 hunk FAILED -- saving rejects to file languages/messages/MessagesQqq.php.rej patching file maintenance/language/messages.inc Hunk #1 succeeded at 2450 (offset 203 lines). -- You are receiving this mail because: You are the assignee for the bug. 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 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 --- Comment #7 from John Du Hart --- No. -- You are receiving this mail because: You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 Chad H. changed: What|Removed |Added CC|innocentkil...@gmail.com| -- You are receiving this mail because: You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 Andre Klapper changed: What|Removed |Added CC||aklap...@wikimedia.org --- Comment #6 from Andre Klapper --- Hi John, would you have time to use Developer access https://www.mediawiki.org/wiki/Developer_access to submit this as a Git branch directly into Gerrit: https://www.mediawiki.org/wiki/Git/Tutorial Putting your branch in Git makes it easier for us to review it quickly. Thanks again! We appreciate your contribution. -- You are receiving this mail because: You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 Sumana Harihareswara changed: What|Removed |Added CC||suma...@panix.com --- Comment #5 from Sumana Harihareswara 2012-05-25 03:00:37 UTC --- Hey John, would you mind putting this in Gerrit? Thanks. -- 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 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 --- Comment #4 from John Du Hart 2011-08-17 17:43:04 UTC --- I didn't think it was worth checking to see if the page has edited. Imo who cares if someone edited the page after you open the form, it doesn't affect the move you're about to do. -- 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 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 --- Comment #3 from Umherirrender 2011-08-17 17:38:06 UTC --- It is not enough to check the starttime against the latest revisiontimestamp like EditPage do for normal edit? That means, edit-move conflict is also detected, when someone has edit the page, while a user is trying to move the same page. The other way round is also interesting (bug 5408, bug 23044). -- 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 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 John Du Hart changed: What|Removed |Added Keywords||need-review, patch -- 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 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 John Du Hart changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|wikibugs-l@lists.wikimedia. |j...@compwhizii.net |org | --- Comment #2 from John Du Hart 2011-08-11 18:53:57 UTC --- Created attachment 8910 --> https://bugzilla.wikimedia.org/attachment.cgi?id=8910 Race condition check Error message could probably be better worded. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. 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 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 --- Comment #1 from John Du Hart 2011-08-11 02:54:34 UTC --- The example is really weird. Both moves are A -> B, however in my tests trying to move A -> B (Open two move windows, submit one after the other), I get an error on the last one about B already existing and asking for my permission (Because I'm a sysop) before deleting B to make way for A. It's possible that there was DB lag which caused for nothing to happen for user 2. On a similar note, trying to do 2 moves in a similar fashion except as A -> B and A -> C results in a double redirect of A -> C -> B. Probably the best prevention of this is race condition check (Similar to how edit conflicts are handled). -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. 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 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 Chad H. changed: What|Removed |Added CC||innocentkil...@gmail.com Severity|enhancement |normal -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. 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 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 John Du Hart changed: What|Removed |Added Priority|Normal |Low CC||j...@compwhizii.net Severity|normal |enhancement -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. 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 30211] Detect conflict on move
https://bugzilla.wikimedia.org/show_bug.cgi?id=30211 Mark A. Hershberger changed: What|Removed |Added Priority|Unprioritized |Normal CC||m...@everybody.org -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. 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