[Koha-bugs] [Bug 12478] Elasticsearch support for Koha

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12478

--- Comment #125 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
Robin,
Another set of patches is coming.
1/ Reintroduce the SearchEngine pref
2/ Fix to correct the number of facets displayed (again)
3/ Remove an unnecessary "&limit=" (minor)
4/ Replace the Koha::ItemType[s] classes you have introduced with a
Koha::Object[s] way
5/ Display description instead of code for locations in facets.

About 4:
I know you are now agreeing completely with Koha::Object[s] and how it has been
implemented.
But I think we need to have an homogeneous code, and all new classes in the
Koha namespace should follow the same directions.
I would please you to have a look at bug 14828 (and the see also ones), which
introduces the Koha::ItemType[s] classes as well and rewrites the
admin/itemtypes.pl script, to see how easy it is to use it.
To avoid future conflicts and don't play to a race against each other we should
agree on this kind of decisions now.

-- 
You are receiving this mail because:
You are watching all bug changes.