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/

Reply via email to