Display more helpful message for "Username not set" error when clicking on the sync error icon (previously displayed a full message at sync time for foreground syncs but updated the error icon dialog only for background syncs)
This commit is contained in:
parent
456406fc10
commit
749b4ee464
1 changed files with 12 additions and 15 deletions
|
@ -2261,21 +2261,18 @@ Zotero.Sync.Server = new function () {
|
|||
// instead of creating its own dialog, but setSyncIcon() doesn't yet provide full control
|
||||
// over dialog title and primary button text/action, which is why this version of the
|
||||
// dialog is a bit uglier than the manual click version
|
||||
if (Zotero.Sync.Runner.background) {
|
||||
// TODO: localize and combine with below
|
||||
var msg = "The Zotero sync server did not accept your username and password.\n\n"
|
||||
+ "Please check that you have entered your zotero.org login information correctly in the Zotero sync preferences.";
|
||||
e.data = {};
|
||||
e.data.dialogText = msg;
|
||||
e.data.dialogButtonText = Zotero.getString('sync.openSyncPreferences');
|
||||
e.data.dialogButtonCallback = function () {
|
||||
var wm = Components.classes["@mozilla.org/appshell/window-mediator;1"]
|
||||
.getService(Components.interfaces.nsIWindowMediator);
|
||||
var win = wm.getMostRecentWindow("navigator:browser");
|
||||
win.ZoteroPane.openPreferences("zotero-prefpane-sync");
|
||||
};
|
||||
break;
|
||||
}
|
||||
// TODO: localize and combine with below
|
||||
var msg = "The Zotero sync server did not accept your username and password.\n\n"
|
||||
+ "Please check that you have entered your zotero.org login information correctly in the Zotero sync preferences.";
|
||||
e.data = {};
|
||||
e.data.dialogText = msg;
|
||||
e.data.dialogButtonText = Zotero.getString('sync.openSyncPreferences');
|
||||
e.data.dialogButtonCallback = function () {
|
||||
var wm = Components.classes["@mozilla.org/appshell/window-mediator;1"]
|
||||
.getService(Components.interfaces.nsIWindowMediator);
|
||||
var win = wm.getMostRecentWindow("navigator:browser");
|
||||
win.ZoteroPane.openPreferences("zotero-prefpane-sync");
|
||||
};
|
||||
|
||||
// Manual click
|
||||
setTimeout(function () {
|
||||
|
|
Loading…
Reference in a new issue