From 26271624807ac0e9042a3e64deca6a062daacacc Mon Sep 17 00:00:00 2001 From: Mathew May Date: Wed, 13 Feb 2019 14:06:30 +0800 Subject: [PATCH] MDL-58428 renderer: Move renderer override from Question bank --- question/renderer.php | 12 +++++++++++- .../output/core_question/bank_renderer.php | 18 ++++-------------- 2 files changed, 15 insertions(+), 15 deletions(-) rename theme/{boost => bootstrapbase}/classes/output/core_question/bank_renderer.php (65%) diff --git a/question/renderer.php b/question/renderer.php index 70b9127f66f..76ef282cda5 100644 --- a/question/renderer.php +++ b/question/renderer.php @@ -41,7 +41,17 @@ class core_question_bank_renderer extends plugin_renderer_base { * @return string */ public function extra_horizontal_navigation() { - // Overwrite in child themes if needed. + // Horizontal navigation for question bank. + if ($questionnode = $this->page->settingsnav->find("questionbank", \navigation_node::TYPE_CONTAINER)) { + if ($children = $questionnode->children) { + $tabs = []; + foreach ($children as $key => $node) { + $tabs[] = new \tabobject($node->key, $node->action, $node->text); + } + $active = $questionnode->find_active_node()->key; + return \html_writer::div(print_tabs([$tabs], $active, null, null, true), 'questionbank-navigation'); + } + } return ''; } diff --git a/theme/boost/classes/output/core_question/bank_renderer.php b/theme/bootstrapbase/classes/output/core_question/bank_renderer.php similarity index 65% rename from theme/boost/classes/output/core_question/bank_renderer.php rename to theme/bootstrapbase/classes/output/core_question/bank_renderer.php index 8f4962a7e78..2e191cbcc4e 100644 --- a/theme/boost/classes/output/core_question/bank_renderer.php +++ b/theme/bootstrapbase/classes/output/core_question/bank_renderer.php @@ -17,12 +17,12 @@ /** * Question renderer. * - * @package theme_boost + * @package theme_bootstrapbase * @copyright 2017 onwards Ankit Agarwal * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later */ -namespace theme_boost\output\core_question; +namespace theme_bootstrapbase\output\core_question; defined('MOODLE_INTERNAL') || die(); require_once($CFG->dirroot . '/' . $CFG->admin . '/renderer.php'); @@ -30,7 +30,7 @@ require_once($CFG->dirroot . '/' . $CFG->admin . '/renderer.php'); /** * Question renderer class. * - * @package theme_boost + * @package theme_bootstrapbase * @copyright 2017 onwards Ankit Agarwal * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later */ @@ -42,17 +42,7 @@ class bank_renderer extends \core_question_bank_renderer { * @return string */ public function extra_horizontal_navigation() { - // Horizontal navigation for question bank. - if ($questionnode = $this->page->settingsnav->find("questionbank", \navigation_node::TYPE_CONTAINER)) { - if ($children = $questionnode->children) { - $tabs = []; - foreach ($children as $key => $node) { - $tabs[] = new \tabobject($node->key, $node->action, $node->text); - } - $active = $questionnode->find_active_node()->key; - return \html_writer::div(print_tabs([$tabs], $active, null, null, true), 'questionbank-navigation'); - } - } + // Overwrite in child themes if needed. return ''; } } -- 2.43.0