| Summary: | Too many DateTime manipulation in tools/additional-contents.pl | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
| Component: | Architecture, internals, and plumbing | Assignee: | Jonathan Druart <jonathan.druart> |
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
| Severity: | minor | ||
| Priority: | P5 - low | CC: | andrew, kyle, m.de.rooy, martin.renvoize, tomascohen |
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | Trivial patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: |
22.05.00,21.11.02
|
|
| Circulation function: | |||
| Bug Depends on: | 24387 | ||
| Bug Blocks: | |||
| Attachments: |
Bug 29717: Remove unecessary DT manipulations from tools/additional-contents.pl
Bug 29717: Remove unecessary DT manipulations from tools/additional-contents.pl Bug 29717: Remove unecessary DT manipulations from tools/additional-contents.pl |
||
|
Description
Jonathan Druart
2021-12-16 15:48:01 UTC
Created attachment 128630 [details] [review] Bug 29717: Remove unecessary DT manipulations from tools/additional-contents.pl DBIC handles DateTime correctly, no need for this output_pref call. Test plan: Create a new content, set the dates, confirm they are set correctly Modify the content, modify the dates, confirm they are stored correctly Created attachment 129044 [details] [review] Bug 29717: Remove unecessary DT manipulations from tools/additional-contents.pl DBIC handles DateTime correctly, no need for this output_pref call. Test plan: Create a new content, set the dates, confirm they are set correctly Modify the content, modify the dates, confirm they are stored correctly Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Created attachment 129149 [details] [review] Bug 29717: Remove unecessary DT manipulations from tools/additional-contents.pl DBIC handles DateTime correctly, no need for this output_pref call. Test plan: Create a new content, set the dates, confirm they are set correctly Modify the content, modify the dates, confirm they are stored correctly Signed-off-by: Lucas Gass <lucas@bywatersolutions.com> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Pushed to master for 22.05, thanks to everybody involved [U+1F984] Pushed to 21.11.x for 21.11.02 Missing dependency, not backported to 21.05 |