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

Add creator fix to integrity check, and run at startup if necessary

Browse source
This commit is contained in:
Dan Stillman 2017-11-16 06:43:18 -05:00
parent 1cb49cc5d3
commit 7d3311679e
2 changed files with 28 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
chrome/content/zotero/xpcom/data/creators.js
View file

@ -31,10 +31,12 @@ Zotero.Creators = new function() {
var _cache = {};
this.init = Zotero.Promise.coroutine(function* () {
var repaired = false;
var sql = "SELECT * FROM creators";
var rows = yield Zotero.DB.queryAsync(sql);
for (let i = 0; i < rows.length; i++) {
let row = rows[i];
try {
_cache[row.creatorID] = this.cleanData({
// Avoid "DB column 'name' not found" warnings from the DB row Proxy
firstName: row.firstName,
@ -42,6 +44,21 @@ Zotero.Creators = new function() {
fieldMode: row.fieldMode
});
}
catch (e) {
// Automatically fix DB errors and try again
if (!repaired) {
Zotero.logError(e);
Zotero.logError("Trying integrity check to fix creator error");
yield Zotero.Schema.integrityCheck(true);
repaired = true;
rows = yield Zotero.DB.queryAsync(sql);
i = -1;
continue;
}
throw e;
}
}
});
/*

5
chrome/content/zotero/xpcom/schema.js
View file

@ -1334,6 +1334,11 @@ Zotero.Schema = new function(){
[
"SELECT COUNT(*) > 0 FROM itemAttachments WHERE linkMode NOT IN (0,1,2,3)",
"UPDATE itemAttachments SET linkMode=1 WHERE linkMode NOT IN (0,1,2,3)"
],
// Creators with first name can't be fieldMode 1
[
"SELECT COUNT(*) > 0 FROM creators WHERE fieldMode = 1 AND firstName != ''",
"UPDATE creators SET fieldMode = 0 WHERE fieldMode = 1 AND firstName != ''"
]
];