Bug 6608

Summary: Date-pickers broken on guided reports pages
Product: Koha Reporter: Owen Leonard <oleonard>
Component: ReportsAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: normal    
Priority: PATCH-Sent (DO NOT USE) CC: chris, koha.sekjal, nengard
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:
Circulation function:
Attachments: Proposed fix

Description Owen Leonard 2011-07-19 19:37:41 UTC
There is a Template:Toolkit variable scoping issue with the date-picker JavaScript configuration in the guided reports and dictionary templates.
Comment 1 Owen Leonard 2011-07-19 19:47:02 UTC
Created attachment 4671 [details] [review]
Proposed fix

Template variable scope errors.

The change to dictionary.tt can be tested by creating a dictionary definition based on a date-type table column (date of birth, registration date, etc). The date-picker widgets for "start of date range" and "end of date range" should work correctly.

The change to guided_reports_start.tt cannot be tested as far as I know because of bug 6607.
Comment 2 Ian Walls 2011-07-19 20:05:27 UTC
Template-only change, easy to read, marking as Passed QA
Comment 3 Chris Cormack 2011-07-19 21:39:36 UTC
Pushed, please test