Bug 8724

Summary: Upgrade from NoZebra installs is broken
Product: Koha Reporter: Tomás Cohen Arazi <tomascohen>
Component: Installation and upgrade (command-line installer)Assignee: Tomás Cohen Arazi <tomascohen>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: normal    
Priority: P5 - low CC: jonathan.druart, kyle, paul.poulain
Version: 3.10   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7440
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 7440    
Attachments: Bug 8724 - Upgrade from NoZebra installs is broken
Bug 8724 - Upgrade from NoZebra installs is broken
Bug 8724 - Upgrade from NoZebra installs is broken

Description Tomás Cohen Arazi 2012-09-04 17:33:14 UTC
If the NoZebra syspref is set Koha gets busted after an upgrade, which doesnt warn the user about that.
Comment 1 Tomás Cohen Arazi 2012-09-04 18:07:55 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall 2012-09-07 13:44:11 UTC Comment hidden (obsolete)
Comment 3 Jared Camins-Esakov 2012-10-06 04:15:02 UTC
Upgrading severity.
Comment 4 Jonathan Druart 2012-10-19 14:53:52 UTC
Hi Tomás
Why don't you remove completely the sysprefs in the DB?
Comment 5 Tomás Cohen Arazi 2012-10-19 15:40:58 UTC
(In reply to comment #4)
> Hi Tomás
> Why don't you remove completely the sysprefs in the DB?

I was told on IRC that we don't usually delete them. Anyway, there's another bug for removing every NoZebra vestiges, I plan to contribute there.
Comment 6 Jared Camins-Esakov 2012-10-19 15:44:26 UTC
(In reply to comment #4)
> Hi Tomás
> Why don't you remove completely the sysprefs in the DB?

I encouraged Tomás not to remove the sysprefs yet, since there is still code that might be checking that syspref.
Comment 7 Jonathan Druart 2012-10-22 07:10:46 UTC
Created attachment 12994 [details] [review]
Bug 8724 - Upgrade from NoZebra installs is broken

Properly set the relevant NoZebra and QueryRemoveStopwords sysprefs to 0 to fix issues on upgrades.
Also removes the sysprefs from preferences/searching.pref

Regards
To+

Sponsored-by: Universidad Nacional de Córdoba
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 8 Jonathan Druart 2012-10-22 07:11:46 UTC
(In reply to comment #6)
> I encouraged Tomás not to remove the sysprefs yet, since there is still code
> that might be checking that syspref.

So this patch passes QA.
Comment 9 Paul Poulain 2012-10-22 13:01:52 UTC
Patch pushed to master (& bug reassigned to patch author, in case of a follow-up needed)