Created attachment 28786 [details]
data display
Created attachment 28878 [details] [review] Fix for switched phone numbers on patron details page. This patch fixes switched phone numbers on patron details page and also makes phone number labels more uniform. Brendon, Every patch needs a test plan. So after your description type something like : To Test: 1. 2. etc etc So that people can test to be sure the patch works. Sorry about that Nicole. Thanks for the info. To Test: 1. Create/modify a patron entering some unique data into the "Secondary phone:" and "Other phone:" fields (different data for each). 2. Save the patron record. 3. Go to the patrons details page and make sure the data entered into the "Secondary phone:" field is displaying next to the "Secondary phone:" label and that the data entered into the "Other phone:" field is displaying next to the "Other phone:" label. Created attachment 28879 [details] [review] Fix for switched phone numbers on patron details page. I have added the test plan to the patch and updated it to include actually applying the patch. To Test: 1. Apply the patch. 2. Create/modify a patron entering some unique data into the "Secondary phone:" and "Other phone:" fields (different data for each). 3. Save the patron record. 4. Go to the patrons details page and make sure the data entered into the "Secondary phone:" field is displaying next to the "Secondary phone:" label and that the data entered into the "Other phone:" field is displaying next to the "Other phone:" label. Created attachment 28904 [details] [review] [SIGNED-OFF] Bug 12407 - phone label or data switched Fix for switched phone numbers on patron details page. This patch fixes switched phone numbers on patron details page and also makes phone number labels more uniform. To Test: 1. Apply the patch. 2. Create/modify a patron entering some unique data into the "Secondary phone:" and "Other phone:" fields (different data for each). 3. Save the patron record. 4. Go to the patrons details page and make sure the data entered into the "Secondary phone:" field is displaying next to the "Secondary phone:" label and that the data entered into the "Other phone:" field is displaying next to the "Other phone:" label. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> This looks similar to bug 8760 - maybe someone can take a look? Created attachment 30393 [details] [review] Bug 12407 - phone label or data switched Fix for switched phone numbers on patron details page. This patch fixes switched phone numbers on patron details page and also makes phone number labels more uniform. To Test: 1. Apply the patch. 2. Create/modify a patron entering some unique data into the "Secondary phone:" and "Other phone:" fields (different data for each). 3. Save the patron record. 4. Go to the patrons details page and make sure the data entered into the "Secondary phone:" field is displaying next to the "Secondary phone:" label and that the data entered into the "Other phone:" field is displaying next to the "Other phone:" label. Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 30394 [details] [review] Bug 12407: [QA Follow-up] Rename Other Phone to Mobile in staff Touches three member templates. Changes label Other into Mobile. This is consistent with database field name and OPAC. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> QA Comment: Looks good to me. With reference to bug 8760, I added a QA follow-up to rename Other to Mobile in three member templates. Also note that the field names in the database such as phonepro for secondary phone do not make it easier. Will put a note on the other report too. Passed QA *** Bug 8760 has been marked as a duplicate of this bug. *** (In reply to M. de Rooy from comment #9) > Created attachment 30394 [details] [review] [review] > Bug 12407: [QA Follow-up] Rename Other Phone to Mobile in staff > > Touches three member templates. > Changes label Other into Mobile. > This is consistent with database field name and OPAC. > > Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Yay! I sent a patch for this ages ago ( bug 9383 ), but it was tangled up in a few too many other changes, so I'm happy to see this pass QA finally! Created attachment 30549 [details] [review] Bug 12407: (QA followup) last occurence of 'Other phone' renamed to 'Mobile phone' Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com> Patch pushed to master. Thanks Brendon! Great job solving this, but we continue to have issues defining multiple phone numbers. Why do we maintain four separate phone numbers: Primary, secondary, other and SMS number? Most people have a mobile phone - that should be listed first - the SMS number does not need to be a separate field from mobile. Some people have a landline, work phone or alternate number, however Koha doesn't use these alternate numbers, they are only used by Circulation staff, and in our system they are redundant and do not get used. How many people have four phone numbers? Two fields and a duplication of mobile to SMS is sufficient. Maybe a plus sign to add additional numbers if absolutely necessary. By confusing primary, secondary, other and SMS, we get confusion about which field means what. Labeling one number as "Mobile" seems like a logical solution. Oops! Looks like I didn't implement the patch yet. Now I see that it is displaying as "mobile." My apologies for the confusion. |
Created attachment 28785 [details] enter data See attached image. You enter date in the secondary and other phone fields on the patron form and it shows next to the opposite label on the patron record.