From 52dda8ae0ba6ee7d9c191de4ce49b163a913b97f Mon Sep 17 00:00:00 2001 From: Dan Stillman Date: Fri, 3 Jul 2009 05:11:22 +0000 Subject: [PATCH] Don't throw error on database backup failure, which was breaking some upgrades for unknown reasons --- chrome/content/zotero/xpcom/db.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/chrome/content/zotero/xpcom/db.js b/chrome/content/zotero/xpcom/db.js index 8acee71ab..9f8595baf 100644 --- a/chrome/content/zotero/xpcom/db.js +++ b/chrome/content/zotero/xpcom/db.js @@ -855,10 +855,10 @@ Zotero.DBConnection.prototype.backupDatabase = function (suffix) { getService(Components.interfaces.mozIStorageService); store.backupDatabaseFile(file, tmpFile.leafName, file.parent); } - catch (e){ + catch (e) { Zotero.debug(e); - // TODO: deal with low disk space - throw (e); + Components.utils.reportError(e); + return false; } // Opened database files can't be moved on Windows, so we have to skip