Skip to content

Commit

Permalink
Fix file conflict resolution after f0770fa
Browse files Browse the repository at this point in the history
merge.xul now returns an object with .data and .selected instead of
returning .data directly
  • Loading branch information
dstillman committed Oct 27, 2017
1 parent 2184952 commit 196e45c
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions chrome/content/zotero/xpcom/collectionTreeView.js
Expand Up @@ -2192,6 +2192,7 @@ Zotero.CollectionTreeView.prototype.drop = Zotero.Promise.coroutine(function* (r
lastWin.openDialog('chrome://zotero/content/merge.xul', '', 'chrome,modal,centerscreen', io);

yield Zotero.DB.executeTransaction(function* () {
// DEBUG: This probably needs to be updated if this starts being used
for (let obj of io.dataOut) {
yield obj.ref.save();
}
Expand Down
3 changes: 2 additions & 1 deletion chrome/content/zotero/xpcom/storage/storageLocal.js
Expand Up @@ -1026,11 +1026,12 @@ Zotero.Sync.Storage.Local = {
if (!io.dataOut) {
return false;
}

yield Zotero.DB.executeTransaction(function* () {
for (let i = 0; i < conflicts.length; i++) {
let conflict = conflicts[i];
let item = Zotero.Items.getByLibraryAndKey(libraryID, conflict.left.key);
let mtime = io.dataOut[i].dateModified;
let mtime = io.dataOut[i].data.dateModified;
// Local
if (mtime == conflict.left.dateModified) {
syncState = this.SYNC_STATE_FORCE_UPLOAD;
Expand Down

0 comments on commit 196e45c

Please sign in to comment.