Bug 8072 - reports wizard dies
Summary: reports wizard dies
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Reports (show other bugs)
Version: 3.8
Hardware: All All
: P5 - low critical (vote)
Assignee: Galen Charlton
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-10 17:43 UTC by Nicole C. Engard
Modified: 2019-06-27 09:24 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 8072: Changes in capitalisation had broken guided reports (1.13 KB, patch)
2012-05-11 00:03 UTC, Chris Cormack
Details | Diff | Splinter Review
Bug 8072: Changes in capitalisation had broken guided reports (1.21 KB, patch)
2012-05-11 08:02 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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