Bugzilla – Attachment 33917 Details for
Bug 13152
Duplicate phone hold notices when using Talking Tech
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 13152 - Duplicate phone hold notices when using Talking Tech
Bug-13152---Duplicate-phone-hold-notices-when-usin.patch (text/plain), 2.21 KB, created by
Jonathan Druart
on 2014-11-25 16:33:23 UTC
(
hide
)
Description:
Bug 13152 - Duplicate phone hold notices when using Talking Tech
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2014-11-25 16:33:23 UTC
Size:
2.21 KB
patch
obsolete
>From 4c2a014c651ad57a0cd53025ba9e2b155c8ec6e1 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Tue, 28 Oct 2014 08:12:46 -0400 >Subject: [PATCH] Bug 13152 - Duplicate phone hold notices when using Talking > Tech > >If a library is using Talking Tech for phone notices, any waiting hold >phone notice will show up twice! > >This is because Koha generates on at the time the hold is set to >waiting, and then the cronjob TalkingTech_itiva_outbound.pl generates >it's own notice as well. > >The former notice will always have a status of 'pending', as the >TalkingTech_itiva_inbound.pl script will update the notice the outbound >script created. > >The solution is to prevent Koha from creating a phone notice for waiting >holds if TT is enabled, and let the cron script do it. > >Test Plan: >1) Enable Talking Tech from the system preferences >2) Set a hold waiting phone notice in the notices and slips editor >3) Choose a patron, enable hold phone notices for that patron >4) Place a hold for a patron, and check it in so it's marked as waiting >5) Note the phone notice generated for the patron >6) Apply this patch >7) Repeat step 4 >8) Note that this time, a phone hold waiting notice is not generated > >Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> >--- > C4/Reserves.pm | 10 ++++++---- > 1 file changed, 6 insertions(+), 4 deletions(-) > >diff --git a/C4/Reserves.pm b/C4/Reserves.pm >index d3e26de..5058c51 100644 >--- a/C4/Reserves.pm >+++ b/C4/Reserves.pm >@@ -2054,10 +2054,12 @@ sub _koha_notify_reserve { > }; > > while ( my ( $mtt, $letter_code ) = each %{ $messagingprefs->{transports} } ) { >- if ( ($mtt eq 'email' and not $to_address) or ($mtt eq 'sms' and not $borrower->{smsalertnumber}) ) { >- # email or sms is requested but not exist >- next; >- } >+ next if ( >+ ( $mtt eq 'email' and not $to_address ) # No email address >+ or ( $mtt eq 'sms' and not $borrower->{smsalertnumber} ) # No SMS number >+ or ( $mtt eq 'phone' and C4::Context->preference('TalkingTechItivaPhoneNotification') ) # Notice is handled by TalkingTech_itiva_outbound.pl >+ ); >+ > &$send_notification($mtt, $letter_code); > $notification_sent++; > } >-- >2.1.0
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 13152
:
32821
|
32823
|
33917
|
33926