Bugzilla – Attachment 148439 Details for
Bug 33260
Show message queue on staff interface and make notices printable
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 33260: Add a patron getter method to Koha::Notice::Message
Bug-33260-Add-a-patron-getter-method-to-KohaNotice.patch (text/plain), 2.00 KB, created by
Aleisha Amohia
on 2023-03-21 03:27:12 UTC
(
hide
)
Description:
Bug 33260: Add a patron getter method to Koha::Notice::Message
Filename:
MIME Type:
Creator:
Aleisha Amohia
Created:
2023-03-21 03:27:12 UTC
Size:
2.00 KB
patch
obsolete
>From b8afe4ebf587a236f5bdc585910ee95915a51980 Mon Sep 17 00:00:00 2001 >From: Aleisha Amohia <aleishaamohia@hotmail.com> >Date: Tue, 21 Mar 2023 02:47:29 +0000 >Subject: [PATCH] Bug 33260: Add a patron getter method to > Koha::Notice::Message > >--- > Koha/Notice/Message.pm | 16 ++++++++++++++++ > t/db_dependent/Letters.t | 6 ++++-- > 2 files changed, 20 insertions(+), 2 deletions(-) > >diff --git a/Koha/Notice/Message.pm b/Koha/Notice/Message.pm >index fcf7eca2a67..93dcd43ca45 100644 >--- a/Koha/Notice/Message.pm >+++ b/Koha/Notice/Message.pm >@@ -32,6 +32,22 @@ Koha::Notice::Message - Koha notice message Object class, related to the message > > =cut > >+=head3 patron >+ >+ my $patron = $message->patron; >+ >+Returns the Koha::Patron object for the recipient of the queued message >+ >+=cut >+ >+sub patron { >+ my ( $self ) = @_; >+ >+ $self->{_patron} ||= Koha::Patrons->find( $self->borrowernumber ); >+ >+ return $self->{_patron}; >+} >+ > =head3 type > > =cut >diff --git a/t/db_dependent/Letters.t b/t/db_dependent/Letters.t >index 33a518fc096..60f38b123dc 100755 >--- a/t/db_dependent/Letters.t >+++ b/t/db_dependent/Letters.t >@@ -18,7 +18,7 @@ > # along with Koha; if not, see <http://www.gnu.org/licenses>. > > use Modern::Perl; >-use Test::More tests => 87; >+use Test::More tests => 88; > use Test::MockModule; > use Test::Warn; > use Test::Exception; >@@ -150,7 +150,9 @@ is( $messages->[0]->{failure_code}, '', 'Failure code for successful message cor > > # Setting time_queued to something else than now > my $yesterday = dt_from_string->subtract( days => 1 ); >-Koha::Notice::Messages->find($messages->[0]->{message_id})->time_queued($yesterday)->store; >+my $message_obj = Koha::Notice::Messages->find($messages->[0]->{message_id})->time_queued($yesterday)->store; >+my $patron_obj = Koha::Patrons->find( $borrowernumber ); >+is_deeply( $message_obj->patron, $patron_obj, "Koha::Notice::Message->patron sub correctly returns patron object"); > > # SendQueuedMessages > my $messages_processed = C4::Letters::SendQueuedMessages( { type => 'email' }); >-- >2.20.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 33260
:
148437
|
148438
|
148439
|
148440
|
148943
|
149009
|
149413
|
149581
|
149593
|
149594
|
149595
|
149596
|
149597
|
149598
|
150346
|
150347
|
154927
|
154928
|
157383
|
157384
|
157385
|
157386
|
158852
|
158853
|
158854
|
158855
|
162570
|
162571
|
162572
|
162573
|
162574
|
162624
|
162625
|
162677
|
162678
|
162679
|
162680
|
162681
|
164783
|
164784
|
164785
|
164786
|
164787
|
165238
|
165239
|
165717
|
165939
|
165946
|
165947
|
165948
|
165949
|
165950
|
165951
|
166020
|
166178
|
166335
|
166336
|
166337
|
166338
|
166339
|
166340
|
166341
|
166342
|
166343
|
166344
|
166345
|
166362
|
166363
|
166364
|
166365
|
166366
|
166367
|
166368
|
166369
|
166370
|
166371
|
166372
|
166373