[Koha-bugs] [Bug 24573] Catmandu::Store::ElasticSearch and Catmandu::MARC are missing from cpanfile

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

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24573

Marcel de Rooy <m.de.rooy at rijksmuseum.nl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |In Discussion

--- Comment #10 from Marcel de Rooy <m.de.rooy at rijksmuseum.nl> ---
(In reply to Julian Maurice from comment #9)
> (In reply to Marcel de Rooy from comment #7)
> > > Catmandu::Store::ElasticSearch is required for search,
> > > and Catmandu::MARC is required for indexation
> > 
> > No, we still have Zebra. I think it should be a recommend then?
> 
> There is no rule yet for deciding when a module should be required or
> recommended.
> I think it should be required because when you turn on the feature, Koha
> will die if these modules are not installed, and there is nothing preventing
> you to turn on the feature.

It seems to be that they should not be required now, since we did not require
them before in PerlDependencies. In one of the tests we are mocking the module
for such a reason iirc.

In Discussion? Please mail to the dev list? Or dev meeting agenda point?

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