From: Joshua Ferraro <[EMAIL PROTECTED]>
For some reason the loop used to build the list of values was
setting the first value to the $value variable if $value wasn't
set already, and the default_value was inside rather than outside
the loop. I've removed the setting inside the loop and placed
the default value outide the loop. It's possible I just don't
understand what the original intention was.
Signed-off-by: Galen Charlton <[EMAIL PROTECTED]>
---
cataloguing/additem.pl | 5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/cataloguing/additem.pl b/cataloguing/additem.pl
index da09dd0..4c9ecfc 100755
--- a/cataloguing/additem.pl
+++ b/cataloguing/additem.pl
@@ -371,10 +371,9 @@ foreach my $tag (sort keys %{$tagslib}) {
($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);
}
-
+ $value = $default_source unless ($value);
+
#---- "true" authorised value
}
else {
--
1.5.5.GIT
_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha.org/mailman/listinfo/koha-patches
