Last modified: 2012-12-13 11:18:44 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 T41140, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 39140 - wbgetitems to return revid
wbgetitems to return revid
Status: VERIFIED FIXED
Product: MediaWiki extensions
Classification: Unclassified
WikidataRepo (Other open bugs)
unspecified
All All
: High trivial (vote)
: ---
Assigned To: Wikidata bugs
storypoints: 2
:
Depends on:
Blocks: repoapi
  Show dependency treegraph
 
Reported: 2012-08-08 12:50 UTC by denny vrandecic
Modified: 2012-12-13 11:18 UTC (History)
2 users (show)

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


Attachments

Description denny vrandecic 2012-08-08 12:50:31 UTC
When calling the wbgetitems module the result should also contain the revision id in the content for every given item.
Comment 1 denny vrandecic 2012-08-09 09:43:56 UTC
The revid attribute will be inlined with the item on wbgetitems, i.e.

item : {
  350: {
     id : 350,
     revid : 2000,
     sitelinks : ...
  )
}
Comment 2 jeblad 2012-08-10 10:25:45 UTC
This seems to be called "lastrevid" in the rest of the API, using this name to be consistent. Can't see any pros and cons in any direction, except for being consistent.
Comment 3 jeblad 2012-08-10 11:48:31 UTC
Included as default if props are not redefined, and will then be added together with a few other bits of information. This is basically the same as the ordinary query module is returning. The field "revid" is called "lastrevid" to be consistent with the this module.

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

Note that the ordinary id for the item is returned unconditionally as it should be possible to use "props=" to create a lightweight query. It could be argued that lastrevid should also be returned unconditionally.
Comment 4 Anja Jentzsch 2012-11-29 12:39:31 UTC
Verified in Wikidata demo time for sprint 12

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


Navigation
Links