[Koha-bugs] [Bug 13813] Remove deprecated module C4::Dates from system
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Wed Apr 15 10:18:02 CEST 2015
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13813
--- Comment #134 from Jonathan Druart <jonathan.druart at biblibre.com> ---
(In reply to Jonathan Druart from comment #93)
> Comment on attachment 36900 [details] [review]
> Bug 13813 - Remove C4::Dates from tools/koha-news.pl
>
> Review of attachment 36900 [details] [review]:
> -----------------------------------------------------------------
>
> ::: tools/koha-news.pl
> @@ +40,5 @@
> > my $id = $cgi->param('id');
> > my $title = $cgi->param('title');
> > my $new = $cgi->param('new');
> > +my $expirationdate = output_pref({ dt => dt_from_string( $cgi->param('expirationdate') ), dateformat => 'iso', dateonly => 1 });
> > +my $timestamp = output_pref({ dt => dt_from_string( $cgi->param('timestamp') ), dateformat => 'iso', dateonly => 1 });
>
> Not sure about this but it does not seem to break anything.
For QAer, have a look at this. I don't see the timestamp param set in the
template.
On the list view, before the patch, I got:
koha-news.pl: Usage: POSIX::strftime(fmt, sec, min, hour, mday, mon, year, wday
= -1, yday = -1, isdst = -1) at /home/koha/src/C4/Dates.pm line 187.
koha-news.pl: ...caught at /home/koha/src/tools/koha-news.pl line 45.
--
You are receiving this mail because:
You are watching all bug changes.
More information about the Koha-bugs
mailing list