From 83696cb119e86ad7bee386e5dc49e48878f51d2c Mon Sep 17 00:00:00 2001 From: Phil Ringnalda Date: Fri, 26 Feb 2021 13:39:37 -0800 Subject: [PATCH] Bug 26964: Newly created macros should be selected in the Advanced editor The problem is that the newly created macro isn't selected, because the return values from the call to create it are accidentally thrown away. There isn't actually any option to save, just a timeout that automatically saves, but you can tell you are in this broken state because the Delete macro button is disabled. Test plan: 1. Prior to applying the patch, Administration - set the preference EnableAdvancedCatalogingEditor to Enable 2. Cataloging - Advanced editor - Macros... - New macro... 3. Give it a name, click OK, notice that the Delete macro button is disabled 4. Apply patch, shift+reload 5. Macros... - New macro... 6. Give it a name, click OK, notice that the Delete macro button is enabled 7. Type something in the editor, delete 999 will do nicely, wait half a second to see Saved appear at the far right, close the macro popup 8. Macros..., click the name of your macro, verify it really was saved --- koha-tmpl/intranet-tmpl/prog/en/includes/cateditor-ui.inc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/cateditor-ui.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/cateditor-ui.inc index 62bbe96a94..d0b12903a0 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/cateditor-ui.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/cateditor-ui.inc @@ -902,7 +902,7 @@ require( [ 'koha-backend', 'search', 'macros', 'marc-editor', 'marc-record', 'pr }) }; $.ajax(options) - .then(function(undef, result) { + .then(function(result) { showSavedMacros(); loadMacro( result.name, result.macro_id ); }) -- 2.26.1