[Bug 14006] New: about.pl checks the wrongzebra index mode
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 Bug ID: 14006 Summary: about.pl checks the wrongzebra index mode Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: major Priority: P5 - low Component: About Assignee: nengard@gmail.com Reporter: magnus@enger.priv.no QA Contact: testopia@bugs.koha-community.org 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')? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 Magnus Enger <magnus@enger.priv.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|about.pl checks the |about.pl checks the wrong |wrongzebra index mode |zebra index mode -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|major |normal CC| |tomascohen@gmail.com -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Patch complexity|--- |Trivial patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 --- Comment #1 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Created attachment 38207 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=38207&action=edit Bug 14006: about.pl checks the wrong zebra index mode This trivial patch makes the about page correctly test for configuration mismatches. Specifically it fixes the situation in which the koha-config.xml file points to zebra-biblios-dom.xml but has 'zebra_bib_index_mode' set to grs1. The test incorrectly queried for 'zebra_auth_index_mode' being set to grs1.' To test: - Have a DOM setup. - Change koha-conf.xml so zebra_bib_index_mode = grs1 => FAIL: No configuration missmatch is shown. - Apply the patch - Reload the about page => SUCCESS: A zebra configuration missmatch is shown. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 M. Tompsett <mtompset@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mtompset@hotmail.com --- Comment #2 from M. Tompsett <mtompset@hotmail.com> --- (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? -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 Eivin Giske Skaaren <eskaaren@yahoo.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #38207|0 |1 is obsolete| | --- Comment #3 from Eivin Giske Skaaren <eskaaren@yahoo.no> --- Created attachment 38463 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=38463&action=edit 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> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 Eivin Giske Skaaren <eskaaren@yahoo.no> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off CC| |eskaaren@yahoo.no -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #38463|0 |1 is obsolete| | --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Created attachment 38632 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=38632&action=edit [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> -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 Tomás Cohen Arazi <tomascohen@gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #5 from Tomás Cohen Arazi <tomascohen@gmail.com> --- Fix pushed to master. -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |chris@bigballofwax.co.nz Status|Pushed to Master |Pushed to Stable --- Comment #6 from Chris Cormack <chris@bigballofwax.co.nz> --- Pushed to 3.18.x will be in 3.18.7 -- You are receiving this mail because: You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14006 Mason James <mtj@kohaaloha.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mtj@kohaaloha.com --- Comment #7 from Mason James <mtj@kohaaloha.com> --- Pushed to 3.16.x, will be in 3.16.11 -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon＠bugs.koha-community.org