-
Bug
-
Resolution: Fixed
-
Major
-
2.1.1, 3.0, 3.0.4, 3.1.1, 3.2
-
Any
-
MOODLE_21_STABLE, MOODLE_30_STABLE, MOODLE_31_STABLE, MOODLE_32_STABLE
-
MOODLE_30_STABLE, MOODLE_31_STABLE
-
MDL-29738-master -
-
- git grep for file_modify_html_header
- Confirm that the only mention is in lib/upgrade.txt and lib/deprecatedlib.php
- git grep for file_modify_html_header
We originally wanted to apply our stylesheets to all HTML content and did so via use of the $CFG->stylesheets variable, and manipulating the content of stored files as they served.
However, the section of the code fetching the stylesheets was commented out in 2009 (MDL-20204), and the $ufo code was removed back in 2014 (MDL-36669).
Since then this code has just been wasting cycles by performing totally unnecessary preg_match and preg_replace calls on all HTML content as it is served.
This issue was originally about fixing that code, but is now about removing it.
Most places where we did this in Moodle have been templatised (e.g. all outbound e-mails).
Original description follows
The forum module sends copy of discussion topics to people who has subscribed the forum. The mail sent is an html page. In this page the head section is empty, so no css link is reported. Subscribers receive text and images, but don't receive neither box nor font information nor colors nor any style information.
I checked the problem subscribing a forum of moodle.org: I received only text without any css information.
Add the style information manually in the discussion topic html doesn't solve the problem: they are automatically deleted by moodle.
(Edited to add description provided by Luciano in a later comment.)
http://moodle.org/mod/forum/discuss.php?d=187579
http://tracker.moodle.org/browse/MDL-29152
see MDL-21120 too.
SAME AS CLOSED 29152 - OPENED BECAUSE OF SUGGESTION OF MAUNO KOPERLAINEN
http://moodle.org/mod/forum/discuss.php?d=187581
- has a non-specific relationship to
-
MDL-52126 Mustache template over-escaping in forum emails
- Closed
- has been marked as being related by
-
MDL-36669 HTTP 404 errors due to /lib/ufo.js
- Closed
-
MDL-49682 Forum email template is not editable by themes
- Closed
-
MDL-55114 Remove unused variables and their usage
- Closed
- will be (partly) resolved by
-
MDL-52990 Enable all emails to be themed, ie add 4 wrapper moustache templates
- Closed
- will help resolve
-
MDL-21120 somehow fix hardcoded page styles in manually created pages and emails
- Closed
-
MDL-29152 No link to CSS is included in html mail sent to subscribers.
- Closed