https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3142
--- Comment #25 from Jonathan Druart <jonathan.druart+k...@gmail.com> --- (In reply to Marcel de Rooy from comment #23) > + my @hold_not_allowed_itypes = Koha::CirculationRules->search( > + { > + rule_name => 'holdallowed', > + branchcode => undef, > + categorycode => undef, > + rule_value => 'not_allowed', > + } > + )->get_column('itemtype'); > > I have my doubts here btw. > This is too simple. You are looking at circ rules with exception for various > combinations of category, itemtype etc. > You cant just do a get_column here. This depends on context. > Please clarify and add tests. Which context? The logged in branch? Here we are not in C4::Reserves, we don't want to offer all the complexity of CanItemBeReserved. As said previously I am not adding new code of functionality (apart for the additional damaged check). Tests are definitely missing, agreed. -- You are receiving this mail because: You are watching all bug changes. You are the QA Contact for the bug. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://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/