[Bug 32037] API: Allow edit conflict detection based on revid
https://bugzilla.wikimedia.org/show_bug.cgi?id=32037 Nemo changed: What|Removed |Added Blocks||70163 -- 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 32037] API: Allow edit conflict detection based on revid
https://bugzilla.wikimedia.org/show_bug.cgi?id=32037 --- Comment #7 from Michael M. --- (In reply to comment #6) > So in short this is bug 34580 aka bug 22783? No. In my use case I want an edit conflict, even if I'm only prepending text (as it could be the same text that was added by the other user causing the potential edit conflict). IMHO in an ideal world edit conflict detection would work this way: # The user didn't send the baseRevision parameter -> Just edit the page according to the other data the user sent, ignore any possible conflict. # The user sent baseRevision === 0 -> If the page is still missing, create it. -> If the page exists, but all edits are from the current user, edit the page without edit conflict. -> Otherwise try to resolve the edit conflict with diff3 (base: empty article), report conflict if that fails. # The user sent baseRevision !== 0 -> If the page exists and the current revision is the user's baseRevision, edit the article. -> If the page exists in a different revision, but all edits between the baseRevision and the current revision are made by the current user, edit without conflict. -> If the page exists in a different revision and edits by other users, try to resolve the conflict with diff3 (base determined by content of baseRevision), report conflict if that fails. -> If the page doesn't exist, tell the user it was deleted while he was editing. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 32037] API: Allow edit conflict detection based on revid
https://bugzilla.wikimedia.org/show_bug.cgi?id=32037 Nemo changed: What|Removed |Added CC||federicol...@tiscali.it See Also||https://bugzilla.wikimedia. ||org/show_bug.cgi?id=22783 --- Comment #6 from Nemo --- (In reply to comment #4) > Consider the following use case: A user wants to add a maintenance template > at > the top of the article he is reading using a fancy ajaxy gadget. [...] So in short this is bug 34580 aka bug 22783? -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 32037] API: Allow edit conflict detection based on revid
https://bugzilla.wikimedia.org/show_bug.cgi?id=32037 Liangent changed: What|Removed |Added CC||liang...@gmail.com --- Comment #5 from Liangent 2012-09-03 08:04:20 UTC --- (In reply to comment #4) > If the page was edited in the meantime, an edit conflict will be detected > (revid sent to API != revid of last version in DB), which is exactly the > desired behavior. It's not correct. A conflict won't be raised if an automatic merge succeeds. -- 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 32037] API: Allow edit conflict detection based on revid
https://bugzilla.wikimedia.org/show_bug.cgi?id=32037
Michael M. changed:
What|Removed |Added
CC||listenle...@gmail.com
--- Comment #4 from Michael M. 2012-09-03 08:00:49 UTC
---
Consider the following use case: A user wants to add a maintenance template at
the top of the article he is reading using a fancy ajaxy gadget.
Currently the script has to do the following steps:
1. Get the wikitext, timestamps and token via API.
2. Check whether the wikitext belongs to the version the user currently is
looking at (perhaps somebody else has added exactly that template since the
page has been viewed)
3. Prepend the article to the text and use the edit API to save the change.
If revids could be used, this would be simpler:
1. Get edit token and revid from the data available to JavaScript.
2. Call the edit API to make the change (with prependtext parameter there is no
need to get the wikitext first)
If the page was edited in the meantime, an edit conflict will be detected
(revid sent to API != revid of last version in DB), which is exactly the
desired behavior.
The current step #2 is not easy at all (e.g. when the user wants to add
{{db-nonsense}}, the script should notice if {{TeMpLaTe:Db-g1}} was added since
the user viewed the page), and we currently need two API calls where one would
be enough.
--
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 32037] API: Allow edit conflict detection based on revid
https://bugzilla.wikimedia.org/show_bug.cgi?id=32037 --- Comment #3 from Krinkle 2012-09-02 20:29:42 UTC --- CC-ing Catrope and Reedy for their insight, I don't know the API well enough. If revision IDs is technically superior / more logical (internally) then go for it. On the other hand, in defense of the current system: page_touched/page_latest are both similarly easy to retrieve as is rev_timestamp/rev_id. In order to make an edit with the API, one should also retrieve the wikitext of a page (through the api, prop=revisions) and in that api ids and timestamp are both just as easy to retrieve. What is inside mw.config should not be relevant because that is not part of the API and not be relied on for API interaction (generally use API output for API input). Moreover because wgCurRevisionId is hardcoded in the page which means there is two unreliability factors: * It may be cached by the browser * The page can be edited since the page has been viewed (e.g. use viewing the page, and performing all kinds of actions, then initiates your script and by that time all kinds of edits could've been made, including but not limited to ajax edits by the user himself, so always get the info from the API when you need it) -- 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 32037] API: Allow edit conflict detection based on revid
https://bugzilla.wikimedia.org/show_bug.cgi?id=32037 --- Comment #2 from Umherirrender 2012-08-31 14:10:37 UTC --- Timestamp is very old MediaWiki way for this, since the revision id exist, that id should be used. The revision id of the latest revision is stored within page table, that makes it easy to retrieve this value for api calls and the revision id is already part of mw.config inside the gui (wgCurRevisionId) -- 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 32037] API: Allow edit conflict detection based on revid
https://bugzilla.wikimedia.org/show_bug.cgi?id=32037 Krinkle changed: What|Removed |Added CC||krinklem...@gmail.com --- Comment #1 from Krinkle 2012-08-30 20:32:15 UTC --- How is the timestamp a problem? 14 digits can't be stored? -- 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 32037] API: Allow edit conflict detection based on revid
https://bugzilla.wikimedia.org/show_bug.cgi?id=32037 Mark A. Hershberger changed: What|Removed |Added Priority|Unprioritized |Normal 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
