Bug 15153

Summary: Koha::DateUtils sub dt_from_string has two different functions
Product: Koha Reporter: Joonas Kylmälä <joonas.kylmala>
Component: Architecture, internals, and plumbingAssignee: Galen Charlton <gmcharlt>
Status: NEW --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low    
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: --- Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:

Description Joonas Kylmälä 2015-11-06 16:48:23 UTC
Koha::DateUtils sub dt_from_string has two different functions: get date from string and return DateTime->now(). I think it would be better to seperate to other subroutine or remove fully the DateTime->now() because if someone uses dt_from_string in code to generate the date now it would be really hard for others to understand what is happening because the subroutine's name doesn't give any clues.