Bug 8072

Summary: reports wizard dies
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: ReportsAssignee: Galen Charlton <gmcharlt>
Status: CLOSED FIXED QA Contact:
Severity: critical    
Priority: P5 - low CC: chris, jonathan.druart, koha.sekjal, paul.poulain
Version: 3.8   
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: Bug 8072: Changes in capitalisation had broken guided reports
Bug 8072: Changes in capitalisation had broken guided reports

Description Nicole C. Engard 2012-05-10 17:43:11 UTC
At step 6 the guided report wizard just dies with no warning ... see: http://screencast.com/t/JQRQDcZZud
Comment 1 Chris Cormack 2012-05-11 00:03:31 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2012-05-11 08:02:55 UTC
Created attachment 9507 [details] [review]
Bug 8072: Changes in capitalisation had broken guided reports
Comment 3 Ian Walls 2012-05-11 13:23:51 UTC
Change in capitalization in strings in conditional checks... now matches what we're passing.  Marking as Passed QA.

In a side note, I think this mechanism of matching on strings is pretty fragile and suboptimal.  A patch to shore things up with something more robust is welcomed and encouraged.
Comment 4 Paul Poulain 2012-05-14 11:57:30 UTC
Patch pushed.

side comment: this is really dirty code, with hardcoded strings used as identifier...
Comment 5 Chris Cormack 2012-05-14 20:32:57 UTC
This fix will be in 3.8.1