https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29768 Emmi Takkinen <emmi.takkinen@koha-suomi.fi> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff --- Comment #22 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Joonas Kylmälä from comment #19)
The hyperlink is not shown in the latest patch version, the hide_patron_infos_if_needed check was reintroduced in this version of the patch, it was the one you removed earlier.
So it seems (how on earth it got there?), removed it again in latest patch.
Also, I'm thinking whether introducing a "no_link" variable or equivalent in addition to the "no_html" would be a good idea? Reading your last patch and patron-title.inc in more detail the patron's title (Mr, Dr, etc.) seems to be enclosed in a span, which would be valid inside hyperlink, however a hyperlink wouldn't be valid inside a hyperlink.
Hmm, I guess this could be useful. Feel free to add it if you have some sort idea how to implement it. -- You are receiving this mail because: You are watching all bug changes.