Attachment 69854 Details for Bug 19808 – Bug 19808: (follow-up) Handle deleted reviewers gracefully - opac-detail

[patch] Bug 19808: (follow-up) Handle deleted reviewers gracefully - opac-detail

Bug-19808-follow-up-Handle-deleted-reviewers-grace.patch (text/plain), 1.43 KB, created by Jonathan Druart on 2017-12-18 16:36:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2017-12-18 16:36:44 UTC

Size: 1.43 KB

patch

obsolete

>From ce63e382540b8b313e2a1ca3954dda67e2dc055b Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 18 Dec 2017 13:32:02 -0300
>Subject: [PATCH] Bug 19808: (follow-up) Handle deleted reviewers gracefully -
> opac-detail
>
>Make the tests easier to read
>---
> opac/opac-detail.pl | 14 ++++++++------
> 1 file changed, 8 insertions(+), 6 deletions(-)
>
>diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl
>index 745c49cd62..9ace469152 100755
>--- a/opac/opac-detail.pl
>+++ b/opac/opac-detail.pl
>@@ -844,13 +844,15 @@ if ( C4::Context->preference('reviewson') ) {
>         my $patron = Koha::Patrons->find( $review->{borrowernumber} );
> 
>         # setting some borrower info into this hash
>-        $review->{patron} = $patron;
>-        if ( $libravatar_enabled and $patron and $patron->email ) {
>-            $review->{avatarurl} = libravatar_url( email => $patron->email, https => $ENV{HTTPS} );
>-        }
>+        if ( $patron ) {
>+            $review->{patron} = $patron;
>+            if ( $libravatar_enabled and $patron->email ) {
>+                $review->{avatarurl} = libravatar_url( email => $patron->email, https => $ENV{HTTPS} );
>+            }
> 
>-        if ( $patron and $patron->borrowernumber eq $borrowernumber ) {
>-            $loggedincommenter = 1;
>+            if ( $patron->borrowernumber eq $borrowernumber ) {
>+                $loggedincommenter = 1;
>+            }
>         }
>     }
> }
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 19808: 69824 | 69828 | 69853 | 69854 | 69855 | 69856 | 69926 | 69934 | 69935 | 69936 | 69937 | 69938 | 69989 | 69990 | 69991 | 69992 | 69993