https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34972
--- Comment #2 from Emily Lamancusa <[email protected]> --- Also making a note for later: GetOtherReserves should probably be removed entirely. Once the bad code is removed, it's basically just a wrapper for CheckReserves that returns a trivial additional message. It's also only called in two places, and has almost no test coverage. ModReserveMinusPriority (which is called by GetOtherReserves and is the actual culprit here) should also be removed. It also has almost no test coverage, only has one remaining reference outside of GetOtherReserves, and is currently redundant and unnecessary in that place also. Will make follow-up bugs for those after I upload patches for this one. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] 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/
