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)
committerMason James <mtj@kohaaloha.com>
Fri, 2 Oct 2015 08:44:46 +0000 (21:44 +1300)
commit37f0bddb17ed53e7049c295c6e2b24c3392c888c
treede282cfa9ee9e61b5bb318af2c5de025d807781b
parent53e060308d163fed9411c3d4a4b5027aba6832bc
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: Mason James <mtj@kohaaloha.com>
reports/guided_reports.pl