[Koha-patches] [PATCH] Bug 11596: Missing indexing options in koha-conf.xml should be reported
Tomas Cohen Arazi
tomascohen at gmail.com
Wed Jan 22 15:32:50 CET 2014
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
---
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 @@
<tr><th scope="row"><b>Warning</b> </th><td>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.</td></tr>
[% END %]
</table>
+ [% ELSIF xml_config_warnings.size %]
+ <table>
+ <caption>XML configuration file</caption>
+ [% FOREACH config_entry IN xml_config_warnings %]
+ [% IF config_entry.error == 'zebra_bib_index_mode_warn' %]
+ <tr><th scope="row"><b>Warning</b> </th><td>The <zebra_bib_index_mode> entry is missing in your configuration file. It should be set to <strong>dom</strong> or <strong>grs1</strong>. It will default to <strong>grs1</strong> but this could change in the future.</td></tr>
+ [% ELSIF config_entry.error == 'zebra_auth_index_mode_warn' %]
+ <tr><th scope="row"><b>Warning</b> </th><td>The <zebra_auth_index_mode> entry is missing in your configuration file. It should be set to <strong>dom</strong> or <strong>grs1</strong>. It will default to <strong>dom</strong> but this could change in the future.</td></tr>
+ [% END %]
+ [% END %]
+ </table>
[% ELSE %]
<p>No warnings</p>
[% END %]
--
1.8.3.2
More information about the Koha-patches
mailing list