Bug 20915 - Use date --iso-8601 instead of date +%Y-%m-%d to be more readable and crontab friendly
Summary: Use date --iso-8601 instead of date +%Y-%m-%d to be more readable and crontab...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Command-line Utilities (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement
Assignee: Victor Grousset/tuxayo
QA Contact: Testopia
URL:
Keywords:
Depends on: 10419
Blocks:
  Show dependency tree / graph
 
Reported: 2018-06-12 14:01 UTC by Victor Grousset/tuxayo
Modified: 2024-07-25 11:17 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: String patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:

Attachments
Bug 20915: Use date --iso-8601 instead of date +%Y-%m-%d (1.14 KB, patch)
2018-06-12 14:09 UTC, Victor Grousset/tuxayo 		Details | Diff | Splinter Review
Bug 20915: Use date --iso-8601 instead of date +%Y-%m-%d (1.14 KB, patch)
2018-06-12 14:15 UTC, Victor Grousset/tuxayo 		Details | Diff | Splinter Review
Bug 20915: Use date --iso-8601 instead of date +%Y-%m-%d (1.16 KB, patch)
2018-09-14 14:27 UTC, Mark Tompsett 		Details | Diff | Splinter Review
Bug 20915: Use date --iso-8601 instead of date +%Y-%m-%d (1.23 KB, patch)
2018-09-21 21:01 UTC, Jonathan Druart 		Details | Diff | Splinter Review
Show Obsolete (3) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Victor Grousset/tuxayo 2018-06-12 14:01:19 UTC 
Because in crontab you have to write ​date ​'+\%Y-\%m-\%d'
Comment 1 Victor Grousset/tuxayo 2018-06-12 14:09:00 UTC Comment hidden (obsolete)
Comment 2 Victor Grousset/tuxayo 2018-06-12 14:15:58 UTC 
Created attachment 75976 [details] [review]
Bug 20915: Use date --iso-8601 instead of date +%Y-%m-%d
Comment 3 Mark Tompsett 2018-09-14 14:27:34 UTC 
Created attachment 78661 [details] [review]
Bug 20915: Use date --iso-8601 instead of date +%Y-%m-%d

In delete_patrons.pl's POD
To be more readable and crontab friendly

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Comment 4 Jonathan Druart 2018-09-21 21:01:42 UTC 
Created attachment 79256 [details] [review]
Bug 20915: Use date --iso-8601 instead of date +%Y-%m-%d

In delete_patrons.pl's POD
To be more readable and crontab friendly

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 5 Nick Clemens (kidclamp) 2018-09-26 19:34:34 UTC 
Awesome work all!

Pushed to master for 18.11
Comment 6 Martin Renvoize (ashimema) 2018-09-27 12:40:58 UTC 
This will not be backported to 18.05.x series.