Bugzilla – Attachment 185684 Details for
Bug 38924
Introduce an organization level loan 'Quota' system for Koha
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 38924: Overload delete method in Koha::Old::Checkout
Bug-38924-Overload-delete-method-in-KohaOldCheckou.patch (text/plain), 1.36 KB, created by
Victor Grousset/tuxayo
on 2025-08-21 22:53:27 UTC
(
hide
)
Description:
Bug 38924: Overload delete method in Koha::Old::Checkout
Filename:
MIME Type:
Creator:
Victor Grousset/tuxayo
Created:
2025-08-21 22:53:27 UTC
Size:
1.36 KB
patch
obsolete
>From fc03996dc1ab4c05e51a372e68f927b6c9e91b0a Mon Sep 17 00:00:00 2001 >From: Jacob O'Mara <jacobomara901@gmail.com> >Date: Thu, 13 Feb 2025 13:45:43 +0000 >Subject: [PATCH] Bug 38924: Overload delete method in Koha::Old::Checkout > >This overloads the delete method for old::checkouts to ensure that the >issue_id for quotas usage is nulled in the event that an old_checkout is >deleted. >Would like to make this the standard for all the tables we have had to >handle issue/old_issues for but is currently just hard-coded for quotas >as many of the other tables with issue_id columns are not nullable. >--- > Koha/Old/Checkout.pm | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > >diff --git a/Koha/Old/Checkout.pm b/Koha/Old/Checkout.pm >index b2187ef650..e24ecaff52 100644 >--- a/Koha/Old/Checkout.pm >+++ b/Koha/Old/Checkout.pm >@@ -165,6 +165,26 @@ sub to_api_mapping { > }; > } > >+=head3 delete >+ >+Overloaded delete method to ensure quota usages are updated. >+ >+=cut >+ >+sub delete { >+ my ($self) = @_; >+ >+ my $schema = Koha::Database->new->schema; >+ $schema->txn_do( >+ sub { >+ # Update any quota usages linked to this checkout >+ Koha::Patron::Quota::Usages->search( { issue_id => $self->issue_id } )->update( { issue_id => undef } ); >+ >+ return $self->SUPER::delete(); >+ } >+ ); >+} >+ > =head2 Internal methods > > =head3 _type >-- >2.50.1
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 38924
:
178582
|
178583
|
178584
|
178585
|
178586
|
178587
|
178588
|
178589
|
178590
|
178591
|
178592
|
178593
|
178594
|
178595
|
178596
|
178597
|
178598
|
178599
|
178600
|
178601
|
178602
|
178603
|
178604
|
178605
|
178606
|
178607
|
178608
|
185631
|
185632
|
185633
|
185634
|
185635
|
185636
|
185637
|
185638
|
185639
|
185640
|
185641
|
185642
|
185643
|
185644
|
185645
|
185646
|
185647
|
185648
|
185649
|
185650
|
185651
|
185652
|
185653
|
185654
|
185655
|
185656
|
185657
|
185675
|
185676
|
185677
|
185678
|
185679
|
185680
|
185681
|
185682
|
185683
| 185684 |
185685
|
185686
|
185687
|
185688
|
185689
|
185690
|
185691
|
185692
|
185693
|
185694
|
185695
|
185696
|
185697
|
185698
|
185699
|
185700
|
185701
|
185702