| 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 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | 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 |