diff --git a/lib/deprecatedlib.php b/lib/deprecatedlib.php index 833ca428641..77e7c2a3343 100644 --- a/lib/deprecatedlib.php +++ b/lib/deprecatedlib.php @@ -1757,11 +1757,11 @@ class custom_corners_renderer_factory extends standard_renderer_factory { require_once($CFG->themedir . '/custom_corners/renderers.php'); } /* Implement the subclass method. */ - public function get_renderer($module, $page) { + public function get_renderer($module, $page, $subtype=null) { if ($module == 'core') { return new custom_corners_core_renderer($page); } - return parent::get_renderer($module, $page); + return parent::get_renderer($module, $page, $subtype); } } diff --git a/lib/outputlib.php b/lib/outputlib.php index 877b0aed160..c084b27f547 100644 --- a/lib/outputlib.php +++ b/lib/outputlib.php @@ -59,9 +59,10 @@ interface renderer_factory { * * @param string $component name such as 'core', 'mod_forum' or 'qtype_multichoice'. * @param moodle_page $page the page the renderer is outputting content for. + * @param string $subtype optional subtype such as 'news' resulting to 'mod_forum_news' * @return object an object implementing the requested renderer interface. */ - public function get_renderer($component, $page); + public function get_renderer($component, $page, $subtype=null); } @@ -431,8 +432,9 @@ class theme_config { * @param string $module the name of part of moodle. E.g. 'core', 'quiz', 'qtype_multichoice'. * @param moodle_page $page the page we are rendering * @return moodle_renderer_base the requested renderer. + * @param string $subtype optional subtype such as 'news' resulting to 'mod_forum_news' */ - public function get_renderer($module, $page) { + public function get_renderer($module, $page, $subtype=null) { if (is_null($this->rf)) { if (CLI_SCRIPT) { $classname = 'cli_renderer_factory'; @@ -442,7 +444,7 @@ class theme_config { $this->rf = new $classname($this); } - return $this->rf->get_renderer($module, $page); + return $this->rf->get_renderer($module, $page, $subtype); } /** @@ -882,16 +884,21 @@ abstract class renderer_factory_base implements renderer_factory { * the class definition of the default renderer has been loaded. * * @param string $component name such as 'core', 'mod_forum' or 'qtype_multichoice'. + * @param string $subtype optional subtype such as 'news' resulting to 'mod_forum_news' * @return string the name of the standard renderer class for that module. */ - protected function standard_renderer_class_for_module($component) { + protected function standard_renderer_class_for_module($component, $subtype=null) { if ($component != 'core') { $pluginrenderer = get_component_directory($component) . '/renderer.php'; if (file_exists($pluginrenderer)) { include_once($pluginrenderer); } } - $class = 'moodle_' . $component . '_renderer'; + if (is_null($subtype)) { + $class = 'moodle_' . $component . '_renderer'; + } else { + $class = 'moodle_' . $component . '_' . $subtype . '_renderer'; + } if (!class_exists($class)) { throw new coding_exception('Request for an unknown renderer class ' . $class); } @@ -910,11 +917,11 @@ abstract class renderer_factory_base implements renderer_factory { */ class standard_renderer_factory extends renderer_factory_base { /* Implement the subclass method. */ - public function get_renderer($module, $page) { + public function get_renderer($module, $page, $subtype=null) { if ($module == 'core') { return new moodle_core_renderer($page); } else { - $class = $this->standard_renderer_class_for_module($module); + $class = $this->standard_renderer_class_for_module($module, $subtype); return new $class($page, $this->get_renderer('core', $page)); } } @@ -930,11 +937,11 @@ class standard_renderer_factory extends renderer_factory_base { */ class cli_renderer_factory extends standard_renderer_factory { /* Implement the subclass method. */ - public function get_renderer($module, $page) { + public function get_renderer($module, $page, $subtype=null) { if ($module == 'core') { return new cli_core_renderer($page); } else { - parent::get_renderer($module, $page); + parent::get_renderer($module, $page, $subtype); } } } @@ -984,9 +991,13 @@ class theme_overridden_renderer_factory extends standard_renderer_factory { } /* Implement the subclass method. */ - public function get_renderer($module, $page) { + public function get_renderer($module, $page, $subtype=null) { foreach ($this->prefixes as $prefix) { - $classname = $prefix . $module . '_renderer'; + if (is_null($subtype)) { + $classname = $prefix . $module . '_renderer'; + } else { + $classname = $prefix . $module . '_' . $subtype . '_renderer'; + } if (class_exists($classname)) { if ($module == 'core') { return new $classname($page); @@ -995,7 +1006,7 @@ class theme_overridden_renderer_factory extends standard_renderer_factory { } } } - return parent::get_renderer($module, $page); + return parent::get_renderer($module, $page, $subtype); } } @@ -1057,18 +1068,21 @@ class template_renderer_factory extends renderer_factory_base { } /* Implement the subclass method. */ - public function get_renderer($module, $page) { + public function get_renderer($module, $page, $subtype=null) { // Refine the list of search paths for this module. $searchpaths = array(); foreach ($this->searchpaths as $rootpath) { $path = $rootpath . '/' . $module; + if (!is_null($subtype)) { + $path .= '/' . $subtype; + } if (is_dir($path)) { $searchpaths[] = $path; } } // Create a template_renderer that copies the API of the standard renderer. - $copiedclass = $this->standard_renderer_class_for_module($module); + $copiedclass = $this->standard_renderer_class_for_module($module, $subtype); return new template_renderer($copiedclass, $searchpaths, $page); } } diff --git a/lib/simpletest/testoutputlib.php b/lib/simpletest/testoutputlib.php index dfa5f0ed9cc..28ff563ba73 100644 --- a/lib/simpletest/testoutputlib.php +++ b/lib/simpletest/testoutputlib.php @@ -92,13 +92,15 @@ class testable_renderer_factory extends renderer_factory_base { parent::__construct(null); } - public function get_renderer($module, $page) { - $this->createcalls[] = $module; + public function get_renderer($module, $page, $subtype=null) { + if (!in_array(array($module, $subtype), $this->createcalls)) { + $this->createcalls[] = array($module, $subtype); + } return new moodle_core_renderer($page); } - public function standard_renderer_class_for_module($module) { - return parent::standard_renderer_class_for_module($module); + public function standard_renderer_class_for_module($module, $subtype=null) { + return parent::standard_renderer_class_for_module($module, $subtype); } } @@ -136,6 +138,23 @@ class moodle_mod_test_renderer extends moodle_core_renderer { } +/** + * Renderer class for testing subrendering feature + * + * @copyright 2009 David Mudrak + * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later + */ +class moodle_mod_test_subtype_renderer extends moodle_core_renderer { + public function __construct($containerstack, $page) { + parent::__construct($containerstack, $page, null); + } + + public function signature($user = 'Administrator') { + return '