Bug 11196

Summary: SearchEngine pref not in sysprefs.sql
Product: Koha Reporter: Srdjan Jankovic <srdjan>
Component: SearchingAssignee: Srdjan Jankovic <srdjan>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: major    
Priority: P1 - high CC: gmcharlt, jonathan.druart, mtompset
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 8233    
Bug Blocks:    
Attachments: bug_11196: put SearchEngine to sysprefs.sql
[SIGNED OFF] bug_11196: put SearchEngine to sysprefs.sql
bug_11196: put SearchEngine to sysprefs.sql
Bug 11196: Add SearchEngine syspref in the DB if missing

Description Srdjan Jankovic 2013-11-04 00:48:31 UTC
SearchEngine pref was introduced in updatedatabase.pl, but not in sysprefs.sql, so new installs don't have it set by default
Comment 1 Srdjan Jankovic 2013-11-04 01:06:19 UTC Comment hidden (obsolete)
Comment 2 Mark Tompsett 2013-11-11 06:55:40 UTC Comment hidden (obsolete)
Comment 3 Mark Tompsett 2013-11-11 07:32:50 UTC
I dropped my koha database, and did a fresh install from master. Indeed it was missing. I changed the version number back to 3.6000000 in the systempreferences table and forced an upgrade. Lo and behold it appeared. I dropped my koha database again, applied the patch, and did a fresh install again. This time the system preference was there as expected.

I even discovered Frameworksloaded, marcflavor, and Version are all created by the installer, when I deleted everything in the system preferences table and sourced the sql file directly. It was disorienting to see that the variable count had dropped. No worries, though. This all was a test system. :)
Comment 4 Jonathan Druart 2013-11-13 11:15:10 UTC
Created attachment 22904 [details] [review]
bug_11196: put SearchEngine to sysprefs.sql

Test:
Check that sysprefs.sql runs correctly.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 5 Jonathan Druart 2013-11-13 11:15:16 UTC
Created attachment 22905 [details] [review]
Bug 11196: Add SearchEngine syspref in the DB if missing

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 6 Jonathan Druart 2013-11-13 11:16:39 UTC
The pref should be added too for installation created between 3.09.00.023 and now.

Marked as Passed QA.
Comment 7 Srdjan Jankovic 2013-11-13 23:53:59 UTC
I should have thought of that myself... We resolved it by going to the syspref admin.
Comment 8 Galen Charlton 2013-11-14 19:08:35 UTC
Since this is easy to test and quiets log noise, I've pushed this to master and 3.14.x.  Thanks, Srdjan!