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

Fix "Move to Top" logic for creators

Browse source
This commit is contained in:
Dan Stillman 2019-03-05 19:15:50 -05:00
parent d7dc5670d5
commit ae9c54b76d
1 changed files with 15 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
chrome/content/zotero/bindings/itembox.xml
View file

@ -2196,10 +2196,21 @@
newIndex = index + 1;
break;
}
var a = this.item.getCreator(index);
var b = this.item.getCreator(newIndex);
this.item.setCreator(newIndex, a);
this.item.setCreator(index, b);
let creator = this.item.getCreator(index);
// When moving to top, increment index of all other creators
if (dir == 'top') {
let otherCreators = this.item.getCreators();
this.item.setCreator(newIndex, creator);
for (let i = 0; i < index; i++) {
this.item.setCreator(i + 1, otherCreators[i]);
}
}
// When moving up or down, swap places with next creator
else {
let otherCreator = this.item.getCreator(newIndex);
this.item.setCreator(newIndex, creator);
this.item.setCreator(index, otherCreator);
}
if (this.saveOnEdit) {
// See note in transformText()
yield this.blurOpenField();