Attachment 166641 Details for Bug 35508 – Bug 35508: Move logic to Koha::Patron::Attribute::store()

[patch] Bug 35508: Move logic to Koha::Patron::Attribute::store()

Bug-35508-Move-logic-to-KohaPatronAttributestore.patch (text/plain), 3.29 KB, created by Baptiste Bayche on 2024-05-13 13:55:13 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Baptiste Bayche

Created: 2024-05-13 13:55:13 UTC

Size: 3.29 KB

patch

obsolete

>From 42c014b1b9b8db5e5d89130dbc2423e38d0ae486 Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>Date: Wed, 17 Apr 2024 09:29:29 -0400
>Subject: [PATCH] Bug 35508: Move logic to Koha::Patron::Attribute::store()
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> Koha/Patron/Attribute.pm               |  4 +++
> members/memberentry.pl                 |  1 -
> t/db_dependent/Koha/Patron/Attribute.t | 41 +++++++++++++++++++++++++-
> 3 files changed, 44 insertions(+), 2 deletions(-)
>
>diff --git a/Koha/Patron/Attribute.pm b/Koha/Patron/Attribute.pm
>index 5fc1b9111d..863d55850b 100644
>--- a/Koha/Patron/Attribute.pm
>+++ b/Koha/Patron/Attribute.pm
>@@ -58,6 +58,10 @@ sub store {
> 
>     Koha::Exceptions::Patron::Attribute::InvalidAttributeValue->throw( attribute => $self )
>         unless $self->value_ok();
>+    C4::Context->dbh->do(
>+        "UPDATE borrowers SET updated_on = NOW() WHERE borrowernumber = ?", undef,
>+        $self->borrowernumber
>+    );
> 
>     return $self->SUPER::store();
> }
>diff --git a/members/memberentry.pl b/members/memberentry.pl
>index 72fcad3209..f647234691 100755
>--- a/members/memberentry.pl
>+++ b/members/memberentry.pl
>@@ -571,7 +571,6 @@ if ((!$nok) and $nodouble and ($op eq 'cud-insert' or $op eq 'cud-save')){
>         if (C4::Context->preference('ExtendedPatronAttributes') and $input->param('setting_extended_patron_attributes')) {
>             $patron->extended_attributes->filter_by_branch_limitations->delete;
>             $patron->extended_attributes($extended_patron_attributes);
>-            $patron->set( { updated_on => "" } )->store;
>         }
> 
>         if ( $destination eq 'circ' and not C4::Auth::haspermission( C4::Context->userenv->{id}, { circulate => 'circulate_remaining_permissions' } ) ) {
>diff --git a/t/db_dependent/Koha/Patron/Attribute.t b/t/db_dependent/Koha/Patron/Attribute.t
>index f68692e57b..e9e9f9c7fb 100755
>--- a/t/db_dependent/Koha/Patron/Attribute.t
>+++ b/t/db_dependent/Koha/Patron/Attribute.t
>@@ -33,7 +33,46 @@ my $builder = t::lib::TestBuilder->new;
> 
> subtest 'store() tests' => sub {
> 
>-    plan tests => 5;
>+    plan tests => 6;
>+
>+    subtest 'Update an attribute should update the patron "updated_on" field' => sub {
>+
>+        plan tests => 1;
>+
>+        $schema->storage->txn_begin;
>+
>+        my $patron     = $builder->build_object( { class => 'Koha::Patrons' } );
>+        my $updated_on = $patron->updated_on;
>+
>+        sleep 1;    # Timestamps are in one second increments, so we need to wait one second
>+
>+        my $type = $builder->build_object(
>+            {
>+                class => 'Koha::Patron::Attribute::Types',
>+                value => {
>+                    mandatory     => 0,
>+                    repeatable    => 0,
>+                    unique_id     => 0,
>+                    category_code => undef
>+                }
>+            }
>+        );
>+
>+        my $attr = $patron->add_extended_attribute(
>+            {
>+                code      => $type->code,
>+                attribute => 'TEST'
>+            }
>+        );
>+
>+        $attr->set( { attribute => 'TEST' } )->store();
>+
>+        $patron->discard_changes;
>+
>+        isnt( $updated_on, $patron->updated_on, "Updated on was updated by attribute storage" );
>+
>+        $schema->storage->txn_rollback;
>+    };
> 
>     subtest 'repeatable attributes tests' => sub {
> 
>-- 
>2.34.1

Actions: View | Diff | Splinter Review

Attachments on bug 35508: 159838 | 159895 | 165012 | 165013 | 166640 | 166641 | 167229 | 172946 | 172947 | 172948 | 172949