Duplicates the code for searching by reviewer in the form that is used in
searching via a link.
It looks like it was the intention of the original writer to filter by borrower
number or by name. Filtering by name was not implemented. Added a FIXME
reference to indicate this.
---
tags/review.pl | 7 ++++---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/tags/review.pl b/tags/review.pl
index 0667f3d..d4db972 100755
--- a/tags/review.pl
+++ b/tags/review.pl
@@ -178,11 +178,12 @@ if ($filter = $input->param('to')) {
}
}
if ($filter = $input->param('approver')) { # name (or
borrowernumber) from input box
- if (($filter =~ /^\d+$/ and $filter > 0) or
- (1) ){ # $filter=get borrowernumber from name
+ if ($filter =~ /^\d+$/ and $filter > 0) {
+ # $filter=get borrowernumber from name
+ # FIXME: get borrowernumber from name not implemented.
$template->param(filter_approver=>$filter);
$filters{approved_by} = $filter;
- # } else {
+ } else {
push @errors, {approver=>$filter};
}
}
--
1.5.6.5
_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha.org/mailman/listinfo/koha-patches
