Summary: | Saved reports: sort broken on creation date | ||
---|---|---|---|
Product: | Koha | Reporter: | Adrien SAURAT <adrien.saurat> |
Component: | Reports | Assignee: | Adrien SAURAT <adrien.saurat> |
Status: | CLOSED INVALID | QA Contact: | |
Severity: | minor | ||
Priority: | P5 - low | CC: | jonathan.druart |
Version: | Main | ||
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: | ||
Circulation function: | |||
Bug Depends on: | 9108, 9182 | ||
Bug Blocks: | |||
Attachments: |
proposed patch
[SIGNED-OFF] Bug 9349: fix on date sort in Saved Reports |
Description
Adrien SAURAT
2013-01-04 16:25:38 UTC
Created attachment 14423 [details] [review] proposed patch TEST PLAN : Try with different date formats (system preference "dateformat") Before patch, the "dd/mm/yyyy" is not well sorted. Created attachment 14426 [details] [review] [SIGNED-OFF] Bug 9349: fix on date sort in Saved Reports Allows the dd/mm/yyyy format to be correctly sorted in the Saved Reports table. TEST PLAN : Try with different date formats (system preference "dateformat") Before patch, the "dd/mm/yyyy" is not well sorted. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Hi Adrien, I link this bug with 9108. Have a look to it and see the new way to check the dateformat pref. This new way will become a new best practice. So your patch will be: + [% IF ( dateformat == "metric" ) %]dateFormat: 'uk',[% END %] Marked as Failed QA |