Bug 14006

Summary: about.pl checks the wrong zebra index mode
Product: Koha Reporter: Magnus Enger <magnus>
Component: AboutAssignee: Nicole C. Engard <nengard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: chris, eivin, mtj, mtompset, tomascohen
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:
Attachments: Bug 14006: about.pl checks the wrong zebra index mode
Bug 14006: about.pl checks the wrong zebra index mode
[PASSED QA] Bug 14006: about.pl checks the wrong zebra index mode

Description Magnus Enger 2015-04-16 19:13:37 UTC
about.pl has these lines:

104 if ( (C4::Context->config('zebra_bib_index_mode') eq 'dom') && ($context->{'server'}->{'biblioserver'}->{'config'} !~ /zebra-biblios-dom.cfg/) )     {
105     push @xml_config_warnings, {
106         error => 'zebra_bib_index_mode_mismatch_warn'
107     };
108 }
109 
110 if ( (C4::Context->config('zebra_auth_index_mode') eq 'grs1') && ($context->{'server'}->{'biblioserver'}->{'config'} =~ /zebra-biblios-dom.cfg/)     ) {
111     push @xml_config_warnings, {
112         error => 'zebra_bib_index_mode_mismatch_warn'
113     };
114 }

I think line 110 should be looking at C4::Context->config('zebra_*bib*_index_mode'), not C4::Context->config('zebra_auth_index_mode')?
Comment 1 Tomás Cohen Arazi 2015-04-20 18:48:25 UTC Comment hidden (obsolete)
Comment 2 Mark Tompsett 2015-04-22 03:00:24 UTC
(In reply to Tomás Cohen Arazi from comment #1)
> - Reload the about page
> => SUCCESS: A zebra configuration missmatch is shown.

Without the patch, I get one warning. With the patch I get two.
The first is a grs1 is deprecated message, which is valid.
The second is bothersome, because it says I have dom set, but my koha-conf.xml was tweaked to grs1. It also says my configuration looks like it is for grs1, which I had it set up for dom before the single line configuration file tweak. So, the error triggered does not represent the actual problem.

How should we proceed with this?
Comment 3 Eivin Giske Skaaren 2015-04-23 19:48:47 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2015-04-29 05:59:53 UTC
Created attachment 38632 [details] [review]
[PASSED QA] Bug 14006: about.pl checks the wrong zebra index mode

When setting zebra_bib_index_mode to grs1 I get two warnings when not applying the patch:

"The <zebra_bib_index_mode> entry is set to grs1. GRS-1 support is now deprecated and will be removed in future releases. Please use DOM instead by setting <zebra_bib_index_mode> to dom (full reindex required)."

"You have set <use_zebra_facets> but the <zebra_bib_index_mode> is not set to dom. Falling back to legacy facet calculation."

When applying the patch a third warning appears in addition to the two previous ones:

"The <zebra_bib_index_mode> entry is set to dom, but your system still appears to be set up for grs1 indexing."

Seems like the patch does what it should to me regarding the configuration mismatch warning.

Signed-off-by: Eivin Giske Skaaren <eskaaren@yahoo.no>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Tomás Cohen Arazi 2015-05-05 18:44:17 UTC
Fix pushed to master.
Comment 6 Chris Cormack 2015-05-17 05:36:48 UTC
Pushed to 3.18.x will be in 3.18.7
Comment 7 Mason James 2015-05-29 08:22:14 UTC
Pushed to 3.16.x, will be in 3.16.11