Bugzilla – Attachment 92821 Details for
Bug 22602
OverDrive circulation integration is broken when user is referred to Koha from another site
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 23272: (RM followup) Fix failing tests
Bug-23272-RM-followup-Fix-failing-tests.patch (text/plain), 1.60 KB, created by
Bouzid Fergani
on 2019-09-13 20:08:56 UTC
(
hide
)
Description:
Bug 23272: (RM followup) Fix failing tests
Filename:
MIME Type:
Creator:
Bouzid Fergani
Created:
2019-09-13 20:08:56 UTC
Size:
1.60 KB
patch
obsolete
>From d3e6128c09a1c003c114b5050ad35b9684f3bd76 Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Date: Fri, 13 Sep 2019 11:19:28 +0100 >Subject: [PATCH] Bug 23272: (RM followup) Fix failing tests > >This bug introduced a regression whereby if any authorized value has no >branch limitations then the authorized value administration page would >fail to display at all. > >A simple ternary corrects the issue, but I'm now wondering about the >logic of returning 'undef' from 'get_branch_limits'.. perhaps an empty >(or full) resultset might be a better option to prevent us from >requireing ternaries before calling chained methods on such objects. > >Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Signed-off-by: Bouzid Fergani <bouzid.fergani@inlibro.com> > >https://bugs.koha-community.org/show_bug.cgi?id=22602 >--- > admin/authorised_values.pl | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff --git a/admin/authorised_values.pl b/admin/authorised_values.pl >index 6080dbb..3e23426 100755 >--- a/admin/authorised_values.pl >+++ b/admin/authorised_values.pl >@@ -231,7 +231,7 @@ if ( $op eq 'list' ) { > $row_data{lib} = $av->lib; > $row_data{lib_opac} = $av->lib_opac; > $row_data{imageurl} = getitemtypeimagelocation( 'intranet', $av->imageurl ); >- $row_data{branches} = $av->library_limits->as_list; >+ $row_data{branches} = $av->library_limits ? $av->library_limits->as_list : []; > $row_data{id} = $av->id; > push(@loop_data, \%row_data); > } >-- >2.7.4
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 22602
:
87352
|
92501
|
92633
|
92821
|
93181