http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9582 --- Comment #1 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- Created attachment 15208 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15208&action=edit Bug 9582: Remove unused code from members/pay.pl The subroutines writeoff() and get_writeoff_sth() (and the two package variables $writeoff_sth and $add_writeoff_sth) defined in members/pay.pl are not used at all. To test: 1) View members/pay.pl by looking at the record of a patron who has at least seven fines/fees and clicking the "Pay fines" tab after clicking the "Fines" tab on the left side 2) Write off a single item in the table on members/pay.pl using the button labelled "Write off" 3) Pay a single fine by clicking the "Pay" button 4) Pay a portion of a fine by selecting a checkbox on one of the lines in the table and clicking the "Pay amount" button 5) Select two lines in the table and click the "Pay selected" button 6) Write off all remaining fines If all your actions show up in the table on the Account tab, this patch works. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9582 Kyle M Hall <kyle@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #15208|0 |1 is obsolete| | --- Comment #3 from Kyle M Hall <kyle@bywatersolutions.com> --- Created attachment 15352 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=15352&action=edit Bug 9582: Remove unused code from members/pay.pl The subroutines writeoff() and get_writeoff_sth() (and the two package variables $writeoff_sth and $add_writeoff_sth) defined in members/pay.pl are not used at all. To test: 1) View members/pay.pl by looking at the record of a patron who has at least seven fines/fees and clicking the "Pay fines" tab after clicking the "Fines" tab on the left side 2) Write off a single item in the table on members/pay.pl using the button labelled "Write off" 3) Pay a single fine by clicking the "Pay" button 4) Pay a portion of a fine by selecting a checkbox on one of the lines in the table and clicking the "Pay amount" button 5) Select two lines in the table and click the "Pay selected" button 6) Write off all remaining fines If all your actions show up in the table on the Account tab, this patch works. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9582 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |jonathan.druart@biblibre.co | |m QA Contact| |jonathan.druart@biblibre.co | |m --- Comment #4 from Jonathan Druart <jonathan.druart@biblibre.com> --- QA comment: Hum, really strange :) These routines have been removed by Kyle (commit db946ffd58094ec6287f991493a8df7bd7298f71 - Bug 7597 - Part 2 - move sub writeoff to C4::Accounts) and reintroduced by Stéphane ( commit a8b168dc63df9d60e8e562f368f8bdbec520ffd8 - Bug 7671 : add a real primary key accountlines_id in accountlines). But the latter does not used them! Marked as Passed QA. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9582 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #6 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This patch has been pushed to master. -- You are receiving this mail because: You are watching all bug changes.