Attachment #104049 for bug #25333

View | Details | Raw Unified | Return to bug 25333
Collapse All | Expand All

Lines 1805-1811 sub _koha_notify_reserve { Link Here

(-)a/C4/Reserves.pm (-1 / +1 lines)
1805	next if (	1805	next if (
1806	( $mtt eq 'email' and not $to_address ) # No email address	1806	( $mtt eq 'email' and not $to_address ) # No email address
1807	or ( $mtt eq 'sms' and not $patron->smsalertnumber ) # No SMS number	1807	or ( $mtt eq 'sms' and not $patron->smsalertnumber ) # No SMS number
1808	or ( $mtt eq 'phone' and C4::Context->preference('TalkingTechItivaPhoneNotification') ) # Notice is handled by TalkingTech_itiva_outbound.pl	1808	or ( $mtt eq 'talkingtech' and C4::Context->preference('TalkingTechItivaPhoneNotification') ) # Notice is handled by TalkingTech_itiva_outbound.pl
1809	);	1809	);
1810		1810
1811	&$send_notification($mtt, $letter_code);	1811	&$send_notification($mtt, $letter_code);

Line 0 Link Here

(-)a/installer/data/mysql/atomicupdate/bug_25333.perl (+9 lines)
	1	$DBversion = 'XXX'; # will be replaced by the RM
	2	if( CheckVersion( $DBversion ) ) {
	3	$dbh->do(q{
	4	UPDATE message_transport_types SET message_transport_type = "talkingtech" WHERE message_transport_type = "phone"
	5	});
	6
	7	# Always end with this (adjust the bug info)
	8	NewVersion( $DBversion, 25333, q{Change message transport type for Talking Tech from "phone" to "talkingtech"});
	9	}

Lines 4-7 values Link Here

(-)a/installer/data/mysql/mandatory/message_transport_types.sql (-1 / +1 lines)
4	('email'),	4	('email'),
5	('print'),	5	('print'),
6	('sms'),	6	('sms'),
7	('phone');	7	('talkingtech');




insert into `message_transports`
(`message_attribute_id`, `message_transport_type`, `is_digest`, `letter_module`, `letter_code`)
values
(1, 'email',       0, 'circulation', 'DUE'),
(1, 'email',       1, 'circulation', 'DUEDGST'),
(1, 'sms',         0, 'circulation', 'DUE'),
(1, 'sms',         1, 'circulation', 'DUEDGST'),
(2, 'email',       0, 'circulation', 'PREDUE'),
(2, 'email',       1, 'circulation', 'PREDUEDGST'),
(2, 'sms',         0, 'circulation', 'PREDUE'),
(2, 'sms',         1, 'circulation', 'PREDUEDGST'),
(2, 'talkingtech', 0, 'circulation', 'PREDUE'),
(2, 'talkingtech', 1, 'circulation', 'PREDUEDGST'),
(4, 'email',       0, 'reserves',    'HOLD'),
(4, 'sms',         0, 'reserves',    'HOLD'),
(4, 'talkingtech', 0, 'reserves',    'HOLD'),
(5, 'email',       0, 'circulation', 'CHECKIN'),
(5, 'sms',         0, 'circulation', 'CHECKIN'),
(6, 'email',       0, 'circulation', 'CHECKOUT'),
(6, 'sms',         0, 'circulation', 'CHECKOUT');




      [% END %]

      [% IF Koha.Preference('TalkingTechItivaPhoneNotification') %]
        [% IF ( messaging_preference.transport_talkingtech ) %]
          <td>
          [% IF ( messaging_form_inactive ) %]
              [% IF ( messaging_preference.transports_talkingtech ) %]
                 <input type="checkbox"
                 id="talkingtech[% messaging_preference.message_attribute_id | html %]"
                 name="[% messaging_preference.message_attribute_id | html %]"
                 value="talkingtech" checked="checked" disabled="disabled" />
              [% ELSE %]
                 <input type="checkbox"
                 id="talkingtech[% messaging_preference.message_attribute_id | html %]"
                 name="[% messaging_preference.message_attribute_id | html %]"
                 value="talkingtech" disabled="disabled" />
              [% END %]
          [% ELSE %]
              [% IF ( messaging_preference.transports_talkingtech ) %]
                 <input type="checkbox"
                 id="talkingtech[% messaging_preference.message_attribute_id | html %]"
                 name="[% messaging_preference.message_attribute_id | html %]"
                 value="talkingtech" checked="checked" class="active_notify" data-attr-id="[% messaging_preference.message_attribute_id | html %]" />
              [% ELSE %]
                 <input type="checkbox"
                 id="talkingtech[% messaging_preference.message_attribute_id | html %]"
                 name="[% messaging_preference.message_attribute_id | html %]"
                 value="talkingtech" class="active_notify" data-attr-id="[% messaging_preference.message_attribute_id | html %]" />
              [% END %]
          [% END %]
          </td>

Lines 401-407 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/letter.tt (-1 / +1 lines)
401	SMS	401	SMS
402	[% CASE 'feed' %]	402	[% CASE 'feed' %]
403	Feed	403	Feed
404	[% CASE 'phone' %]	404	[% CASE 'talkingtech' %]
405	Phone	405	Phone
406	[% CASE %]	406	[% CASE %]
407	[% letter.message_transport_type \| html %]	407	[% letter.message_transport_type \| html %]




                biblio      => $issues->{'biblionumber'},
                biblioitems => $issues->{'biblionumber'},
            },
            message_transport_type => 'talkingtech',
        );

        die "No letter found for type $type!... dying\n" unless $letter;

            $message_id = C4::Letters::EnqueueLetter(
                {   letter                 => $letter,
                    borrowernumber         => $issues->{'borrowernumber'},
                    message_transport_type => 'talkingtech',
                }
            );
        }

                JOIN overduerules USING (categorycode)
                JOIN overduerules_transport_types USING ( overduerules_id )
                WHERE ( overduerules.branchcode = borrowers.branchcode or overduerules.branchcode = '')
                AND overduerules_transport_types.message_transport_type = 'talkingtech'
                AND ( (TO_DAYS(NOW())-TO_DAYS(date_due) ) = delay1
                  OR  (TO_DAYS(NOW())-TO_DAYS(date_due) ) = delay2
                  OR  (TO_DAYS(NOW())-TO_DAYS(date_due) ) = delay3 )

                JOIN borrower_message_transport_preferences USING (borrower_message_preference_id)
                JOIN message_attributes USING (message_attribute_id)
                WHERE ( TO_DAYS( date_due ) - TO_DAYS( NOW() ) ) = days_in_advance
                AND message_transport_type = 'talkingtech'
                AND message_name = 'Advance_Notice'
                $patron_branchcode_filter
                ";

                JOIN borrower_message_transport_preferences USING (borrower_message_preference_id)
                JOIN message_attributes USING (message_attribute_id)
                WHERE ( reserves.found = 'W' )
                AND message_transport_type = 'talkingtech'
                AND message_name = 'Hold_Filled'
                $patron_branchcode_filter
                ";
- 

Return to bug 25333

Lines 74-104 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/includes/messaging-preference-form.inc (-11 / +11 lines)
74	[% END %]	74	[% END %]
75		75
76	[% IF Koha.Preference('TalkingTechItivaPhoneNotification') %]	76	[% IF Koha.Preference('TalkingTechItivaPhoneNotification') %]
77	[% IF ( messaging_preference.transport_phone ) %]	77	[% IF ( messaging_preference.transport_talkingtech ) %]
78	<td>	78	<td>
79	[% IF ( messaging_form_inactive ) %]	79	[% IF ( messaging_form_inactive ) %]
80	[% IF ( messaging_preference.transports_phone ) %]	80	[% IF ( messaging_preference.transports_talkingtech ) %]
81	<input type="checkbox"	81	<input type="checkbox"
82	id="phone[% messaging_preference.message_attribute_id \| html %]"	82	id="talkingtech[% messaging_preference.message_attribute_id \| html %]"
83	name="[% messaging_preference.message_attribute_id \| html %]"	83	name="[% messaging_preference.message_attribute_id \| html %]"
84	value="phone" checked="checked" disabled="disabled" />	84	value="talkingtech" checked="checked" disabled="disabled" />
85	[% ELSE %]	85	[% ELSE %]
86	<input type="checkbox"	86	<input type="checkbox"
87	id="phone[% messaging_preference.message_attribute_id \| html %]"	87	id="talkingtech[% messaging_preference.message_attribute_id \| html %]"
88	name="[% messaging_preference.message_attribute_id \| html %]"	88	name="[% messaging_preference.message_attribute_id \| html %]"
89	value="phone" disabled="disabled" />	89	value="talkingtech" disabled="disabled" />
90	[% END %]	90	[% END %]
91	[% ELSE %]	91	[% ELSE %]
92	[% IF ( messaging_preference.transports_phone ) %]	92	[% IF ( messaging_preference.transports_talkingtech ) %]
93	<input type="checkbox"	93	<input type="checkbox"
94	id="phone[% messaging_preference.message_attribute_id \| html %]"	94	id="talkingtech[% messaging_preference.message_attribute_id \| html %]"
95	name="[% messaging_preference.message_attribute_id \| html %]"	95	name="[% messaging_preference.message_attribute_id \| html %]"
96	value="phone" checked="checked" class="active_notify" data-attr-id="[% messaging_preference.message_attribute_id \| html %]" />	96	value="talkingtech" checked="checked" class="active_notify" data-attr-id="[% messaging_preference.message_attribute_id \| html %]" />
97	[% ELSE %]	97	[% ELSE %]
98	<input type="checkbox"	98	<input type="checkbox"
99	id="phone[% messaging_preference.message_attribute_id \| html %]"	99	id="talkingtech[% messaging_preference.message_attribute_id \| html %]"
100	name="[% messaging_preference.message_attribute_id \| html %]"	100	name="[% messaging_preference.message_attribute_id \| html %]"
101	value="phone" class="active_notify" data-attr-id="[% messaging_preference.message_attribute_id \| html %]" />	101	value="talkingtech" class="active_notify" data-attr-id="[% messaging_preference.message_attribute_id \| html %]" />
102	[% END %]	102	[% END %]
103	[% END %]	103	[% END %]
104	</td>	104	</td>

Lines 1-20 Link Here

(-)a/installer/data/mysql/mandatory/sample_notices_message_transports.sql (-17 / +17 lines)
1	insert into `message_transports`	1	insert into `message_transports`
2	(`message_attribute_id`, `message_transport_type`, `is_digest`, `letter_module`, `letter_code`)	2	(`message_attribute_id`, `message_transport_type`, `is_digest`, `letter_module`, `letter_code`)
3	values	3	values
4	(1, 'email', 0, 'circulation', 'DUE'),	4	(1, 'email', 0, 'circulation', 'DUE'),
5	(1, 'email', 1, 'circulation', 'DUEDGST'),	5	(1, 'email', 1, 'circulation', 'DUEDGST'),
6	(1, 'sms', 0, 'circulation', 'DUE'),	6	(1, 'sms', 0, 'circulation', 'DUE'),
7	(1, 'sms', 1, 'circulation', 'DUEDGST'),	7	(1, 'sms', 1, 'circulation', 'DUEDGST'),
8	(2, 'email', 0, 'circulation', 'PREDUE'),	8	(2, 'email', 0, 'circulation', 'PREDUE'),
9	(2, 'email', 1, 'circulation', 'PREDUEDGST'),	9	(2, 'email', 1, 'circulation', 'PREDUEDGST'),
10	(2, 'sms', 0, 'circulation', 'PREDUE'),	10	(2, 'sms', 0, 'circulation', 'PREDUE'),
11	(2, 'sms', 1, 'circulation', 'PREDUEDGST'),	11	(2, 'sms', 1, 'circulation', 'PREDUEDGST'),
12	(2, 'phone', 0, 'circulation', 'PREDUE'),	12	(2, 'talkingtech', 0, 'circulation', 'PREDUE'),
13	(2, 'phone', 1, 'circulation', 'PREDUEDGST'),	13	(2, 'talkingtech', 1, 'circulation', 'PREDUEDGST'),
14	(4, 'email', 0, 'reserves', 'HOLD'),	14	(4, 'email', 0, 'reserves', 'HOLD'),
15	(4, 'sms', 0, 'reserves', 'HOLD'),	15	(4, 'sms', 0, 'reserves', 'HOLD'),
16	(4, 'phone', 0, 'reserves', 'HOLD'),	16	(4, 'talkingtech', 0, 'reserves', 'HOLD'),
17	(5, 'email', 0, 'circulation', 'CHECKIN'),	17	(5, 'email', 0, 'circulation', 'CHECKIN'),
18	(5, 'sms', 0, 'circulation', 'CHECKIN'),	18	(5, 'sms', 0, 'circulation', 'CHECKIN'),
19	(6, 'email', 0, 'circulation', 'CHECKOUT'),	19	(6, 'email', 0, 'circulation', 'CHECKOUT'),
20	(6, 'sms', 0, 'circulation', 'CHECKOUT');	20	(6, 'sms', 0, 'circulation', 'CHECKOUT');

Lines 145-151 foreach my $type (@types) { Link Here

(-)a/misc/cronjobs/thirdparty/TalkingTech_itiva_outbound.pl (-6 / +5 lines)
145	biblio => $issues->{'biblionumber'},	145	biblio => $issues->{'biblionumber'},
146	biblioitems => $issues->{'biblionumber'},	146	biblioitems => $issues->{'biblionumber'},
147	},	147	},
148	message_transport_type => 'phone',	148	message_transport_type => 'talkingtech',
149	);	149	);
150		150
151	die "No letter found for type $type!... dying\n" unless $letter;	151	die "No letter found for type $type!... dying\n" unless $letter;
Lines 155-161 foreach my $type (@types) { Link Here
155	$message_id = C4::Letters::EnqueueLetter(	155	$message_id = C4::Letters::EnqueueLetter(
156	{ letter => $letter,	156	{ letter => $letter,
157	borrowernumber => $issues->{'borrowernumber'},	157	borrowernumber => $issues->{'borrowernumber'},
158	message_transport_type => 'phone',	158	message_transport_type => 'talkingtech',
159	}	159	}
160	);	160	);
161	}	161	}
Lines 250-256 sub GetOverdueIssues { Link Here
250	JOIN overduerules USING (categorycode)	250	JOIN overduerules USING (categorycode)
251	JOIN overduerules_transport_types USING ( overduerules_id )	251	JOIN overduerules_transport_types USING ( overduerules_id )
252	WHERE ( overduerules.branchcode = borrowers.branchcode or overduerules.branchcode = '')	252	WHERE ( overduerules.branchcode = borrowers.branchcode or overduerules.branchcode = '')
253	AND overduerules_transport_types.message_transport_type = 'phone'	253	AND overduerules_transport_types.message_transport_type = 'talkingtech'
254	AND ( (TO_DAYS(NOW())-TO_DAYS(date_due) ) = delay1	254	AND ( (TO_DAYS(NOW())-TO_DAYS(date_due) ) = delay1
255	OR (TO_DAYS(NOW())-TO_DAYS(date_due) ) = delay2	255	OR (TO_DAYS(NOW())-TO_DAYS(date_due) ) = delay2
256	OR (TO_DAYS(NOW())-TO_DAYS(date_due) ) = delay3 )	256	OR (TO_DAYS(NOW())-TO_DAYS(date_due) ) = delay3 )
Lines 292-298 sub GetPredueIssues { Link Here
292	JOIN borrower_message_transport_preferences USING (borrower_message_preference_id)	292	JOIN borrower_message_transport_preferences USING (borrower_message_preference_id)
293	JOIN message_attributes USING (message_attribute_id)	293	JOIN message_attributes USING (message_attribute_id)
294	WHERE ( TO_DAYS( date_due ) - TO_DAYS( NOW() ) ) = days_in_advance	294	WHERE ( TO_DAYS( date_due ) - TO_DAYS( NOW() ) ) = days_in_advance
295	AND message_transport_type = 'phone'	295	AND message_transport_type = 'talkingtech'
296	AND message_name = 'Advance_Notice'	296	AND message_name = 'Advance_Notice'
297	$patron_branchcode_filter	297	$patron_branchcode_filter
298	";	298	";
Lines 323-329 sub GetWaitingHolds { Link Here
323	JOIN borrower_message_transport_preferences USING (borrower_message_preference_id)	323	JOIN borrower_message_transport_preferences USING (borrower_message_preference_id)
324	JOIN message_attributes USING (message_attribute_id)	324	JOIN message_attributes USING (message_attribute_id)
325	WHERE ( reserves.found = 'W' )	325	WHERE ( reserves.found = 'W' )
326	AND message_transport_type = 'phone'	326	AND message_transport_type = 'talkingtech'
327	AND message_name = 'Hold_Filled'	327	AND message_name = 'Hold_Filled'
328	$patron_branchcode_filter	328	$patron_branchcode_filter
329	";	329	";
330	-