Attachment 51604 Details for Bug 16517 – [PASSED QA] Bug 16517: Do not crash when creating a list with an existing name

[patch] [PASSED QA] Bug 16517: Do not crash when creating a list with an existing name

PASSED-QA-Bug-16517-Do-not-crash-when-creating-a-l.patch (text/plain), 1.42 KB, created by Katrin Fischer on 2016-05-18 22:09:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Katrin Fischer

Created: 2016-05-18 22:09:50 UTC

Size: 1.42 KB

patch

obsolete

>From d500e73fa51bf818784655390df9f1a0f67d9e1b Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Fri, 13 May 2016 20:21:12 +0100
>Subject: [PATCH] [PASSED QA] Bug 16517: Do not crash when creating a list with
> an existing name
>
>A typo has been made during a merge conflict when rebasing bug 16517.
>
>it should be
>  eval { Koha::Object->new->store; };
>instead of
>  eval { Koha::Object->new; }->store;
>
>Test plan:
>From the result list (intranet side), check some entries and add them to
>a new list
>Use an existing list's name
>=> Without this patch you will get a server error (exception 'Duplicate shelf
>object' raised by Koha::Virtualshelf->new->store)
>=> With this patch you must get a friendly error message
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>---
> virtualshelves/addbybiblionumber.pl | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/virtualshelves/addbybiblionumber.pl b/virtualshelves/addbybiblionumber.pl
>index 790904f..71270b7 100755
>--- a/virtualshelves/addbybiblionumber.pl
>+++ b/virtualshelves/addbybiblionumber.pl
>@@ -132,8 +132,8 @@ sub HandleNewVirtualShelf {
>                 sortfield => $sortfield,
>                 owner => $loggedinuser,
>             }
>-        );
>-    }->store;
>+        )->store;
>+    };
>     if ( $@ or not $shelf ) {
>         $authorized = 0;
>         $errcode    = 1;
>-- 
>1.9.1

Actions: View | Diff | Splinter Review

Attachments on bug 16517: 51482 | 51491 | 51604