]> git.koha-community.org Git - koha.git/commit
Bug 14382: Non-ISO Date parameters generate empty reports.
authorMark Tompsett <mtompset@hotmail.com>
Wed, 24 Jun 2015 16:13:46 +0000 (12:13 -0400)
committerLiz Rea <wizzyrea@gmail.com>
Thu, 16 Jul 2015 10:19:12 +0000 (10:19 +0000)
commita3099a7e5aeb635a879aec4a10fe7ebe3834a1dd
treeefba3726e559eede383a4d7a1807024154a2e2a3
parentf69c9023d7c24a06ee06db24b9825015ec3e5229
Bug 14382: Non-ISO Date parameters generate empty reports.

The issue is that SQL expects ISO dates, but the user may wish to view dates according to the dateformat system preference.

By detecting a date preference, the non-ISO dates can be converted to ISO dates before being stuffed back into the SQL query to be executed.

TEST PLAN
---------
1) Add a report with date parameters.
   -- I used 'Holds placed in date range' from
      http://wiki.koha-community.org/wiki/SQL_Reports_Library
2) Set your dateformat to YYYY-MM-DD
3) Run the report
   -- Note the SQL reads
      "... BETWEEN '{date formatted in YYYY-MM-DD}'..."
   -- If there is supposed to be data, there is some.
4) Set your dateformat to MM/DD/YYYY
5) Run the report
   -- Note the SQL reads
      "... BETWEEN '{date formatted in MM/DD/YYYY}'..."
   -- If there is supposed to be data, there is none.
6) Apply patch
7) Repeat steps 2-5
   -- The SQL will always read YYYY-MM-DD (ISO) format.
   -- The report will have data, if there is some.
8) koha qa test tools.

Signed-off-by: Amit Gupta <amit.gupta@informaticsglobal.com>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Adding a QA follow-up.
Signed-off-by: Tomas Cohen Arazi <tomascohen@unc.edu.ar>
(cherry picked from commit c3fea53039a6c53c766b0403eedd57f644c6f772)
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Conflicts:
reports/guided_reports.pl

Signed-off-by: Liz Rea <wizzyrea@gmail.com>
reports/guided_reports.pl