mirrors/zotero
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Don't accept libraryVersion in syncAPIClient::getVersions()

Browse source 
Since /settings is always called first with If-Modified-Since-Version,
it's never necessary to include that in the versions request.
This commit is contained in:
Dan Stillman 2015-11-12 02:49:36 -05:00
parent ae277391b6
commit 69e5c5ae05
1 changed files with 3 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
chrome/content/zotero/xpcom/sync/syncAPIClient.js
View file

@ -158,9 +158,10 @@ Zotero.Sync.APIClient.prototype = {
* @param {Integer} libraryTypeID userID or groupID
* @param {String} objectType 'item', 'collection', 'search'
* @param {Object} queryParams Query parameters (see buildRequestURI())
* @return {Promise<Object>|FALSE} Object with 'libraryVersion' and 'results'
* @return {Promise<Object>|false} - Object with 'libraryVersion' and 'results', or false if
* nothing changed since specified library version
*/
getVersions: Zotero.Promise.coroutine(function* (libraryType, libraryTypeID, objectType, queryParams, libraryVersion) {
getVersions: Zotero.Promise.coroutine(function* (libraryType, libraryTypeID, objectType, queryParams) {
var objectTypePlural = Zotero.DataObjectUtilities.getObjectTypePlural(objectType);
var params = {
@ -184,11 +185,6 @@ Zotero.Sync.APIClient.prototype = {
var options = {
successCodes: [200, 304]
};
if (libraryVersion) {
options.headers = {
"If-Modified-Since-Version": libraryVersion
};
}
var xmlhttp = yield this.makeRequest("GET", uri, options);
if (xmlhttp.status == 304) {
return false;