Last modified: 2012-08-14 17:35:57 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 T39914, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 37914 - Parsoid / Web service: add oldid support to the web service to support retrieval of old revisions
Parsoid / Web service: add oldid support to the web service to support retrie...
Status: RESOLVED FIXED
Product: Parsoid
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Normal enhancement
: ---
Assigned To: Gabriel Wicke
:
Depends on:
Blocks: 37821
  Show dependency treegraph
 
Reported: 2012-06-24 20:51 UTC by Gabriel Wicke
Modified: 2012-08-14 17:35 UTC (History)
3 users (show)

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


Attachments

Description Gabriel Wicke 2012-06-24 20:51:14 UTC

    
Comment 1 Mark Holmquist 2012-08-06 18:10:51 UTC
https://gerrit.wikimedia.org/r/17829

That should do it for now, but if I've missed part of the problem here, I can easily do something more.

Basically, anywhere there's a TemplateRequest, I've checked for an oldid query string parameter, and used it in the relevant API call. This only happens in GET requests right now, and it might not necessarily get the old versions of templates, etc., that were in use at the time. I can change both of those things, I think, with some small pointers to where the changes are necessary.
Comment 2 James Forrester 2012-08-06 19:25:53 UTC
Mass-moving bugs into the new 'Parsoid' product.
Comment 3 Gabriel Wicke 2012-08-06 19:57:01 UTC
Works nicely for GET requests, and should also be used by the VE for POST requests. We don't use the oldid on POST yet, but will likely do so in the future for reconciliation and possibly retrieval of rt info.

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


Navigation
Links