www

commit 92f0cdd1af4dee3f5a22961c733e0ba02c2ba3e3
parent ab1983ebf26809bb7214801ef9bf12eeaf428220
Author: Dan Stillman <dstillman@zotero.org>
Date:   Mon,  1 Mar 2010 20:01:46 +0000

Restore proper feedback for Reset options and add a warning to Restore to Server that there may be a delay before data shows up online


Diffstat:
M
chrome/content/zotero/preferences/preferences.js
 | 
46
+++++++++++++++++++++++++++++++++++-----------

1 file changed, 35 insertions(+), 11 deletions(-)
diff --git a/chrome/content/zotero/preferences/preferences.js b/chrome/content/zotero/preferences/preferences.js
@@ -423,7 +423,9 @@ function handleSyncReset(action) {
 				// TODO: localize
 				Zotero.getString('general.warning'),
 				"All data belonging to user '" + account + "' on the Zotero server "
-					+ "will be erased and replaced with data from this copy of Zotero.",
+					+ "will be erased and replaced with data from this copy of Zotero.\n\n"
+					+ "Depending on the size of your library, there may be a delay before "
+					+ "your data is available on the server.",
 				buttonFlags,
 				"Replace Server Data",
 				null, null, null, {}
@@ -433,11 +435,22 @@ function handleSyncReset(action) {
 				case 0:
 					// TODO: better error handling
 					Zotero.Sync.Server.clear(function () {
-						Zotero.Sync.Server.sync(function () {
-							pr.alert(
-								"Restore Completed",
-								"Data on the Zotero server has been successfully restored."
-							);
+						Zotero.Sync.Server.sync({
+							onSuccess: function () {
+								pr.alert(
+									"Restore Completed",
+									"Data on the Zotero server has been successfully restored."
+								);
+							},
+							onError: function () {
+								// TODO: combine with error dialog for regular syncs
+								pr.alert(
+									"Restore Failed",
+									"An error occurred uploading your data to the server.\n\n"
+										+ "Click the sync error icon in the Zotero toolbar "
+										+ "for further information."
+								);
+							}
 						});
 					});
 					break;
@@ -468,11 +481,22 @@ function handleSyncReset(action) {
 				case 0:
 					// TODO: better error handling
 					Zotero.Sync.Server.resetClient();
-					Zotero.Sync.Server.sync(function () {
-						pr.alert(
-							"Full Sync Completed",
-							"The local Zotero library has been merged with data from the Zotero server."
-						);
+					Zotero.Sync.Server.sync({
+						onSuccess: function () {
+							pr.alert(
+								"Full Sync Completed",
+								"The local Zotero library has been merged with data from the Zotero server."
+							);
+						},
+						onError: function () {
+							// TODO: combine with error dialog for regular syncs
+							pr.alert(
+								"Full Sync Failed",
+								"An error occurred while performing the full sync.\n\n"
+									+ "Click the sync error icon in the Zotero toolbar "
+									+ "for further information."
+							);
+						}
 					});
 					break;