[Koha-bugs] [Bug 17553] Move GetOverduesForPatron to Koha::Patron

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

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

--- Comment #5 from Jonathan Druart <jonathan.druart at bugs.koha-community.org> ---
(In reply to Marcel de Rooy from comment #4)
> Just from a glance, I can understand the ugly-qualification.
> 
> Do we really need these hashes where we dump all data from various tables?
> The original select was quite a lazy one.
> Why should an overdue object contain a title, if it contains an item or
> biblionumber? I guess most columns will not be used at all.

Yes all columns can be used later, they are used to build notifications.

> Are the assumptions on relation[0] and rs->[0] really future proof?

It works with current uses, that will need to be improve if we use other kind
of joins.

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