https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15400
--- Comment #59 from Jonathan Druart <[email protected]> --- (In reply to Jonathan Druart from comment #57) > + [%- IF ( patron.get_age ) %] ([% patron.get_age | html %] years) > + [%- ELSIF ( patron.age ) %] ([% patron.age | html %] years)[% END > -%] > > That smells. > You should either test for the type of patron, or pass another variable > patron_object vs patron > or > patron vs borrower (that reads wrong but it is what is usually done in > controllers) Sorry, I was reading the patch and not the final change. I added a follow-up. However I still think it's lot of info that could be considered not needed by some. Especially in the circ autocomplete. Letting someone else adding the QA stamp. -- 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/
