Bug 17829: (follow-up) Move GetMember to Koha::Patron

Prevent crash at the opac when no user is logged in Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
pianohacker · Jul 11, 2017 · c91eb80 · c91eb80
1 parent fd28b02
commit c91eb80
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/opac/opac-ISBDdetail.pl b/opac/opac-ISBDdetail.pl
@@ -174,7 +174,7 @@ =head1 FUNCTIONS
         && !$itemtypes->{$itm->{'itype'}}->{notforloan}
         && $itm->{'itemnumber'};
 
-    $allow_onshelf_holds = C4::Reserves::OnShelfHoldsAllowed($itm, $patron->unblessed)
+    $allow_onshelf_holds = C4::Reserves::OnShelfHoldsAllowed( $itm, ( $patron ? $patron->unblessed : {} ) )
       unless $allow_onshelf_holds;
 }
 

diff --git a/opac/opac-MARCdetail.pl b/opac/opac-MARCdetail.pl
@@ -127,9 +127,9 @@ =head1 DESCRIPTION
 }
 
 my $allow_onshelf_holds;
-my $patron = Koha::Patrons->find( $loggedinuser )->unblessed;
+my $patron = Koha::Patrons->find( $loggedinuser );
 for my $itm (@all_items) {
-    $allow_onshelf_holds = C4::Reserves::OnShelfHoldsAllowed($itm, $patron);
+    $allow_onshelf_holds = C4::Reserves::OnShelfHoldsAllowed( $itm, ( $patron ? $patron->unblessed : {} ) );
     last if $allow_onshelf_holds;
 }
 

diff --git a/opac/opac-detail.pl b/opac/opac-detail.pl
@@ -668,7 +668,7 @@ BEGIN
         && !$itemtypes->{$itm->{'itype'}}->{notforloan}
         && $itm->{'itemnumber'};
 
-    $allow_onshelf_holds = C4::Reserves::OnShelfHoldsAllowed( $itm, $patron->unblessed )
+    $allow_onshelf_holds = C4::Reserves::OnShelfHoldsAllowed( $itm, ( $patron ? $patron->unblessed : {} ) )
       unless $allow_onshelf_holds;
 
     # get collection code description, too