Attachment 823 Details for Bug 2746 – fix the bug

[patch] fix the bug

0001--bug-2746-Do-not-allow-the-user-to-set-an-empty-ex.patch (text/plain), 1.08 KB, created by Chris Cormack on 2008-10-30 09:14:00 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Cormack

Created: 2008-10-30 09:14:00 UTC

Size: 1.08 KB

patch

obsolete

>From 36fb5ea3c05a2ac82db0e482f5eda611793917b7 Mon Sep 17 00:00:00 2001
>From: Nahuel Angelinetti <nahuel.angelinetti@biblibre.com>
>Date: Thu, 30 Oct 2008 17:06:56 +0100
>Subject: [PATCH] [bug #2746] Do not allow the user to set an empty expiration date
>
>If we empty the expiration date field, 0000-00-00 date is putted in the db.
>This patch keep the expiration date in the db, if the posted form have an empty account expiration date.
>---
> members/memberentry.pl |    1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
>diff --git a/members/memberentry.pl b/members/memberentry.pl
>index dba1818..60af896 100755
>--- a/members/memberentry.pl
>+++ b/members/memberentry.pl
>@@ -129,6 +129,7 @@ if ($op eq 'insert' || $op eq 'modify' || $op eq 'save') {
> 			push(@errors,"ERROR_$_");
> 		}
> 	}
>+    $newdata{'dateexpiry'} = $data{'dateexpiry'} unless $newdata{'dateexpiry'};
>   # check permission to modify login info.
>     if (ref($borrower_data) && ($borrower_data->{'category_type'} eq 'S') && ! (C4::Auth::haspermission($dbh,$userenv->{'id'},{'staffaccess'=>1})) )  {
> 		$NoUpdateLogin =1;
>-- 
>1.5.6.3
>

Actions: View | Diff | Splinter Review

Attachments on bug 2746: 823 | 824