Attachment 117891 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), 2.05 KB, created by Katrin Fischer on 2021-03-07 15:59:54 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2021-03-07 15:59:54 UTC

Size: 2.05 KB

patch

obsolete

>From 6863c7db7d13cb25aa214090a18e5d5f9ad2ab31 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
>
>Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> 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 39f03f75b5..352e9a0c53 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.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 26964: 117404 | 117405 | 117891