Attachment 50015 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.26 KB, created by Jonathan Druart on 2016-04-07 09:42:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2016-04-07 09:42:38 UTC

Size: 1.26 KB

patch

obsolete

>From f2f79fd6490a113d5a16123990cfbd42f9c607a1 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)
>---
> 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.7.0

Actions: View | Diff | Splinter Review

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