https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14504
Jesse Weaver <jwea...@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jwea...@bywatersolutions.co | |m --- Comment #31 from Jesse Weaver <jwea...@bywatersolutions.com> --- To add my own 2 cents: The primary reason that these patches modify 10 different files is that it was asked, as part of the QA process, that the $dbh parameter be removed from DelItemCheck. (Which requires that all of the connecting files be modified.) While the alternate solution is reasonable, it is not an existing coding pattern in Koha, and is none of this is in the coding guidelines. Barton and I will be working on rebasing the patches, but please keep the above in mind. -- 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/