https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14504
Jesse Weaver <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] | |m --- Comment #31 from Jesse Weaver <[email protected]> --- 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 [email protected] 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/
