Attachment 80751 Details for Bug 21015 – Bug 21015: fix performance issue with C4::Members

[patch] Bug 21015: fix performance issue with C4::Members

Bug-21015-fix-performance-issue-with-C4Members.patch (text/plain), 1.47 KB, created by Jonathan Druart on 2018-10-17 18:49:41 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2018-10-17 18:49:41 UTC

Size: 1.47 KB

patch

obsolete

>From e915ab7febc0a37feb979307819b293144d74d01 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Joonas=20Kylm=C3=A4l=C3=A4?= <joonas.kylmala@iki.fi>
>Date: Fri, 29 Jun 2018 11:09:41 +0300
>Subject: [PATCH] Bug 21015: fix performance issue with C4::Members
>
>loading Koha::Schema ("use Koha::Schema;") takes significantly time as
>it loads almost a couple hundred classes. Koha::Database has done that
>already once and we can use it to get the ResultSet "Borrower" as
>well, so let's use that. This also make the code more unified because
>Koha::Database is used throughout the code instead of Koha::Schema.
>
>Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>---
> C4/Members.pm | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
>diff --git a/C4/Members.pm b/C4/Members.pm
>index 0cd424b395..9fcc733695 100644
>--- a/C4/Members.pm
>+++ b/C4/Members.pm
>@@ -46,7 +46,6 @@ use Koha::Holds;
> use Koha::List::Patron;
> use Koha::Patrons;
> use Koha::Patron::Categories;
>-use Koha::Schema;
> 
> our (@ISA,@EXPORT,@EXPORT_OK,$debug);
> 
>@@ -342,7 +341,7 @@ database column.
> =cut
> 
> sub get_cardnumber_length {
>-    my $borrower = Koha::Schema->resultset('Borrower');
>+    my $borrower = Koha::Database->new->schema->resultset('Borrower');
>     my $field_size = $borrower->result_source->column_info('cardnumber')->{size};
>     my ( $min, $max ) = ( 0, $field_size ); # borrowers.cardnumber is a nullable varchar(20)
>     $min = 1 if C4::Context->preference('BorrowerMandatoryField') =~ /cardnumber/;
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 21015: 76556 | 80751 | 80752 | 81865 | 81866