Bug 13087

Summary: Add a notice for use_zebra_facets problems in about page
Product: Koha Reporter: Tomás Cohen Arazi (tcohen) <tomascohen>
Component: AboutAssignee: Tomás Cohen Arazi (tcohen) <tomascohen>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: tomascohen
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: Sponsored
Patch complexity: Small patch Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 11232    
Bug Blocks:    
Attachments: Bug 13087: Add a notice for use_zebra_facets problems in about page
[SIGNED-OFF] Bug 13087: Add a notice for use_zebra_facets problems in about page
[PASSED QA] Bug 13087: Add a notice for use_zebra_facets problems in about page

Description Tomás Cohen Arazi (tcohen) 2014-10-15 16:06:00 UTC
11232 introduced the use of Zebra facets. A way of telling the user that a conflicting configuration is set needs to be added to the about page.
Comment 1 Tomás Cohen Arazi (tcohen) 2014-10-22 19:08:13 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2014-10-23 15:05:39 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2014-10-24 05:42:57 UTC
Created attachment 32652 [details] [review]
[PASSED QA] Bug 13087: Add a notice for use_zebra_facets problems in about page

This patch adds two checks to about.pl:

- use_zebra_facets entry presence in koha-conf.xml
- use_zebra_facets==1 and zebra_bib_index_mode==dom

The idea is to alert the user if it has conflicting or wrong settings. Even
if there's a fallback to a working configuration.

To test:
- On clean master install, go to the about page
=> SUCCESS: No warnings regarding use_zebra_facets
- Comment out the use_zebra_facets entry, reload
=> SUCCESS: The about page mentions the use_zebra_facets entry is missing
- Uncomment the use_zebra_facets entry, make sure it is set to 1
- Change zebra_bib_index_mode to 'grs1', reload
=> SUCCESS: The about page says zebra_bib_index_mode is not set to DOM even though
  you have use_zebra_facets set.
- Sign off :-D

Regards
To+

Sponsored-by: Universidad Nacional de Cordoba
Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Followed test plan with success

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, tested by changing the koha-conf.xml file.
Note: There is no warning if use_zebra_facets is 0.
Comment 4 Tomás Cohen Arazi (tcohen) 2014-10-24 13:04:15 UTC
Patch pushed to master.