Bug 13012

Summary: suggestion.suggesteddate should be set to NOW if not defined
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: fridolin.somers, katrin.fischer, kyle, veron
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 11391    
Bug Blocks:    
Attachments: Bug 13012 - suggestion.suggesteddate should be set to NOW if not defined
Bug 13012 - suggestion.suggesteddate should be set to NOW if not defined
Bug 13012 - suggestion.suggesteddate should be set to NOW if not defined
Bug 13012: Add more tests

Description Jonathan Druart 2014-09-30 10:47:10 UTC
Bug 11391 remove the default value 0 for the suggesteddate.
Actually it should be NOW.
Comment 1 Jonathan Druart 2014-09-30 10:54:04 UTC
Which is not possible. We should make this field a timestamp instead of a date.
Comment 2 Kyle M Hall 2017-07-19 18:58:17 UTC
Created attachment 65127 [details] [review]
Bug 13012 - suggestion.suggesteddate should be set to NOW if not defined

Test Plan:
1) Apply this patch
2) prove t/db_dependent/Koha/Suggestions.t
Comment 3 Marc Véron 2017-07-20 09:08:05 UTC
Created attachment 65139 [details] [review]
Bug 13012 - suggestion.suggesteddate should be set to NOW if not defined

Test Plan:
1) Apply this patch
2) prove t/db_dependent/Koha/Suggestions.t

Signed-off-by: Marc Véron <veron@veron.ch>
Comment 4 Jonathan Druart 2017-08-10 18:55:41 UTC
Created attachment 65845 [details] [review]
Bug 13012 - suggestion.suggesteddate should be set to NOW if not defined

Test Plan:
1) Apply this patch
2) prove t/db_dependent/Koha/Suggestions.t

Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 5 Jonathan Druart 2017-08-10 18:55:46 UTC
Created attachment 65846 [details] [review]
Bug 13012: Add more tests

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 6 Jonathan Druart 2017-08-10 19:30:19 UTC
Pushed to master for 17.11, thanks to everybody involved!
Comment 7 Fridolin Somers 2017-09-29 09:50:14 UTC
Pushed to 17.05.x, will be in 17.05.05.
Comment 8 Katrin Fischer 2017-10-01 21:37:05 UTC
These patches have been pushed to 16.11.x and will be in 16.11.13.