[Koha-bugs] [Bug 14876] Show number of holds per record on the search results

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

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

Jonathan Druart <jonathan.druart at bugs.koha-community.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |katrin.fischer at bsz-bw.de,
                   |                            |m.de.rooy at rijksmuseum.nl,
                   |                            |martin.renvoize at ptfs-europe
                   |                            |.com, tomascohen at gmail.com

--- Comment #9 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
CCing Katrin, Tomas, Marcel and Martin.

It's not clear to me when we need a TT plugin.
Sometimes it's obvious (get branch names, get patron category description,
filter (json, date, price), pref values, etc.)
But sometimes it's not: Borrowers.IsDebarred, Borrowers.HasOverdues.
Holds.count is part of this second group.

I don't think plugins should be a lazy solution for developers.

We need to define when we can use a plugin, otherwise any methods from Koha
modules could have their plugin wrapper.

And we will have to maintain:
  Koha::Holds->count({biblionumber => $biblionumber });
  Koha::Template::Plugin::Holds->count($biblionumber);
and
  Koha::REST::V1::Hold->count($biblionumber); # does not exist (yet)

(I don't switch to in discussion otherwise you will hate me)

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