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

--- Comment #29 from Jonathan Druart <[email protected]> 
---
Comment on attachment 41054
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=41054
Bug 14395: Code changes

Review of attachment 41054:
 --> 
(http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=14395&attachment=41054)
-----------------------------------------------------------------

Please provide a better commit description.

::: C4/Circulation.pm
@@ +2772,5 @@
> +        {
> +            $soonestrenewal->truncate( to => 'day' );
> +        }
> +
> +        if ( $soonestrenewal > DateTime->now( time_zone => C4::Context->tz() 
> ) )

Prefer dt_from_string to get a DateTime object (yes this already existed before
your patch, but it's worth to make things consistent).

-- 
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