Attachment 67546 Details for Bug 14399 – Bug 14399: [16.05.X] Prevent display of double zero notforloan status

[patch] Bug 14399: [16.05.X] Prevent display of double zero notforloan status

Bug-14399-1605X-Prevent-display-of-double-zero-not.patch (text/plain), 1.73 KB, created by Marcel de Rooy on 2017-10-03 13:43:41 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2017-10-03 13:43:41 UTC

Size: 1.73 KB

patch

obsolete

>From 2bb5640a21eb5dadb8bb0f4a17d365d14ec4500b Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Mon, 3 Apr 2017 13:05:13 +0200
>Subject: [PATCH] Bug 14399: [16.05.X] Prevent display of double zero
> notforloan status
>Content-Type: text/plain; charset=utf-8
>
>On bug 12913 a zero status is added on the inventory form. This prevents
>a lot of false warnings for a wrong notforloan status. The zero status
>is not included in the default setup. But if you would add one, the
>status will display here twice now.
>
>This patch checks if the status already exists.
>
>Test plan:
>[1] Add a zero NOTFORLOAN status if it does not yet exist in Authorized
>    values.
>[2] Check that you do not see two "For loan" statuses on inventory form.
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>---
> tools/inventory.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/tools/inventory.pl b/tools/inventory.pl
>index 8a6f3ac..76dad26 100755
>--- a/tools/inventory.pl
>+++ b/tools/inventory.pl
>@@ -104,7 +104,7 @@ for my $statfield (qw/items.notforloan items.itemlost items.withdrawn items.dama
>         if ( $statfield eq 'items.notforloan') {
>             # Add notforloan == 0 to the list of possible notforloan statuses
>             # The lib value is replaced in the template
>-            push @$arr, { authorised_value => 0, id => 'stat0' , lib => 'ignore' };
>+            push @$arr, { authorised_value => 0, id => 'stat0' , lib => 'ignore' } if ! grep { $_->{authorised_value} eq '0' } @$arr;
>             @notforloans = map { $_->{'authorised_value'} } @$arr;
>         }
>         $hash->{values} = $arr;
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 14399: 61685 | 62096 | 62097 | 62098 | 62099 | 62139 | 62140 | 62733 | 62734 | 62735 | 62736 | 63465 | 63466 | 63467 | 63468 | 67546 | 67547 | 67548 | 67549