From 53d85de3c05c926f646b6ddf6bbb369ee4ceec85 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Wed, 22 Jan 2014 11:27:07 -0300 Subject: [PATCH] Bug 11596: Missing indexing options in koha-conf.xml should be reported To test: - Have a clean master install - Verify that koha-conf.xml contains valid entries for: zebra_bib_index_mode and zebra_auth_index_mode - Go to More > About Koha > System information => No "Warnings regarding the system configuration" - Apply the patch - Go to More > About Koha > System information => No "Warnings regarding the system configuration" - Edit your koha-conf.xml file commenting one or both indexing mode entries. - Go to More > About Koha > System information => You get "XML configuration file" warnings for each commented entry. - Sign off :-D Regards To+ Sponsored-by: Universidad Nacional de Cordoba Signed-off-by: Mark Tompsett Signed-off-by: Jonathan Druart --- about.pl | 15 +++++++++++++++ koha-tmpl/intranet-tmpl/prog/en/modules/about.tt | 11 +++++++++++ 2 files changed, 26 insertions(+) diff --git a/about.pl b/about.pl index 6b44ea8..e23e945 100755 --- a/about.pl +++ b/about.pl @@ -77,6 +77,20 @@ my $errZebraConnection = C4::Context->Zconn("biblioserver",0)->errcode(); my $warnIsRootUser = (! $loggedinuser); +my @xml_config_warnings; + +if ( ! defined C4::Context->config('zebra_bib_index_mode') ) { + push @xml_config_warnings, { + error => 'zebra_bib_index_mode_warn' + }; +} + +if ( ! defined C4::Context->config('zebra_auth_index_mode') ) { + push @xml_config_warnings, { + error => 'zebra_auth_index_mode_warn' + }; +} + $template->param( kohaVersion => $kohaVersion, osVersion => $osVersion, @@ -93,6 +107,7 @@ $template->param( warnPrefAnonymousPatron => $warnPrefAnonymousPatron, errZebraConnection => $errZebraConnection, warnIsRootUser => $warnIsRootUser, + xml_config_warnings => \@xml_config_warnings ); my @components = (); diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt index 21db9d3..847047b 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/about.tt @@ -117,6 +117,17 @@ Warning System preference 'OPACPrivacy' set, but AnonymousPatron preference is set to '0'. Set it to a valid borrower number if you want that this feature works correctly. [% END %] + [% ELSIF xml_config_warnings.size %] + + + [% FOREACH config_entry IN xml_config_warnings %] + [% IF config_entry.error == 'zebra_bib_index_mode_warn' %] + + [% ELSIF config_entry.error == 'zebra_auth_index_mode_warn' %] + + [% END %] + [% END %] +
XML configuration file
Warning The <zebra_bib_index_mode> entry is missing in your configuration file. It should be set to dom or grs1. It will default to grs1 but this could change in the future.
Warning The <zebra_auth_index_mode> entry is missing in your configuration file. It should be set to dom or grs1. It will default to dom but this could change in the future.
[% ELSE %]

No warnings

[% END %] -- 1.7.10.4