Attachment 50514 Details for Bug 15757 – Bug 15757: Make issuelength default to null instead of 0

[patch] Bug 15757: Make issuelength default to null instead of 0

Bug-15757-Make-issuelength-default-to-null-instead.patch (text/plain), 1.40 KB, created by Kyle M Hall (khall) on 2016-04-21 14:21:13 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2016-04-21 14:21:13 UTC

Size: 1.40 KB

patch

obsolete

>From 02db004d5117522bcecf8fb3622f3b5df9d9f057 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Thu, 7 Apr 2016 10:13:43 +0100
>Subject: [PATCH] Bug 15757: Make issuelength default to null instead of 0
>
>When editing circ rules, if the Loan period "issuelength" is not defined
>(empty string), the default value was 0, not it's inserted in the DB as
>NULL.
>
>Test plan:
>1/ Create or edit a circ rule
>2/ Do not fill the Loan period column
>3/ Save
>=> Without this patch, the value was 0
>=> With this patch it's now an empty string (in the DB it's set to NULL)
>
>Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
>Works as described.
>No errors
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> admin/smart-rules.pl | 1 +
> 1 file changed, 1 insertion(+)
>
>diff --git a/admin/smart-rules.pl b/admin/smart-rules.pl
>index da4e577..ade2993 100755
>--- a/admin/smart-rules.pl
>+++ b/admin/smart-rules.pl
>@@ -140,6 +140,7 @@ elsif ($op eq 'add') {
>     $maxonsiteissueqty =~ s/\s//g;
>     $maxonsiteissueqty = undef if $maxonsiteissueqty !~ /^\d+/;
>     my $issuelength  = $input->param('issuelength');
>+    $issuelength = $issuelength eq q{} ? undef : $issuelength;
>     my $lengthunit  = $input->param('lengthunit');
>     my $hardduedate = $input->param('hardduedate') || undef;
>     $hardduedate = eval { dt_from_string( $input->param('hardduedate') ) } if ( $hardduedate );
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 15757: 50013 | 50014 | 50015 | 50029 | 50131 | 50132 | 50133 | 50513 | 50514 | 50515 | 50516