move error callback into options and rename it

This commit is contained in:
Felix Wiedenbach 2021-02-13 08:29:13 +01:00
parent 1ed721fb15
commit 9d85baee37
2 changed files with 6 additions and 11 deletions

View file

@ -279,9 +279,8 @@ var MM = (function () {
* @param {number} speed The speed of the show animation.
* @param {Function} callback Called when the animation is done.
* @param {object} [options] Optional settings for the show method.
* @param {Function} errorCallback Called when the module failed to show.
*/
var showModule = function (module, speed, callback, errorCallback, options) {
var showModule = function (module, speed, callback, options) {
options = options || {};
// remove lockString if set in options.
@ -296,8 +295,8 @@ var MM = (function () {
// Otherwise cancel show action.
if (module.lockStrings.length !== 0 && options.force !== true) {
Log.log("Will not show " + module.name + ". LockStrings active: " + module.lockStrings.join(","));
if (typeof errorCallback === "function") {
errorCallback(new Error("ERR_ACTIVE_LOCK_STRINGS"));
if (typeof options.onError === "function") {
options.onError(new Error("ERR_ACTIVE_LOCK_STRINGS"));
}
return;
}
@ -590,11 +589,10 @@ var MM = (function () {
* @param {number} speed The speed of the show animation.
* @param {Function} callback Called when the animation is done.
* @param {object} [options] Optional settings for the show method.
* @param {Function} errorCallback Called when the module failed to show.
*/
showModule: function (module, speed, callback, errorCallback, options) {
showModule: function (module, speed, callback, options) {
// do not change module.hidden yet, only if we really show it later
showModule(module, speed, callback, errorCallback, options);
showModule(module, speed, callback, options);
}
};
})();