зеркало из https://github.com/mozilla/pjs.git
fix js warning about savePage() and restorePage() not always returning a value.
r/sr=seth@craptastic.org
This commit is contained in:
Родитель
f5cacdc174
Коммит
d881adb4ca
|
@ -703,9 +703,10 @@ function loadPage(pageId)
|
||||||
//
|
//
|
||||||
// save the values of the widgets to the given server
|
// save the values of the widgets to the given server
|
||||||
//
|
//
|
||||||
function savePage(serverId) {
|
function savePage(serverId)
|
||||||
|
{
|
||||||
if (!serverId) return;
|
if (!serverId)
|
||||||
|
return;
|
||||||
|
|
||||||
// tell the page that it's about to save
|
// tell the page that it's about to save
|
||||||
if ("onSave" in top.frames["contentFrame"])
|
if ("onSave" in top.frames["contentFrame"])
|
||||||
|
@ -713,11 +714,11 @@ function savePage(serverId) {
|
||||||
|
|
||||||
var accountValues = getValueArrayFor(serverId);
|
var accountValues = getValueArrayFor(serverId);
|
||||||
if (!accountValues)
|
if (!accountValues)
|
||||||
return true;
|
return;
|
||||||
|
|
||||||
var pageElements = getPageFormElements();
|
var pageElements = getPageFormElements();
|
||||||
|
if (!pageElements)
|
||||||
if (pageElements == null) return;
|
return;
|
||||||
|
|
||||||
// store the value in the account
|
// store the value in the account
|
||||||
for (var i=0; i<pageElements.length; i++) {
|
for (var i=0; i<pageElements.length; i++) {
|
||||||
|
@ -733,7 +734,6 @@ function savePage(serverId) {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
||||||
function setAccountValue(accountValues, type, slot, value) {
|
function setAccountValue(accountValues, type, slot, value) {
|
||||||
|
@ -823,15 +823,18 @@ function getAccountValue(account, accountValues, type, slot, preftype, isGeneric
|
||||||
//
|
//
|
||||||
// restore the values of the widgets from the given server
|
// restore the values of the widgets from the given server
|
||||||
//
|
//
|
||||||
function restorePage(pageId, serverId) {
|
function restorePage(pageId, serverId)
|
||||||
if (!serverId) return;
|
{
|
||||||
|
if (!serverId)
|
||||||
|
return;
|
||||||
|
|
||||||
var accountValues = getValueArrayFor(serverId);
|
var accountValues = getValueArrayFor(serverId);
|
||||||
if (!accountValues)
|
if (!accountValues)
|
||||||
return true;
|
return;
|
||||||
|
|
||||||
var pageElements = getPageFormElements();
|
var pageElements = getPageFormElements();
|
||||||
|
if (!pageElements)
|
||||||
if (pageElements == null) return;
|
return;
|
||||||
|
|
||||||
var account = getAccountFromServerId(serverId);
|
var account = getAccountFromServerId(serverId);
|
||||||
|
|
||||||
|
|
Загрузка…
Ссылка в новой задаче