http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11869 Frédérick Capovilla <fcapovilla@live.ca> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25712|0 |1 is obsolete| | --- Comment #3 from Frédérick Capovilla <fcapovilla@live.ca> --- Created attachment 25726 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=25726&action=edit New version of the patch.
Why did you change the order of things? todaysdate followed the cardnumber link before. Why did you not just insert the address line? This is an old patch we did on Koha 3.6 and ported to 3.8, 3.12 and master. I guess the order got switched around during one of our merges. Also, C4::Dates::format_date($accountline->{date}); -- Why not just pass the date, and use the KohaDate plug in in the template to make sure it is formatted correctly? Again, since its an old patch, it still used the old method to generate dates.
Here is a new version of the patch, fixing those two problems. Thanks for the advices. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.