Bug 15485: (QA followup) Fix behaviour and default values
authorTomas Cohen Arazi <tomascohen@theke.io>
Tue, 14 Jun 2016 16:16:13 +0000 (13:16 -0300)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 24 Jun 2016 13:30:31 +0000 (13:30 +0000)
commitadd61ef614d171dbc4ecda95ec8cfb70a1ad40a0
tree1299bb5f5520af48c169fa5f63e659a0e0cac7aa
parent04143f6a9cf2247d8c3db9da2f492f311dc753ec
Bug 15485: (QA followup) Fix behaviour and default values

Jonathan noticed the current behaviour is that the lists rendering
falls back to the XSLTResults, and if not defined (i.e. != 'default' and
!= some_path) it falls back to a legacy display (non-XSLT).

The patchset changed this behaviour because 'default' is not a valid value
for the lists. So it should fallback to the current behaivour (i.e. Results
XSLT configuration) if not defined. This patch fixes this by adding
 || C4::Context->preference('XSLTResultsDisplay')
(and the OPAC counterpart).

It also fixes minor glitches on the update message (oops) and bad default value
in sysprefs.sql  for 'XSLTListsDisplay'.

Thanks Jonathan!

Signed-off-by: Hector Castro <hector.hecaxmmx@gmail.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
installer/data/mysql/atomicupdate/bug_15485_xslt_for_lists.sql
installer/data/mysql/sysprefs.sql
opac/opac-shelves.pl
virtualshelves/shelves.pl