MDL-58428 renderer: Move renderer override from Question bank

thebenkahn · Feb 26, 2019 · 2627162 · 2627162
1 parent 9f0595c
commit 2627162
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 15 deletions.
diff --git 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/...es/output/core_question/bank_renderer.php → ...es/output/core_question/bank_renderer.php b/...es/output/core_question/bank_renderer.php → ...es/output/core_question/bank_renderer.php
@@ -17,20 +17,20 @@
 /**
  * Question renderer.
  *
- * @package    theme_boost
+ * @package    theme_bootstrapbase
  * @copyright  2017 onwards Ankit Agarwal <[email protected]>
  * @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');
 
 /**
  * Question renderer class.
  *
- * @package    theme_boost
+ * @package    theme_bootstrapbase
  * @copyright  2017 onwards Ankit Agarwal <[email protected]>
  * @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 '';
     }
 }