https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=41105
--- Comment #7 from Martin Renvoize (ashimema) <[email protected]> --- Created attachment 201365 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=201365&action=edit Bug 41105: (QA follow-up) DRY out gender label rendering The gender-code-to-label mapping (F/M/O -> Female/Male/Other) was duplicated between the results table cell and the edit-form select options in modborrowers.tt. Extract it into a shared patron-sex-label.inc so there is a single place defining the mapping, and drop the now-unused "lib" values from the sex options in tools/modborrowers.pl (labels are supplied by the include, keyed on the option value, rather than by Perl). Test plan: 1. Navigate to Patrons and click Search, to display a list of patrons. 2. Select all results and click "Batch patron modification". 3. Verify the "Gender" column displays Female/Male/Other correctly. 4. Verify the "Gender" edit dropdown still offers Female/Male/Other and saves correctly. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
