Bug 8440

Summary: Dates does not appear in suggestions management
Product: Koha Reporter: Fridolin Somers <fridolin.somers>
Component: AcquisitionsAssignee: Fridolin Somers <fridolin.somers>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: minor    
Priority: P5 - low CC: chris, jonathan.druart, paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8443
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Proposed patch
[SIGNED-OFF] Bug 8440: Dates does not appear in suggestions management
Proposed patch (follow-up)
[SIGNED-OFF] Proposed patch (follow-up)

Description Fridolin Somers 2012-07-13 13:14:18 UTC
In acquisition, suggestions management, dates does not appear in Suggested by/on nor Managed by/on.
Comment 1 Fridolin Somers 2012-07-13 14:42:52 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2012-07-17 15:47:09 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2012-07-20 08:34:39 UTC
QA comments:

Works as exepted.
Remove duplicate code

Marking as Passed QA.

It would be great to use KohaDates instead of format_date in suggestion.pl
Comment 4 Paul Poulain 2012-07-24 15:07:10 UTC
(In reply to comment #3)
> It would be great to use KohaDates instead of format_date in suggestion.pl

Agreed (and agreed it's out of the scope of this patch

A minor inconsistency that is due to this patch but a follow-up is welcomed:

Suggested by/on says :  BERNARDI, Nicole / 24/05/2011 
Managed by/on says:  BLOT, Sylvie , 10/06/2011 

("/" as separator for one "," for the other)
Comment 5 Chris Cormack 2012-07-25 03:31:21 UTC
Pushed to 3.8.x, will be in 3.8.4
Comment 6 Fridolin Somers 2012-07-25 07:44:14 UTC Comment hidden (obsolete)
Comment 7 Fridolin Somers 2012-07-25 07:45:39 UTC
comment #6 ...
I choosed to separate patron and date by a "-" since patron uses already the "," and date the "/" (save ISO format).
Comment 8 Jonathan Druart 2012-07-25 10:05:50 UTC
Comment on attachment 10906 [details] [review]
[SIGNED-OFF] Bug 8440: Dates does not appear in suggestions management

already pushed
Comment 9 Jonathan Druart 2012-07-25 10:06:33 UTC
Created attachment 11122 [details] [review]
[SIGNED-OFF] Proposed patch (follow-up)

Simple patch
Comment 10 Jonathan Druart 2012-08-09 12:03:39 UTC
QA Comments:
Make strings consistent.
Marked as Passed QA
Comment 11 Chris Cormack 2012-08-09 19:53:51 UTC
Pushed follow up to 3.8.x, will be in 3.8.4
Comment 12 Paul Poulain 2012-09-03 15:52:47 UTC
Follow-up pushed to master. Already on stable, setting status accordingly