http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=11869
Frédérick Capovilla <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #25712|0 |1 is obsolete| | --- Comment #3 from Frédérick Capovilla <[email protected]> --- 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. _______________________________________________ Koha-bugs mailing list [email protected] http://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/
