From eb57e3b92ab93a196100161ad4fb36e711fbc73e Mon Sep 17 00:00:00 2001 From: arensb Date: Sat, 12 Oct 2002 18:57:41 +0000 Subject: [PATCH] (bug workaround): Commented out some code that removes everything from the list of branches the user can select. Added some FIXME comments. --- circ/selectbranchprinter.pl | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/circ/selectbranchprinter.pl b/circ/selectbranchprinter.pl index 83a980403f..a8d5794cc1 100644 --- a/circ/selectbranchprinter.pl +++ b/circ/selectbranchprinter.pl @@ -68,7 +68,13 @@ my $branchoptions; my $printeroptions; foreach (keys %$branches) { (next) unless ($_); - (next) unless ($branches->{$_}->{'IS'}); + # FIXME - What is this "IS" field? I suspect it's a leftover from + # some previous iteration of the code, and means "this is a branch + # that does issues". But it never gets set, so no branches ever + # get selected, and the user can't choose a branch. + # FIXME - Also, shouldn't librarians be able to select any branch, + # not just ones that handle issues? +# (next) unless ($branches->{$_}->{'IS'}); $branchcount++; my $selected=''; ($selected='selected') if ($_ eq $oldbranch); -- 2.20.1