Maniphest T20189

Request for &rvdiffprop in &prop=revisions to fetch diff info (size)
Closed, DeclinedPublic
Actions

Assigned To

None

Authored By

	• bzimport
	Mar 26 2009, 11:55 PM

Description

Author: herd

Description:
It would be useful to be able to return the size of a diff and/or cache status before actually preloading it via the browser. For example:

http://en.wikipedia.org/w/api.php?action=query&prop=revisions&revids=279481248&rvdiffto=279702980

This generates 168k gzipped, but in actuality it is 5.5 megs plaintext.

With &rvdiffinfo it would be nice, if the diff is already cached, to show the size (and possibly titles, cache key, other diff info) without necessarily fetching it.

I am ambivalent on whether it would be a good idea to return the info if the diff is not cached, as that would involve generating it, and might be a bad practice (that they could then generate huge diffs without having to download them). Probably if it is not cached, the user should get a cached=false or notcached="" parameter, so they know they must &rvdiffto to generate it first.

Version: 1.15.x
Severity: enhancement

Details

Reference: bz18189

	Subject	Repo	Branch	Lines +/-
	API: Rewrite ApiComparePages	mediawiki/core	master	+1 K -69

Customize query in gerrit

Related Objects

Mentioned In: T164106: Deprecate parsing and diff options in ApiQueryRevisionsBase
Mentioned Here: T164106: Deprecate parsing and diff options in ApiQueryRevisionsBase

Event Timeline

• bzimport raised the priority of this task from to Low.Nov 21 2014, 10:32 PM

• bzimport added a project: MediaWiki-Action-API.

• bzimport set Reference to bz18189.

• bzimport added a subscriber: Unknown Object (MLST).

• bzimport created this task.Mar 26 2009, 11:55 PM

I wonder if there is any relation to (fixed) https://bugzilla.wikimedia.org/show_bug.cgi?id=37301 for user contribs and if that fixed one would make it easier to implement this request here?

Anomie moved this task from Unsorted to Needs details or plan on the MediaWiki-Action-API board.Feb 20 2015, 7:51 PM

Anomie mentioned this in T164106: Deprecate parsing and diff options in ApiQueryRevisionsBase.Apr 28 2017, 7:12 PM

Per T164106: Deprecate parsing and diff options in ApiQueryRevisionsBase, diffing in prop=revisions is deprecated.

Change 352191 had a related patch set uploaded (by Anomie; owner: Anomie):
[mediawiki/core@master] API: Rewrite ApiComparePages

https://gerrit.wikimedia.org/r/352191

gerritbot added a project: Patch-For-Review.May 5 2017, 7:53 PM

Change 352191 merged by jenkins-bot:
[mediawiki/core@master] API: Rewrite ApiComparePages