Bug 10134 - set XSLT usage ON as default, for new Koha installs
Summary: set XSLT usage ON as default, for new Koha installs
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Mason James
QA Contact:
URL:
Keywords:
: 9867 (view as bug list)
Depends on:
Blocks:
 
Reported: 2013-04-27 08:27 UTC by Mason James
Modified: 2013-12-05 20:09 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Bug 10134 - set XSLT usage ON as default, for new Koha installs (2.93 KB, patch)
2013-05-08 09:14 UTC, Magnus Enger 		Details | Diff | Splinter Review
[SIGNED-OFF] Bug 10134 - set XSLT usage ON as default, for new Koha installs (3.06 KB, patch)
2013-05-09 12:54 UTC, Owen Leonard 		Details | Diff | Splinter Review
[PASSED QA] Bug 10134 - set XSLT usage ON as default, for new Koha installs (3.22 KB, patch)
2013-05-12 09:17 UTC, Katrin Fischer 		Details | Diff | Splinter Review
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mason James 2013-04-27 08:27:29 UTC 
currently XSLT usage is set OFF as default, for new Koha installs

this patch will set XSLT usage as ON, instead...
Comment 1 Magnus Enger 2013-04-27 09:22:25 UTC 
*** Bug 9867 has been marked as a duplicate of this bug. ***
Comment 2 Magnus Enger 2013-05-08 09:14:39 UTC Comment hidden (obsolete)
Comment 3 Owen Leonard 2013-05-09 12:54:22 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2013-05-12 09:17:36 UTC 
Created attachment 18090 [details] [review]
[PASSED QA] Bug 10134 - set XSLT usage ON as default, for new Koha installs

Currently, new installs are created with XSLT display set to "off",
but XSLT seems to be the preferred way to display records, so
having the default be "on" instead seems to make sense. This patch
will of course only affect new installs.

To test:
- Apply the patch
- Run through the webinstaller on an empty database
- Log in to the staff client and search for "xslt" in system
  preferences
- Verify that XSLTDetailsDisplay, XSLTResultsDisplay,
  OPACXSLTDetailsDisplay and OPACXSLTResultsDisplay are all set to
  "default"

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Passes test plan correctly.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Changes syspref.sql for new installations to use XSLT views
by default. No changes to existing installations, no
string changes.
Passes all tests.
Comment 5 Jared Camins-Esakov 2013-05-12 14:05:58 UTC 
This patch has been pushed to master and 3.12.x.
Comment 6 Chris Cormack 2013-05-15 21:20:26 UTC 
This changes the default behaviour, so not putting this in 3.10.x or 3.8.x