@@ -, +, @@ --- Koha/AdvancedEditorMacro.pm | 2 +- Koha/AdvancedEditorMacros.pm | 6 ++- api/v1/swagger/parameters/advancededitormacro.json | 2 +- .../prog/en/includes/cateditor-ui.inc | 49 +++++++++++++++++++--- 4 files changed, 50 insertions(+), 9 deletions(-) --- a/Koha/AdvancedEditorMacro.pm +++ a/Koha/AdvancedEditorMacro.pm @@ -24,7 +24,7 @@ use base qw(Koha::Object); =head1 NAME -Koha::AdvancedEditorMacro - Koha Advanced Editor Macro Object class +Koha::AdvancedEditorMacro - Koha Advanced editor macro object class =head1 API --- a/Koha/AdvancedEditorMacros.pm +++ a/Koha/AdvancedEditorMacros.pm @@ -26,7 +26,7 @@ use base qw(Koha::Objects); =head1 NAME -Koha::AdvancedEditorMacros - Koha Advanced Editor Macro Object set class +Koha::AdvancedEditorMacros - Koha Advanced editor macro object set class =head1 API @@ -42,6 +42,10 @@ sub _type { return 'AdvancedEditorMacro'; } +=head3 object_class + +=cut + sub object_class { return 'Koha::AdvancedEditorMacro'; } --- a/api/v1/swagger/parameters/advancededitormacro.json +++ a/api/v1/swagger/parameters/advancededitormacro.json @@ -2,7 +2,7 @@ "advancededitormacro_id_pp": { "name": "advancededitormacro_id", "in": "path", - "description": "Advanced Editor Macro internal identifier", + "description": "Advanced editor macro internal identifier", "required": true, "type": "integer" } --- a/koha-tmpl/intranet-tmpl/prog/en/includes/cateditor-ui.inc +++ a/koha-tmpl/intranet-tmpl/prog/en/includes/cateditor-ui.inc @@ -714,7 +714,15 @@ require( [ 'koha-backend', 'search', 'macros', 'marc-editor', 'marc-record', 'pr showSavedMacros(); }) .fail(function(err) { - humanMsg.displayAlert( _("Failed to delete macro:") + err.responseText, { className: 'humanError' } ); + var err_message; + if( err.status == "404" ){ + err_message = "Macro not found"; + } else if ( err.status == "403" ){ + err_message = _("You do not have permission to delete this macro"); + } else { + err_message = _("There was a problem, please check the logs"); + } + humanMsg.displayAlert( _("Failed to delete macro: " + err_message), { className: 'humanError' } ); }); } @@ -760,7 +768,15 @@ require( [ 'koha-backend', 'search', 'macros', 'marc-editor', 'marc-record', 'pr macroEditor.activeMacroShared = result.shared; }) .fail(function(err) { - humanMsg.displayAlert( _("Failed to load macros:") + err.responseText, { className: 'humanError' } ); + var err_message; + if( err.status == "404" ){ + err_message = "Macro not found"; + } else if ( err.status == "403" ){ + err_message = _("You do not have permission to access this macro"); + } else { + err_message = _("There was a problem, please check the logs"); + } + humanMsg.displayAlert( _("Failed to load macros: ") + err_message, { className: 'humanError' } ); }); } @@ -795,7 +811,13 @@ require( [ 'koha-backend', 'search', 'macros', 'marc-editor', 'marc-record', 'pr }) .fail(function(err) { - humanMsg.displayAlert( _("Failed to create macro:") + err.responseText, { className: 'humanError' } ); + var err_message; + if( err.status == "403" ){ + err_message = _("You do not have permission to create this macro"); + } else { + err_message = _("There was a problem, please check the logs"); + } + humanMsg.displayAlert( _("Failed to create macro: ") + err_message, { className: 'humanError' } ); }); } ); } @@ -832,7 +854,8 @@ require( [ 'koha-backend', 'search', 'macros', 'marc-editor', 'marc-record', 'pr }); }) .fail(function(err) { - humanMsg.displayAlert( _("Failed to load macros:") + err.responseText, { className: 'humanError' } ); + var err_message = _("There was a problem, please check the logs"); + humanMsg.displayAlert( _("Failed to load macros: ") + err_message, { className: 'humanError' } ); }); var $new_li = $( '
  • ' + _("New macro...") + '
  • ' ); $new_li.click( function() { @@ -858,7 +881,13 @@ require( [ 'koha-backend', 'search', 'macros', 'marc-editor', 'marc-record', 'pr loadMacro( result.name, result.macro_id ); }) .fail(function(err) { - humanMsg.displayAlert( _("Failed to create macro:") + err.responseText, { className: 'humanError' } ); + var err_message; + if( err.status == "403" ){ + err_message = _("You do not have permission to access this macro"); + } else { + err_message = _("There was a problem, please check the logs"); + } + humanMsg.displayAlert( _("Failed to create macro: ") + err_message, { className: 'humanError' } ); }); } ); $('#macro-list').append($new_li); @@ -894,7 +923,15 @@ require( [ 'koha-backend', 'search', 'macros', 'marc-editor', 'marc-record', 'pr showSavedMacros(); }) .fail(function(err) { - humanMsg.displayAlert( _("Failed to save macro:") + err.responseText, { className: 'humanError' } ); + var err_message; + if( err.status == "404" ){ + err_message = _("Macro not found"); + } else if ( err.status ="403" ){ + err_message = _("You do not have permission to access this macro"); + } else { + err_message = _("There was a problem, please check the logs"); + } + humanMsg.displayAlert( _("Failed to save macro: ") + err_message, { className: 'humanError' } ); }); } --