Bug 29436

Summary: Cannot reorder facets in staff interface elasticsearch configuration
Product: Koha Reporter: Nick Clemens <nick>
Component: Searching - ElasticsearchAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P5 - low CC: andrewfh, fridolin.somers, jonathan.druart, mkstephens, victor
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
22.05.00,21.11.02,21.05.08
Bug Depends on: 19482    
Bug Blocks:    
Attachments: Bug 29436: ES mappings not saved if zebra is configured
Bug 29436: ES mappings not saved if zebra is configured
Bug 29436: ES mappings not saved if zebra is configured

Description Nick Clemens 2021-11-08 15:33:52 UTC
On master:
Move author to bottom of list, click save
"An error occurred when updating mappings: . "

On 21.05 we are seeing that nothing happens when clicking the 'Save' button
Comment 1 Jonathan Druart 2021-11-16 15:02:11 UTC
I actually get this error when searchengine=zebra, and the message is... "elasticsearch_disabled"...
Comment 2 Jonathan Druart 2021-11-16 15:06:29 UTC
Created attachment 127697 [details] [review]
Bug 29436: ES mappings not saved if zebra is configured

The mappings must be editable even if ES is not turned on yet.

Using a separate array to store the errors as we are testing for $@ ||
@messages.

There is still something wrong that should be improve, but this patch
should be safe for backport.
Comment 3 Jonathan Druart 2021-11-16 15:07:52 UTC
Looks like it has been caused by
  commit 975e06bd7c3e198048c60d4a0f9968ba89079de7
  Bug 19482: Add support for defining 'mandatory' mappings
Comment 4 Jonathan Druart 2021-11-16 15:09:06 UTC
(In reply to Jonathan Druart from comment #2)
> There is still something wrong that should be improve, but this patch
> should be safe for backport.

hum... bug 19482 comment 70
Comment 5 Fridolin Somers 2021-11-19 07:32:23 UTC
Created attachment 127824 [details] [review]
Bug 29436: ES mappings not saved if zebra is configured

The mappings must be editable even if ES is not turned on yet.

Using a separate array to store the errors as we are testing for $@ ||
@messages.

There is still something wrong that should be improve, but this patch
should be safe for backport.

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
Comment 6 Fridolin Somers 2021-11-19 07:33:10 UTC
Looks OK to me.
Comment 7 Nick Clemens 2021-12-17 14:38:00 UTC
Created attachment 128673 [details] [review]
Bug 29436: ES mappings not saved if zebra is configured

The mappings must be editable even if ES is not turned on yet.

Using a separate array to store the errors as we are testing for $@ ||
@messages.

There is still something wrong that should be improve, but this patch
should be safe for backport.

Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 8 Fridolin Somers 2021-12-22 06:55:16 UTC
Pushed to master for 22.05, thanks to everybody involved 🦄
Comment 9 Andrew Fuerste-Henry 2022-01-10 15:11:46 UTC
21.11 log shows this was pushed to 21.11.02.

Pushed to 21.05.x for 21.05.08.
Comment 10 Victor Grousset/tuxayo 2022-02-06 22:21:31 UTC
Not backported to oldoldstable (20.11.x). Feel free to ask if it's needed.