From 8efe31ee9671e77ced832968ab039d4c2080fc7d Mon Sep 17 00:00:00 2001 From: Sara Arjona Date: Tue, 26 Mar 2024 13:28:58 +0100 Subject: [PATCH] MDL-73975 course: Final deprecation methods in management_renderer Final deprecation for management_heading() and course_search_form() in core_course_management_renderer. --- course/classes/management_renderer.php | 64 +++++++------------------- course/upgrade.txt | 2 + theme/boost/scss/moodle/course.scss | 37 --------------- theme/boost/style/moodle.css | 29 ------------ theme/classic/style/moodle.css | 29 ------------ 5 files changed, 18 insertions(+), 143 deletions(-) diff --git a/course/classes/management_renderer.php b/course/classes/management_renderer.php index 60a4844b548..3d64a3a7eaf 100644 --- a/course/classes/management_renderer.php +++ b/course/classes/management_renderer.php @@ -62,40 +62,16 @@ class core_course_management_renderer extends plugin_renderer_base { } /** - * Displays a heading for the management pages. - * * @deprecated since Moodle 4.0. This is now handled/replaced with the tertiary navigation - * @todo Final deprecation MDL-73975 - * @param string $heading The heading to display - * @param string|null $viewmode The current view mode if there are options. - * @param int|null $categoryid The currently selected category if there is one. - * @return string */ - public function management_heading($heading, $viewmode = null, $categoryid = null) { - debugging('management_heading() is deprecated. Use the class manage_categories_action_bar instead.', DEBUG_DEVELOPER); - - $html = html_writer::start_div('coursecat-management-header clearfix'); - if (!empty($heading)) { - $html .= $this->heading($heading); - } - if ($viewmode !== null) { - $html .= html_writer::start_div(); - $html .= $this->view_mode_selector(\core_course\management\helper::get_management_viewmodes(), $viewmode); - if ($viewmode === 'courses') { - $categories = core_course_category::make_categories_list(array('moodle/category:manage', 'moodle/course:create')); - $nothing = false; - if ($categoryid === null) { - $nothing = array('' => get_string('selectacategory')); - $categoryid = ''; - } - $select = new single_select($this->page->url, 'categoryid', $categories, $categoryid, $nothing); - $select->attributes['aria-label'] = get_string('selectacategory'); - $html .= $this->render($select); - } - $html .= html_writer::end_div(); - } - $html .= html_writer::end_div(); - return $html; + #[\core\attribute\deprecated( + replacement: 'manage_categories_action_bar', + since: '4.0', + mdl: 'MDL-73462', + final: true, + )] + public function management_heading() { + \core\deprecation::emit_deprecation_if_present([self::class, __FUNCTION__]); } /** @@ -1301,24 +1277,16 @@ class core_course_management_renderer extends plugin_renderer_base { } /** - * Renders html to display a course search form - * * @deprecated since Moodle 4.0. This is now handled within manage_categories_action_bar - * @todo Final deprecation MDL-73975 - * @param string $value default value to populate the search field - * @return string */ - public function course_search_form($value = '') { - debugging('course_search_form() is deprecated. Use the class manage_categories_action_bar instead.', DEBUG_DEVELOPER); - $data = [ - 'action' => new moodle_url('/course/management.php'), - 'btnclass' => 'btn-primary', - 'extraclasses' => 'my-3 d-flex justify-content-center', - 'inputname' => 'search', - 'searchstring' => get_string('searchcourses'), - 'value' => $value - ]; - return $this->render_from_template('core/search_input', $data); + #[\core\attribute\deprecated( + replacement: 'manage_categories_action_bar', + since: '4.0', + mdl: 'MDL-73462', + final: true, + )] + public function course_search_form() { + \core\deprecation::emit_deprecation_if_present([self::class, __FUNCTION__]); } /** diff --git a/course/upgrade.txt b/course/upgrade.txt index 2131c56c033..77eff50c0dd 100644 --- a/course/upgrade.txt +++ b/course/upgrade.txt @@ -19,6 +19,8 @@ information provided here is intended especially for developers. - `course_purge_section_cache` - `course_purge_module_cache` - `get_array_of_activities` + - `core_course_management_renderer::management_heading` + - `core_course_management_renderer::course_search_form` * New format actions classes. Those classes will eventually replace all course/lib.php content editing functions. All new methods are distributed in three classes formats can extend. Method can be accessed using static methods (see doc block of core_courseformat\formatactions for more information). diff --git a/theme/boost/scss/moodle/course.scss b/theme/boost/scss/moodle/course.scss index 7a3c5c47fa0..a0eeb56ec1d 100644 --- a/theme/boost/scss/moodle/course.scss +++ b/theme/boost/scss/moodle/course.scss @@ -985,43 +985,6 @@ span.editinstructions { } /** Management header styling **/ -.coursecat-management-header { - vertical-align: middle; - - h2 { - display: inline-block; - text-align: left; - } - - > div { - float: right; - - > div { - margin: 10px 0 10px 1em; - display: inline-block; - } - } - - select { - max-width: 300px; - cursor: pointer; - padding: 0.4em 0.5em 0.45em 1em; - vertical-align: baseline; - white-space: nowrap; - } - - .view-mode-selector { - .moodle-actionmenu { - white-space: nowrap; - display: inline-block; - } - - .moodle-actionmenu[data-enhanced].show .menu a { - padding-left: 1em; - } - } -} - .course-being-dragged-proxy { border: 0; color: $link-color; diff --git a/theme/boost/style/moodle.css b/theme/boost/style/moodle.css index 9a190efab09..127b39d4fe1 100644 --- a/theme/boost/style/moodle.css +++ b/theme/boost/style/moodle.css @@ -28884,35 +28884,6 @@ span.editinstructions .alert-link { } /** Management header styling **/ -.coursecat-management-header { - vertical-align: middle; -} -.coursecat-management-header h2 { - display: inline-block; - text-align: left; -} -.coursecat-management-header > div { - float: right; -} -.coursecat-management-header > div > div { - margin: 10px 0 10px 1em; - display: inline-block; -} -.coursecat-management-header select { - max-width: 300px; - cursor: pointer; - padding: 0.4em 0.5em 0.45em 1em; - vertical-align: baseline; - white-space: nowrap; -} -.coursecat-management-header .view-mode-selector .moodle-actionmenu { - white-space: nowrap; - display: inline-block; -} -.coursecat-management-header .view-mode-selector .moodle-actionmenu[data-enhanced].show .menu a { - padding-left: 1em; -} - .course-being-dragged-proxy { border: 0; color: #0f6cbf; diff --git a/theme/classic/style/moodle.css b/theme/classic/style/moodle.css index 2ce2cd99c84..f120e6a6dda 100644 --- a/theme/classic/style/moodle.css +++ b/theme/classic/style/moodle.css @@ -28884,35 +28884,6 @@ span.editinstructions .alert-link { } /** Management header styling **/ -.coursecat-management-header { - vertical-align: middle; -} -.coursecat-management-header h2 { - display: inline-block; - text-align: left; -} -.coursecat-management-header > div { - float: right; -} -.coursecat-management-header > div > div { - margin: 10px 0 10px 1em; - display: inline-block; -} -.coursecat-management-header select { - max-width: 300px; - cursor: pointer; - padding: 0.4em 0.5em 0.45em 1em; - vertical-align: baseline; - white-space: nowrap; -} -.coursecat-management-header .view-mode-selector .moodle-actionmenu { - white-space: nowrap; - display: inline-block; -} -.coursecat-management-header .view-mode-selector .moodle-actionmenu[data-enhanced].show .menu a { - padding-left: 1em; -} - .course-being-dragged-proxy { border: 0; color: #0f6cbf;