Just saw that Makefile.PL still has this: 

 884     $msg = q(
 885 Koha can use the Zebra search engine for high-performance
 886 searching of bibliographic and authority records.  If you
 887 have installed the Zebra software and would like to use it,
 888 please answer 'yes' to the following question.  Otherwise,
 889 Koha will default to using its internal search engine.
 890 
 891 Please note that if you choose *NOT* to install Zebra,
 892 koha-conf.xml will still contain some references to Zebra
 893 settings.  Those references will be ignored by Koha.
 894 
 895 Install the Zebra configuration files?);

(In reply to comment #0)
> NoZebra has been deprecated since 3.4.0. It is time to remove the vestiges
> of NoZebra from the code and eliminate the syspref so that people stop
> thinking they can use it.

'stopwords' code and syspref needs to be removed too

Upgrade from NoZebra installs is broken, as there is code that uses the syspref, which should be completely removed from Koha.

To test:

- Apply the patch and do a fresh install
- Install procedure should "just work as usual"
- The nozebra table should not be created
- Searching for authorities should remain functional
- Add/Modify a biblio/authority record and nothing crashes
- Add an item and nothing crashes

Keep in mind that most of this patch is the removal of unused contitional if NoZebra blocks, templates showind NoZebra stuff and removal of the sysprefs in te relevant SQL files.

Patch doesn't apply anymore on master (conflicts on C4/AuthoritiesMarc.pm and C4/Search.pm).

I did a fresh install after applying the patches and can confirm that Koha is still working

- web installer OK
- nozebra table is not created OK
- add/modify/delete biblio/item/authority OK
- rebuild_zebra.pl OK
- search biblio/authority OK

so I will sign off these patches.

However, it still remains some references to NoZebra that could be removed:

C4/AuthoritiesMarc.pm:331:    if (C4::Context->preference('NoZebra')) {
C4/AuthoritiesMarc.pm:1416:    if (C4::Context->preference('NoZebra')) {
C4/AuthoritiesMarc.pm:1417:        #nozebra way    
C4/AuthoritiesMarc.pm:1419:        my $rq=$dbh->prepare(qq(SELECT biblionumbers from nozebra where indexname="an" and server="biblioserver" and value="$mergefrom" ));
C4/Search.pm:1260:    # no stemming/weight/fuzzy in NoZebra
C4/Search.pm:1261:    if ( C4::Context->preference("NoZebra") ) {
koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/stopwords.tt:5:<p style="color: #990000">Important: If NoZebra is set to 'Use' this option will not appear on the admin

(In reply to comment #11)
> koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/stopwords.tt

Shouldn't that template be removed by this patch?

Created attachment 16437 [details] [review]
[PATCH] Bug 7440 - Remove NoZebra vestiges

Removed NoZebra vestiges. This comprises several code blocks that depend on the NoZebra syspref and NZ related functions/methods.

C4::Biblio->
 GetNoZebraIndexes
 _DelBiblioNoZebra
 _AddBiblioNoZebra

C4::Search->
 NZgetRecords
 NZanalyse
 NZoperatorAND
 NZoperatorOR
 NZoperatorNOT
 NZorder

C4::Installer->
 set_indexing_engine

Sponsored-by: Universidad Nacional de Córdoba
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>

Created attachment 16438 [details] [review]
Bug 7440 - Remove NoZebra and QueryRemoveStopWords sysprefs

Sponsored-by: Universidad Nacional de Córdoba
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>

Created attachment 16439 [details] [review]
Bug 7440 follow-up removing forgotten NoZebra references

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>

QA comments:

investigating, I've found some NoZebra references pending:

koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/stopwords.tt:<p style="color: #990000">Important: If NoZebra is set to 'Use' this option will not appear on the administration menu</p>
C4/Search.pm:    # no stemming/weight/fuzzy in NoZebra
C4/Search.pm:    if ( C4::Context->preference("NoZebra") ) {
C4/AuthoritiesMarc.pm:    if (C4::Context->preference('NoZebra')) {
C4/AuthoritiesMarc.pm:    if (C4::Context->preference('NoZebra')) {

I've attached a follow-up removing them as well.

I've tested as many pages as possible, and could not find anything going wrong

passes koha-qa

passed QA

(In reply to comment #18)
> QA comments:
> 
> investigating, I've found some NoZebra references pending:

There where some bits related to stopwords too, maybe those you find are related. I filled a bug for removing them, for which I haven't provided a bug as spent the time I had rebasing this one.

(In reply to comment #19)
> (In reply to comment #18)
> > QA comments:
> > 
> > investigating, I've found some NoZebra references pending:
> 
> There where some bits related to stopwords too, maybe those you find are
> related. I filled a bug for removing them, for which I haven't provided a
> bug as spent the time I had rebasing this one.

I don't think so, except for the stopword.tt line removed. Anyway, now there's no "NoZebra" references outside from old release notes, updatedatabase, so things are OK

This patch has been pushed to master.