Attachment 42356 Details for Bug 14717 – [PASSED QA] Bug 14717: Invalid dates in debarred column

[patch] [PASSED QA] Bug 14717: Invalid dates in debarred column

PASSED-QA-Bug-14717-Invalid-dates-in-debarred-colu.patch (text/plain), 1.42 KB, created by Katrin Fischer on 2015-09-03 21:22:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2015-09-03 21:22:02 UTC

Size: 1.42 KB

patch

obsolete

>From d000b5affdd4a910c4a6f25fe8207ba669e169f5 Mon Sep 17 00:00:00 2001
>From: Chris Cormack <chrisc@catalyst.net.nz>
>Date: Tue, 25 Aug 2015 10:00:20 +1200
>Subject: [PATCH] [PASSED QA] Bug 14717: Invalid dates in debarred column
>
>To test
>1/ Import a patron using the patron import tool, make sure they have
>no debarred column in the file
>2/ Check the database, notice the debarred column is 0000-00-00
>3/ For bonus points, checkout an item to that borrower, then check it in
>notice Koha errors
>4/ Apply patch
>5/ Import a new patron
>6/ Notice column is now NULL and that checkins work
>
>Signed-off-by: Eugene Espinoza <eugenegf@yahoo.com>
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> C4/Members.pm | 3 +++
> 1 file changed, 3 insertions(+)
>
>diff --git a/C4/Members.pm b/C4/Members.pm
>index 91c89f8..0f4aeba 100644
>--- a/C4/Members.pm
>+++ b/C4/Members.pm
>@@ -759,7 +759,10 @@ sub AddMember {
> 
>     # create a disabled account if no password provided
>     $data{'password'} = ($data{'password'})? hash_password($data{'password'}) : '!';
>+
>+    # we don't want invalid dates in the db (mysql has a bad habit of inserting 0000-00-00
>     $data{'dateofbirth'} = undef if( not $data{'dateofbirth'} );
>+    $data{'debarred'} = undef if ( not $data{'debarred'} );
> 
>     # get only the columns of Borrower
>     my @columns = $schema->source('Borrower')->columns;
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 14717: 41871 | 42331 | 42342 | 42343 | 42344 | 42356 | 42357 | 42358