Attachment 23080 Details for Bug 11282 – Bug 11282 - Not possible to create multiple restrictions from patron details/new restriction overwrites existing

[patch] Bug 11282 - Not possible to create multiple restrictions from patron details/new restriction overwrites existing

Bug-11282---Not-possible-to-create-multiple-restri.patch (text/plain), 1.08 KB, created by Kyle M Hall (khall) on 2013-11-22 13:33:37 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2013-11-22 13:33:37 UTC

Size: 1.08 KB

patch

obsolete

>From 6fff48feac89e69a2eeb0af94650822041761fc8 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Fri, 22 Nov 2013 08:31:31 -0500
>Subject: [PATCH] Bug 11282 - Not possible to create multiple restrictions from patron details/new restriction overwrites existing
>
>Test Plan:
>1) Edit a patron, add a restriction
>2) Edit the patron again, add a second restriction
>3) Note the first restriction has disappeared!
>4) Apply this patch
>5) Edit the patron again, add another restriction
>6) Note the previous restriction is not longer removed
>---
> members/memberentry.pl |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/members/memberentry.pl b/members/memberentry.pl
>index 0d3a3c3..591aa7c 100755
>--- a/members/memberentry.pl
>+++ b/members/memberentry.pl
>@@ -110,7 +110,7 @@ if ( $input->param('add_debarment') ) {
>         { 'dt' => dt_from_string($expiration), 'dateformat' => 'iso' } )
>       : undef;
> 
>-    AddUniqueDebarment(
>+    AddDebarment(
>         {
>             borrowernumber => $borrowernumber,
>             type           => 'MANUAL',
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 11282: 23080 | 23808 | 23816 | 23817