[Koha-patches] [PATCH 3/5] [SIGNED-OFF] Bug 6298 : Hide avatars when ShowReviewer is off

[Nicole C. Engard]

From: Francois Marier <francois at debian.org>

As suggested by Nicole, when a library hides the names of the
reviewers, they probably don't want to show their faces.

The OPAC detail page was already hiding these avatars but this
change also prevents Koha from looking for these avatars (which
could incur DNS queries) if they're not going to be displayed.

Signed-off-by: Francois Marier <francois at debian.org>
Signed-off-by: Nicole C. Engard <nengard at bywatersolutions.com>
---
 opac/opac-detail.pl      |    9 ++++-----
 opac/opac-showreviews.pl |    9 ++++-----
 2 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl
index 19324c9..1e95de1 100755
--- a/opac/opac-detail.pl
+++ b/opac/opac-detail.pl
@@ -295,11 +295,10 @@ $template->param(
     ocoins => GetCOinSBiblio($biblionumber),
 );
 
-my $libravatar_available = 0;
-
+my $libravatar_enabled = 0;
 eval 'use Libravatar::URL';
-if (! $@) {
-    $libravatar_available = 1;
+if (!$@ and C4::Context->preference('ShowReviewer')) {
+    $libravatar_enabled = 1;
 }
 
 my $reviews = getreviews( $biblionumber, 1 );
@@ -310,7 +309,7 @@ foreach ( @$reviews ) {
     $_->{title}     = $borrowerData->{'title'};
     $_->{surname}   = $borrowerData->{'surname'};
     $_->{firstname} = $borrowerData->{'firstname'};
-    if ($libravatar_available and $borrowerData->{'email'}) {
+    if ($libravatar_enabled and $borrowerData->{'email'}) {
         $_->{avatarurl} = libravatar_url(email => $borrowerData->{'email'}, https => $ENV{HTTPS});
     }
     $_->{userid}    = $borrowerData->{'userid'};
diff --git a/opac/opac-showreviews.pl b/opac/opac-showreviews.pl
index fbce8a2..7b7fddb 100755
--- a/opac/opac-showreviews.pl
+++ b/opac/opac-showreviews.pl
@@ -65,11 +65,10 @@ if($format eq "rss"){
         );
 }
 
-my $libravatar_available = 0;
-
+my $libravatar_enabled = 0;
 eval 'use Libravatar::URL';
-if (! $@) {
-    $libravatar_available = 1;
+if (!$@ and C4::Context->preference('ShowReviewer')) {
+    $libravatar_enabled = 1;
 }
 
 my $reviews = getallreviews(1,$offset,$results_per_page);
@@ -99,7 +98,7 @@ for my $result (@$reviews){
 	$result->{timestamp} = $bib->{'timestamp'};
 	$result->{firstname} = $borr->{'firstname'};
 	$result->{surname} = $borr->{'surname'};
-        if ($libravatar_available and $borr->{'email'}) {
+        if ($libravatar_enabled and $borr->{'email'}) {
             $result->{avatarurl} = libravatar_url(email => $borr->{'email'}, size => 40, https => $ENV{HTTPS});
         }
 
-- 
1.7.2.3