From 26383113d0cc7edf58458795349b252dfbee25c6 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 10 Jul 2015 11:56:14 +0100 Subject: [PATCH] Bug 6379: Fix if the reviewer is not the same patron get_approval_rows does a left join on tags_approval.approved_by = borrowers.borrowernumber This means it cannot return the approval tags for a given patron. Note that this patch could be time-consuming if there is a lot of tags in the DB. Signed-off-by: Katrin Fischer Signed-off-by: Tomas Cohen Arazi --- opac/opac-tags.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/opac/opac-tags.pl b/opac/opac-tags.pl index 19994d1d48..4f97896d05 100755 --- a/opac/opac-tags.pl +++ b/opac/opac-tags.pl @@ -227,7 +227,7 @@ my $my_tags = []; if ($loggedinuser) { $my_tags = get_tag_rows({borrowernumber=>$loggedinuser}); - my $my_approved_tags = get_approval_rows({borrowernumber => $loggedinuser, approved => 1}); + my $my_approved_tags = get_approval_rows({ approved => 1 }); foreach my $tag (@$my_tags) { my $biblio = GetBiblioData($tag->{biblionumber}); my $record = &GetMarcBiblio( $tag->{biblionumber} ); -- 2.39.5