]> git.koha-community.org Git - koha.git/commit
Bug 13152 - Duplicate phone hold notices when using Talking Tech
authorKyle M Hall <kyle@bywatersolutions.com>
Tue, 28 Oct 2014 12:12:46 +0000 (08:12 -0400)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Tue, 25 Nov 2014 20:33:56 +0000 (17:33 -0300)
commite3fe0bdb316ef2ea901dabc9f97d4dd30dacf3f9
tree2d85f0f5c409929b1c0b5f0270a87137b5796cdb
parentec6a50b2fe502f14738aef82dc30564a335a224b
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>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Amends condition with an additional or statement. Shoudn't affect
anything but phone notices. Change appears logical.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
C4/Reserves.pm