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

--- Comment #8 from Jonathan Druart <jonathan.dru...@biblibre.com> ---
(In reply to Kyle M Hall from comment #5)
> I don't think so. That could cause future regressions if new found statuses
> are added. For example, there used to be a found status of 'F' for
> 'Finished' ( from before the existence of the old_reserves table iirc ). We
> never want to include any type of found hold when calculating a new hold's
> priority, set there is no need to list everything as a special case.

Looking at the code in C4/Reserves.pm and reserves/*, the 'found' value is
often compared to 'W' or 'T'. Both values are listed explicitly. That's why I
supposed it is safer to use the same way.
I added a small comment in the line (in order to get return if someone grep
found.*W.*T)

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to