Attachment 10414 Details for Bug 6939 – [SIGNED-OFF] Bug 6939 DefaultClassificationSource should be default in 942

[patch] [SIGNED-OFF] Bug 6939 DefaultClassificationSource should be default in 942

SIGNED-OFF-Bug-6939-DefaultClassificationSource-sh.patch (text/plain), 1.31 KB, created by Nicole C. Engard on 2012-06-21 15:55:37 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nicole C. Engard

Created: 2012-06-21 15:55:37 UTC

Size: 1.31 KB

patch

obsolete

>From 0390e3699fd6320a584a740d20fc1c79dbbeb161 Mon Sep 17 00:00:00 2001
>From: Colin Campbell <colin.campbell@ptfs-europe.com>
>Date: Mon, 18 Jun 2012 11:23:42 +0100
>Subject: [PATCH] [SIGNED-OFF] Bug 6939 DefaultClassificationSource should be default in 942
>
>A logic bug in the original code adding DefaultClassificationSource
>caused the default not to be used (the last used value was used instead)
>The bug was fixed in additem by bug 2398 but addbiblio's use
>of the code was not fixed.
>
>Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
>---
> cataloguing/addbiblio.pl |    4 +---
> 1 files changed, 1 insertions(+), 3 deletions(-)
>
>diff --git a/cataloguing/addbiblio.pl b/cataloguing/addbiblio.pl
>index 643fc53..3db1a65 100755
>--- a/cataloguing/addbiblio.pl
>+++ b/cataloguing/addbiblio.pl
>@@ -215,10 +215,8 @@ sub build_authorized_values_list {
>                         ($class_source eq $default_source);
>             push @authorised_values, $class_source;
>             $authorised_lib{$class_source} = $class_sources->{$class_source}->{'description'};
>-            $value = $class_source unless ($value);
>-            $value = $default_source unless ($value);
>         }
>-        #---- "true" authorised value
>+        $value = $default_source unless $value;
>     }
>     else {
>         $authorised_values_sth->execute(
>-- 
>1.7.2.3

Actions: View | Diff | Splinter Review

Attachments on bug 6939: 10347 | 10414