Generally, the patron record details on moremember.pl don't show the labels for fields for which there is no data -- for example, if your patron doesn't have an email address, we don't show the Email field at all. However, we're not consistent about this with SMS Number. If the SMSSendDriver syspref has a value, then every patron will show "SMS Number" under the messaging prefs table on moremember.pl, even if they don't have any value there.
Created attachment 159114 [details] [review] Bug 35356: Show SMS labels only when data exists To test: APPLY PATCH 1. Set SMSSendDriver = Email 2. Admin > SMS cellular providers, have at least 1. 3. Go to the members/moremember.pl 4. If the patron has no SMSnumber or SMS provider then niether of the labels should appear. 5. Add both a SMSnumber and SMS provider, make sure the data appears correctly on members/moremember.pl.
Created attachment 159117 [details] [review] Bug 35356: Show SMS labels only when data exists To test: APPLY PATCH 1. Set SMSSendDriver = Email 2. Admin > SMS cellular providers, have at least 1. 3. Go to the members/moremember.pl 4. If the patron has no SMSnumber or SMS provider then niether of the labels should appear. 5. Add both a SMSnumber and SMS provider, make sure the data appears correctly on members/moremember.pl. Signed-off-by: Andrew Fuerste Henry <andrewfh@dubcolib.org>
Created attachment 159214 [details] [review] Bug 35356: Show SMS labels only when data exists To test: APPLY PATCH 1. Set SMSSendDriver = Email 2. Admin > SMS cellular providers, have at least 1. 3. Go to the members/moremember.pl 4. If the patron has no SMSnumber or SMS provider then niether of the labels should appear. 5. Add both a SMSnumber and SMS provider, make sure the data appears correctly on members/moremember.pl. Signed-off-by: Andrew Fuerste Henry <andrewfh@dubcolib.org> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net>
> Generally, the patron record details on moremember.pl don't show the labels for fields for which there is no data -- for example, if your patron doesn't have an email address, we don't show the Email field at all. However, we're not consistent about this with SMS Number Even though, this really looks more like an enhancement. Regardless, I noticed too late ^^ It works, makes sense, QA script happy, code looks good, passing QA :)
Pushed for 24.05! Well done everyone, thank you!
Small enhancement, I choose to backport Pushed to 23.11.x for 23.11.03
Enhancement, no backport for 23.05.x