Bug 22586

Summary: IntranetReportsHomeHTML no longer renders as HTML on reports-home.pl
Product: Koha Reporter: Lucas Gass (lukeg) <lucas>
Component: TemplatesAssignee: Lucas Gass (lukeg) <lucas>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: trivial    
Priority: P5 - low CC: george, lisettepalouse+koha, m.de.rooy, martin.renvoize, nick
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on: 13618    
Bug Blocks:    
Attachments: Bug 22586: fix IntranetReportsHomeHTML filter
Bug 22586: IntranetReportsHomeHTML no longer renders as HTML on reports-home.pl
Bug 22586: IntranetReportsHomeHTML no longer renders as HTML on reports-home.pl
Bug 22586: IntranetReportsHomeHTML no longer renders as HTML on reports-home.pl

Description Lucas Gass (lukeg) 2019-03-26 21:23:43 UTC
As of 18.11 the IntranetReportsHomeHTML syspref no longer renders HTML on report-home.pl. I think this is being filtered wrong in the template.
Comment 1 Lucas Gass (lukeg) 2019-03-27 16:31:50 UTC
Created attachment 87095 [details] [review]
Bug 22586: fix IntranetReportsHomeHTML filter
Comment 2 Lucas Gass (lukeg) 2019-03-27 16:41:25 UTC
Created attachment 87096 [details] [review]
Bug 22586: IntranetReportsHomeHTML no longer renders as HTML on reports-home.pl

As of 18.11 the IntranetReportsHomeHTML syspref no longer renders HTML
on report-home.pl. I think this is being filtered wrong in the template.

Test Plan:

1) Edit IntranetReportsHomeHTML, set to "<p>Test</p>"
2) View a report, note that you see the html p tags are displayed
litterly
3) Apply this patch
4) Reload the page
5) Note the p tags are rendered as html
Comment 3 ByWater Sandboxes 2019-03-27 19:24:50 UTC
Created attachment 87116 [details] [review]
Bug 22586: IntranetReportsHomeHTML no longer renders as HTML on reports-home.pl

As of 18.11 the IntranetReportsHomeHTML syspref no longer renders HTML
on report-home.pl. I think this is being filtered wrong in the template.

Test Plan:

1) Edit IntranetReportsHomeHTML, set to "<p>Test</p>"
2) View a report, note that you see the html p tags are displayed
litterly
3) Apply this patch
4) Reload the page
5) Note the p tags are rendered as html

Signed-off-by: George Williams <george@nekls.org>
Comment 4 George Williams (NEKLS) 2019-03-28 13:31:12 UTC
Tested successfully on ByWater sandboxes
Comment 5 Marcel de Rooy 2019-03-29 10:18:53 UTC
Created attachment 87169 [details] [review]
Bug 22586: IntranetReportsHomeHTML no longer renders as HTML on reports-home.pl

As of 18.11 the IntranetReportsHomeHTML syspref no longer renders HTML
on report-home.pl. I think this is being filtered wrong in the template.

Test Plan:

1) Edit IntranetReportsHomeHTML, set to "<p>Test</p>"
2) View a report, note that you see the html p tags are displayed
litterly
3) Apply this patch
4) Reload the page
5) Note the p tags are rendered as html

Signed-off-by: George Williams <george@nekls.org>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 6 Nick Clemens (kidclamp) 2019-03-29 11:46:14 UTC
Awesome work all!

Pushed to master for 19.05
Comment 7 Martin Renvoize (ashimema) 2019-04-01 14:08:21 UTC
Pushed to 18.11.x for 18.11.05
Comment 8 Lucas Gass (lukeg) 2019-04-01 16:51:28 UTC
not backporting this to the 18.05.x series