Attachment 46907 Details for Bug 14538 – Bug 14538: editing perl doc for CalcFine

[patch] Bug 14538: editing perl doc for CalcFine

Bug-14538-editing-perl-doc-for-CalcFine.patch (text/plain), 1.78 KB, created by Mark Tompsett on 2016-01-19 16:23:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mark Tompsett

Created: 2016-01-19 16:23:57 UTC

Size: 1.78 KB

patch

obsolete

>From f9e2043c27395547d91836f74bb1f7443496c70e Mon Sep 17 00:00:00 2001
>From: Natasha <tasham_8@hotmail.com>
>Date: Mon, 18 Jan 2016 22:59:03 +0000
>Subject: [PATCH] Bug 14538: editing perl doc for CalcFine
>
>To Test:
>Read perldoc C4/Overdues.pm
>Confirm that it says CalcFine returns four values and that rest of the documentation is correct
>
>NOTE: only a little visual ugliness, but definitely an improvement.
>
>Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
>---
> C4/Overdues.pm | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/C4/Overdues.pm b/C4/Overdues.pm
>index f5e7d17..37711ba 100644
>--- a/C4/Overdues.pm
>+++ b/C4/Overdues.pm
>@@ -204,7 +204,7 @@ sub checkoverdues {
> 
> =head2 CalcFine
> 
>-    ($amount, $chargename,  $daycounttotal) = &CalcFine($item,
>+    ($amount, $chargename,  $units_minus_grace, $chargeable_units) = &CalcFine($item,
>                                   $categorycode, $branch,
>                                   $start_dt, $end_dt );
> 
>@@ -236,10 +236,10 @@ C<$amount> is the fine owed by the patron (see above).
> C<$chargename> is the chargename field from the applicable record in
> the categoryitem table, whatever that is.
> 
>-C<$unitcount> is the number of chargeable units (days between start and end dates, Calendar adjusted where needed,
>-minus any applicable grace period, or hours)
>+C<$units_minus_grace> is the number of chargeable units minus the grace period
> 
>-FIXME - What is chargename supposed to be ?
>+C<$chargeable_units> is the number of chargeable units (days between start and end dates, Calendar adjusted where needed,
>+minus any applicable grace period, or hours)
> 
> FIXME: previously attempted to return C<$message> as a text message, either "First Notice", "Second Notice",
> or "Final Notice".  But CalcFine never defined any value.
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 14538: 46831 | 46907 | 46909