Summary: | reports wizard dies | ||
---|---|---|---|
Product: | Koha | Reporter: | Nicole C. Engard <nengard> |
Component: | Reports | Assignee: | 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
Created attachment 9502 [details] [review] Bug 8072: Changes in capitalisation had broken guided reports Created attachment 9507 [details] [review] Bug 8072: Changes in capitalisation had broken guided reports 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. Patch pushed. side comment: this is really dirty code, with hardcoded strings used as identifier... This fix will be in 3.8.1 |