Bugzilla – Attachment 75969 Details for
Bug 20832
Opac user page crash when there is an overdue fine and not any rental charge for a patron
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 20832: Fix opac user profile page when there is overdue fine and no rental charge
Bug-20832-Fix-opac-user-profile-page-when-there-is.patch (text/plain), 1.61 KB, created by
Jonathan Druart
on 2018-06-12 14:02:07 UTC
(
hide
)
Description:
Bug 20832: Fix opac user profile page when there is overdue fine and no rental charge
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2018-06-12 14:02:07 UTC
Size:
1.61 KB
patch
obsolete
>From 6bd5e477f550cd8efedc8bdd8f0ad66d032570ef Mon Sep 17 00:00:00 2001 >From: Josef Moravec <josef.moravec@gmail.com> >Date: Tue, 29 May 2018 11:48:14 +0000 >Subject: [PATCH] Bug 20832: Fix opac user profile page when there is overdue > fine and no rental charge > >Test plan: >0) Do not apply the patch >1) Have an overdue fine linked with specific item currently checked out >to your patron >2) Do not have any unpaid rental fees >3) Go to this patron profile in opac - page opac-user.pl >4) The page crashes with "Can't call method "get_column" on an undefined >value at >/usr/share/koha/opac/cgi-bin/opac/opac-user.pl line 217" in log >5) Apply the patch >6) Restart plack >7) The page should working and show the right amounts for fines > >Signed-off-by: Andrew Isherwood <andrew.isherwood@ptfs-europe.com> > >Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >--- > opac/opac-user.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/opac/opac-user.pl b/opac/opac-user.pl >index 60a5df9b76..4e54afcf5b 100755 >--- a/opac/opac-user.pl >+++ b/opac/opac-user.pl >@@ -214,7 +214,7 @@ if ( $pending_checkouts->count ) { # Useless test > as => ['rental_fines'] > } > ); >- $issue->{rentalfines} = $charges->count ? $charges->next->get_column('rental_fines') : 0; >+ $issue->{rentalfines} = $rental_fines->count ? $rental_fines->next->get_column('rental_fines') : 0; > > my $marcrecord = GetMarcBiblio({ biblionumber => $issue->{'biblionumber'} }); > $issue->{'subtitle'} = GetRecordValue('subtitle', $marcrecord, GetFrameworkCode($issue->{'biblionumber'})); >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 20832
:
75624
|
75945
| 75969