[Koha-bugs] [Bug 10256] Remove some unused subs from Overdues module, or do not export them

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Thu May 16 10:07:40 CEST 2013


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10256

--- Comment #1 from M. de Rooy <m.de.rooy at rijksmuseum.nl> ---
Created attachment 18165
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=18165&action=edit
Bug 10256: Remove some unused subs from Overdues module

Removes the following exported but unused subs from Overdues.pm:
CreateItemAccountLine
UpdateAccountLines
CheckAccountLineLevelInfo
CheckAccountLineItemInfo
CheckExistantNotifyid
GetNextIdNotify
GetNotifyId
ReplacementCost
ReplacementCost2
GetOverdueDelays
GetOverduerules

Test plan:
It is hard to test the removal of something that was not used :) Try this:
Do a recursive grep on these routine names in the Koha codebase.
Compile some scripts that use the Overdues module.
And just to be sure we do not break something:
Go to Circulation: Do a checkout, checkin, place and confirm a hold.
Go to Patrons: Goto Check out. Goto Fines.
Run the command line scripts: fines.pl and overdue_notices.pl.
Go to opac-user.pl.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list