| Summary: | Allow HTML in notes for saved SQL reports | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Lucas Gass (lukeg) <lucas> |
| Component: | Reports | Assignee: | Lucas Gass (lukeg) <lucas> |
| Status: | In Discussion --- | QA Contact: | Testopia <testopia> |
| Severity: | enhancement | ||
| Priority: | P5 - low | CC: | barbara.johnson, jonathan.druart, kebliss, klas.blomberg, rcoert |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | Trivial patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: |
Bug 27691: Add $raw filter to report notes output
Bug 27691: Add $raw filter to report notes output Bug 27691: Add $raw filter to report notes output |
||
|
Description
Lucas Gass (lukeg)
2021-02-11 23:25:41 UTC
Created attachment 116783 [details] [review] Bug 27691: Add $raw filter to report notes output To test: Add a report with some notes that include HTML: <p>TEST</p> Run the report, the HTML displays literally Apply patch Run the report again. HTML looks good Created attachment 116836 [details] [review] Bug 27691: Add $raw filter to report notes output To test: Add a report with some notes that include HTML: <p>TEST</p> Run the report, the HTML displays literally Apply patch Run the report again. HTML looks good Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov> The html looked great. This is a big improvement. Created attachment 116853 [details] [review] Bug 27691: Add $raw filter to report notes output To test: Add a report with some notes that include HTML: <p>TEST</p> Run the report, the HTML displays literally Apply patch Run the report again. HTML looks good Signed-off-by: Barbara Johnson <barbara.johnson@bedfordtx.gov> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> The problem is that this saved_sql.notes field can be filled in by Mana, so I don't think we can trust the data coming from there. We should first make sure the data is sanitized before it is inserted into the Mana DB. Is there a way we could remove HTML elements before uploading to Mana as a compromise? I believe the tagging feature does "scrubbing" of the input in a way, maybe that could be useful? Yes, it's what I meant with "sanitized", HTML tags must be removed. Could this be solved with a new sort of HTML-cumstomizations - that are linked to one or more reports instead of specific pages? |