diff --git a/js/pages/backoffice/toolbox.js b/js/pages/backoffice/toolbox.js index 5cbdf92a6..70b086e4a 100644 --- a/js/pages/backoffice/toolbox.js +++ b/js/pages/backoffice/toolbox.js @@ -180,7 +180,14 @@ const CombodoBackofficeToolbox = { * @inheritDoc */ CombodoModal.CloseAllModals = function() { - // TODO: Implement + $('.ui-dialog .ui-dialog-content').each(function () { + // Don't try to close dialog if not instantiated yet + if ($(this).dialog('instance') === undefined) { + return false; + } + + $(this).dialog('close'); + }); }; /** * @override @@ -291,9 +298,9 @@ CombodoModal._InstantiateModal = function(oModalElem, oOptions) { return true; }; - /** * @override + * @inheritDoc */ CombodoModal._BindEvents = function (oModalElem) { const me = this; @@ -306,13 +313,12 @@ CombodoModal._BindEvents = function (oModalElem) { oModalObs.observe(oModalElem[0]); } }; - /** * Convert generic buttons definitions to jquery ui dialog definitions. * * @param aButtonsDefinitions * @returns {*[]} - * @constructor + * @private */ CombodoModal._ConvertButtonDefinition = function (aButtonsDefinitions) { const aConverted = []; @@ -330,7 +336,6 @@ CombodoModal._ConvertButtonDefinition = function (aButtonsDefinitions) { ); return aConverted; }; - /** * @override * @inheritDoc @@ -345,13 +350,9 @@ CombodoModal._CenterModalInViewport = function (oModalElem) { position: {my: 'center', at: 'center', of: window}, }); }; - /** * @override * @inheritDoc - * - * @param oOptions array @see CombodoModal.OpenModal + {do_not_show_again_pref_key: string, callback_on_confirm: function, callback_on_cancel} - * @param aData data passed to callbacks */ CombodoModal.OpenConfirmationModal = function(oOptions, aData) { @@ -425,7 +426,6 @@ CombodoModal.OpenConfirmationModal = function(oOptions, aData) { // Open modal CombodoModal.OpenModal(oOptions); } - /** * @override * @inheritDoc diff --git a/js/utils.js b/js/utils.js index 452c3f248..83096163e 100644 --- a/js/utils.js +++ b/js/utils.js @@ -1477,10 +1477,18 @@ let CombodoModal = { /** * Open a standard confirmation modal and put the content into it. * - * @param oOptions - * @returns object The jQuery object of the modal element + * @param oOptions {Object} {@see CombodoModal.OpenModal} + + * ``` + * { + * do_not_show_again_pref_key: string, + * callback_on_confirm: function, + * callback_on_cancel: function + * } + * ``` + * @param aData {Array} Array of arguments to pass to the callbacks + * @return {Object} The jQuery object of the modal element */ - OpenConfirmationModal: function(oOptions) { + OpenConfirmationModal: function(oOptions, aData) { // Meant for overlaoding CombodoJSConsole.Debug('CombodoModal.OpenConfirmationModal not implemented'); },