Attachment 117404 Details for Bug 26964 – Bug 26964: Newly created macros should be selected in the Advanced editor

[patch] Bug 26964: Newly created macros should be selected in the Advanced editor

Bug-26964-Newly-created-macros-should-be-selected-.patch (text/plain), 1.93 KB, created by Phil Ringnalda on 2021-02-26 21:54:53 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Phil Ringnalda

Created: 2021-02-26 21:54:53 UTC

Size: 1.93 KB

patch

obsolete

>From 83696cb119e86ad7bee386e5dc49e48878f51d2c Mon Sep 17 00:00:00 2001
>From: Phil Ringnalda <phil@chetcolibrary.org>
>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

Actions: View | Diff | Splinter Review

Attachments on bug 26964: 117404 | 117405 | 117891