http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13813

--- Comment #134 from Jonathan Druart <[email protected]> ---
(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.
_______________________________________________
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/

Reply via email to