(bug workaround): Commented out some code that removes everything from

the list of branches the user can select.
Added some FIXME comments.

diff --git a/circ/selectbranchprinter.pl b/circ/selectbranchprinter.pl
index 83a9804..a8d5794 100644 (file)
--- 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);