Last modified: 2013-06-18 20:47:26 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T51390, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 49390 - VisualEditor: Restoring oldids broken
VisualEditor: Restoring oldids broken
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
MediaWiki integration (Other open bugs)
unspecified
All All
: High critical
: VE-deploy-2013-06-20
Assigned To: Roan Kattouw
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-06-10 16:35 UTC by James Forrester
Modified: 2013-06-18 20:47 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description James Forrester 2013-06-10 16:35:41 UTC
On attempting to edit and restore an oldid, everything works fine (oldid contents appear, diff shows correct wikitext change, etc.) but on save VE appears to send the wrong page version (presumably the 'newer' version that's meant to be replaced?) to MW so nothing gets saved.
Comment 1 Krinkle 2013-06-17 23:26:42 UTC
I can't reproduce this bug. I'll keep the assignment to Ed but it looks like this has been resolved in the mean time.
Comment 2 Ed Sanders 2013-06-18 16:33:44 UTC
I can reproduce it. Tellingly, if you make a change and then press save you get an edit conflict. I suspect the problem is we are providing the timestamp of the version you choose to edit as the 'basetimestamp' which I don't think is right. When you hit save the API thinks all the versions ahead of the version you're editing were created since you loaded the page (hence the edit conflict scenario).

I think 'basetimestamp' should always be populated with the timestamp of 'cur', regardless of which version you are editing, but would like confirmation from someone who knows the MW API better.
Comment 3 Roan Kattouw 2013-06-18 17:23:16 UTC
Ed is exactly right. EditPage appears to set basetimestamp this way as well.
Comment 4 Gerrit Notification Bot 2013-06-18 19:18:17 UTC
Related URL: https://gerrit.wikimedia.org/r/69344 (Gerrit Change Icacaa5101386e52309278c8d9871ca1193aa86a9)
Comment 5 James Forrester 2013-06-18 20:47:26 UTC
Merged and will go out with next deployment.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links