Repository navigation
theme_boost_o365teams: Remove dead core_course_renderer overrides - #3775
Lai Wei (weilai-irl) wants to merge 1 commit into
Conversation
course_section_cm() and course_section_cm_name_title() no longer exist in the Moodle core course renderer, so the overrides were never called.
There was a problem hiding this comment.
🟢 Approval recommended
It is a clean, fully-verified removal of dead code with no remaining references to the deleted class or file anywhere in the repository.
0 open findings
What changed in this PR
This PR removes a dead core_course_renderer override from the theme_boost_o365teams theme. The overridden methods course_section_cm() and course_section_cm_name_title() no longer exist on Moodle core's core_course_renderer (course module rendering has moved to the core_courseformat\output classes in recent Moodle versions, and the theme requires Moodle 5.0 / 2025040800), so the override class was never invoked and only carried maintenance burden.
Changes:
- Deletes
theme/boost_o365teams/classes/core_course_renderer.php(the dead override class). - Removes the
require_oncefor that file fromtheme/boost_o365teams/renderers.php.
| File | Description |
|---|---|
| theme/boost_o365teams/classes/core_course_renderer.php | Removes the dead renderer override class that is no longer called by core. |
| theme/boost_o365teams/renderers.php | Drops the require_once include for the deleted renderer file. |
I verified repo-wide that no remaining references to the class theme_boost_o365teams_core_course_renderer or the deleted file path exist, so the removal is complete and self-consistent. No issues to comment on within the changed regions.
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
course_section_cm() and course_section_cm_name_title() no longer exist in the Moodle core course renderer, so the overrides were never called.