Bugzilla – Attachment 32821 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.12 KB, created by
Kyle M Hall (khall)
on 2014-10-28 13:00:40 UTC
(
hide
)
Description:
Bug 13152 - Duplicate phone hold notices when using Talking Tech
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2014-10-28 13:00:40 UTC
Size:
2.12 KB
patch
obsolete
>From 7193d8f7cb1f0e7bf87032ad6bc0deb415822dac 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 >--- > C4/Reserves.pm | 8 ++++++-- > 1 files changed, 6 insertions(+), 2 deletions(-) > >diff --git a/C4/Reserves.pm b/C4/Reserves.pm >index 9116080..d2f7d5c 100644 >--- a/C4/Reserves.pm >+++ b/C4/Reserves.pm >@@ -1964,8 +1964,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 >+ 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 >+ ) >+ { > next; > } > &$send_notification($mtt, $letter_code); >-- >1.7.2.5
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