Bug 14872

Summary: Reminder to update SMSnumber when primary phone is updated
Product: Koha Reporter: Lari Taskula <lari.taskula>
Component: PatronsAssignee: Lari Taskula <lari.taskula>
Status: Failed QA --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: dcook, gmcharlt, jonathan.druart, katrin.fischer, kyle.m.hall, martin.renvoize, oleonard
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18217
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11879
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31107
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31424
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 14872: Reminder to update SMSnumber when primary phone is updated
Bug 14872: Reminder to update SMSnumber when primary phone is updated
Bug 14872: Reminder to update SMSnumber when primary phone is updated

Description Lari Taskula 2015-09-22 14:11:31 UTC
It would be useful to remind patron to update his/her smsalertnumber after modifying primary phone number so that sms number also stays up to date.
Comment 1 Lari Taskula 2015-09-24 11:22:24 UTC Comment hidden (obsolete)
Comment 2 Lari Taskula 2015-09-24 11:24:12 UTC Comment hidden (obsolete)
Comment 3 Aleisha Amohia 2016-02-03 22:01:51 UTC
Created attachment 47624 [details] [review]
Bug 14872: Reminder to update SMSnumber when primary phone is updated

Since SMS number (smsalertnumber) and primary phone are separated, it would
be useful to remind patron to update his/her SMS number when making changes
to primary phone. This can easily be forgotten, especially in OPAC where the
two fields are in different page (opac-memberentry.pl vs opac-messaging.pl).

This patch adds a reminder in the "hint"-div that activates whenever phone
field gets input (and phone-smsalertnumber do not match).

To test:
1. Enable SMSSendDriver and EnhancedMessagingPreferences system preferences
2. Navigate to OPAC -> your personal details
3. Input something to primary phone. You should not be notified at all
4. Navigate to Intra -> Patron -> Edit
5. Repeat step 3
6. Apply patch
7. Refresh Opac -> your personal details and Intra -> Patron -> Edit
8. Input something to primary phone and observe you will be notified to
   also update SMS number.

Signed-off-by: Aleisha <aleishaamohia@hotmail.com>

Fixed a small merge conflict in opac-memberentry.tt
Comment 4 Jonathan Druart 2016-02-09 11:21:12 UTC
I don't get the message at the OPAC.

The SMSSendDriver value should be checked using the TT plugin (Koha.Preference).
I don't know if we need a pref to control this behavior, maybe other libraries don't use the primary phone as the mobile phone.
Comment 5 Katrin Fischer 2016-02-09 11:52:49 UTC
(In reply to Jonathan Druart from comment #4)
> I don't get the message at the OPAC.
> 
> The SMSSendDriver value should be checked using the TT plugin
> (Koha.Preference).
> I don't know if we need a pref to control this behavior, maybe other
> libraries don't use the primary phone as the mobile phone.

I am a bit worried about that too. We offer, I think, 3 phone number fields, and some might have the habit/custom to put the landline in the first field and the mobile/work number in another, especially since one of the fields were once labelled differently.
Comment 6 Owen Leonard 2016-07-18 15:32:51 UTC
I wonder if a better long-term solution wouldn't be to have the SMS number field be an option to select one of the existing phone number fields: A dropdown menu where you choose to link SMS messages to primary phone, secondary phone, or other phone. It seems odd to me that we even have a separate field for SMS number.
Comment 7 David Cook 2022-07-06 05:25:29 UTC
(In reply to Owen Leonard from comment #6)
> I wonder if a better long-term solution wouldn't be to have the SMS number
> field be an option to select one of the existing phone number fields: A
> dropdown menu where you choose to link SMS messages to primary phone,
> secondary phone, or other phone. It seems odd to me that we even have a
> separate field for SMS number.

Right?
Comment 8 Martin Renvoize 2022-12-24 06:48:03 UTC
Hmm, is there a valid use case for people to have a distinct SMS alert number compared to their other numbers at all?

I feel like we should/could drop the field from borrowers entirely and impliment a preference to say which of our phone fields is used globally as the SMS alert number.. but that does reduce options for end users if they do want one mobile for calls and another number for texts?

Does the above case actually exist in the real world .  I don't know