Bug 15544 - Zebra facets must be managed from the intranet
Summary: Zebra facets must be managed from the intranet
Status: Failed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: Searching (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Juan Romay Sieira
QA Contact: Testopia
URL:
Keywords:
: 10891 (view as bug list)
Depends on:
Blocks:
 
Reported: 2016-01-11 11:26 UTC by Juan Romay Sieira
Modified: 2020-06-10 06:38 UTC (History)
7 users (show)

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

Attachments
Patch to apply in master branch (22.38 KB, patch)
2016-01-13 11:06 UTC, Juan Romay Sieira 		Details | Diff | Splinter Review
SQL with a new facet table, to save the configuration (788 bytes, application/sql)
2016-01-13 11:07 UTC, Juan Romay Sieira 		Details
Facet SQL samples (1.59 KB, application/sql)
2016-01-13 11:14 UTC, Juan Romay Sieira 		Details
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 Juan Romay Sieira 2016-01-11 11:26:51 UTC 
Add the ability to manage the facets from the intranet. Nowadays the facets are hardcoded.
Comment 1 Juan Romay Sieira 2016-01-13 11:06:22 UTC 
Created attachment 46575 [details] [review]
Patch to apply in master branch
Comment 2 Juan Romay Sieira 2016-01-13 11:07:31 UTC 
Created attachment 46576 [details]
SQL with a new facet table, to save the configuration
Comment 3 Juan Romay Sieira 2016-01-13 11:10:03 UTC 
Test plan:
Apply the patch in master branch. Import the facet.sql file into your koha database.
Next go to Administration -> Facet indexes manager

You can create facets and test it appears into the search (intranet and opac)

There is a SQL with inserts of facet configuration samples.
Comment 4 Juan Romay Sieira 2016-01-13 11:14:29 UTC 
Created attachment 46578 [details]
Facet SQL samples
Comment 5 Katrin Fischer 2016-01-18 22:35:13 UTC 
As this has a new patch, I am marking the older bug 10891 as a duplicate.
Comment 6 Katrin Fischer 2016-01-18 22:35:27 UTC 
*** Bug 10891 has been marked as a duplicate of this bug. ***
Comment 7 Katrin Fischer 2016-08-23 16:19:11 UTC 
It looks like this might have got lost - it was never set to "Needs Sign-off".
Comment 8 Jonathan Druart 2016-08-24 11:22:09 UTC 
This is the old way to dev in Koha :)
You will need to correctly indent the template, use Koha::Objects for CRUD operations.
Moreover the new table is not added to kohastructure.sql and updatedatabase.pl, and you will need to populate it.
Comment 9 Jonathan Druart 2016-08-24 11:22:38 UTC 
Note that there is a patch in the queue for ES: bug 14899
Comment 10 David Cook 2020-06-09 23:39:13 UTC 
I'm not familiar with the Elasticsearch configurable facet code, but I wonder how easy it would be to adapt this Zebra code to work with it too.

(Although these days I think it would make more sense just to work towards fully deprecating Zebra to be honest.)
Comment 11 Katrin Fischer 2020-06-10 06:38:48 UTC 
(In reply to David Cook from comment #10)
> I'm not familiar with the Elasticsearch configurable facet code, but I
> wonder how easy it would be to adapt this Zebra code to work with it too.
> 
> (Although these days I think it would make more sense just to work towards
> fully deprecating Zebra to be honest.)

I think there is still a lot of libraries that profit from the low requirements of Zebra out there. I am not in favour of depracating it at this point in time. But we have certain features that are only developed for Elasticsearch.