Bugfix: Course category breadcrumbs were broken on the course enrolment page due to MDL-80974 and were removed, resolves #727 (#728)

Author: abias

CHANGES.md

@@ -6,6 +6,7 @@ Changes
 
 ### Unreleased
 
+* 2024-10-09 - Bugfix: Course category breadcrumbs were broken on the course enrolment page due to MDL-80974 and were removed, resolves #727.
 * 2024-10-08 - Upstream change: Adopt change from MDL-82298 into smartmenus-[card|more]menu-children.mustache
 * 2024-08-11 - Updated Moodle Plugin CI to latest upstream recommendations
 * 2024-07-24 - Test: Fix broken Behat scenario 'Suppress 'Chat to course participants' link', resolves #696

classes/boostnavbar.php

@@ -168,8 +168,8 @@ protected function prepare_nodes_for_boost(): void {
 
         // Don't display the navbar if there is only one item. Apparently this is bad UX design.
         // Except, leave it in when in course context and categorybreadcrumbs are desired.
-        if (get_config('theme_boost_union', 'categorybreadcrumbs') != THEME_BOOST_UNION_SETTING_SELECT_YES &&
-                $this->page->context->contextlevel == CONTEXT_COURSE) {
+        if (!(get_config('theme_boost_union', 'categorybreadcrumbs') == THEME_BOOST_UNION_SETTING_SELECT_YES &&
+                $this->page->context->contextlevel == CONTEXT_COURSE)) {
             if ($this->item_count() <= 1) {
                 $this->clear_items();
                 return;

tests/behat/theme_boost_union_feelsettings_navigation.feature

@@ -162,20 +162,18 @@ Feature: Configuring the theme_boost_union plugin for the "Navigation" tab on th
       | Category ED    | 1        | CED      | CE       |
       | Category EDC   | 2        | CEDC     | CED      |
       | Category EDCB  | 3        | CEDCB    | CEDC     |
-      | Category EDCBA | 4        | CEDCBA   | CEDCB    |
     And the following "courses" exist:
       | fullname  | shortname | category |
       | Course C1 | CC1       | CE       |
       | Course C2 | CC2       | CED      |
       | Course C3 | CC3       | CEDC     |
       | Course C4 | CC4       | CEDCB    |
-      | Course C5 | CC5       | CEDCBA   |
     And the following "course enrolments" exist:
       | user     | course | role           |
+      | teacher1 | CC1    | editingteacher |
       | teacher1 | CC2    | editingteacher |
       | teacher1 | CC3    | editingteacher |
       | teacher1 | CC4    | editingteacher |
-      | teacher1 | CC5    | editingteacher |
     And the following config values are set as admin:
       | config              | value     | plugin            |
       | categorybreadcrumbs | <setting> | theme_boost_union |
@@ -194,12 +192,6 @@ Feature: Configuring the theme_boost_union plugin for the "Navigation" tab on th
     And "Category ED" "link" <shouldornot> exist in the ".breadcrumb" "css_element"
     And "Category EDC" "link" <shouldornot> exist in the ".breadcrumb" "css_element"
     And "Category EDCB" "link" <shouldornot> exist in the ".breadcrumb" "css_element"
-    And I am on "Course C5" course homepage
-    And "Category E" "link" <shouldornot> exist in the ".breadcrumb" "css_element"
-    And "Category ED" "link" <shouldornot> exist in the ".breadcrumb" "css_element"
-    And "Category EDC" "link" <shouldornot> exist in the ".breadcrumb" "css_element"
-    And "Category EDCB" "link" <shouldornot> exist in the ".breadcrumb" "css_element"
-    And "Category EDCBA" "link" <shouldornot> exist in the ".breadcrumb" "css_element"
 
     Examples:
       | setting | shouldornot |
@@ -213,24 +205,17 @@ Feature: Configuring the theme_boost_union plugin for the "Navigation" tab on th
       | Category ED    | 1        | CED      | CE       |
     And the following "courses" exist:
       | fullname  | shortname | category |
-      | Course C1 | CC1       | CE       |
-      | Course C2 | CC2       | CED      |
+      | Course C1 | CC1       | CED      |
     And the following "course enrolments" exist:
       | user     | course | role           |
-      | teacher1 | CC2    | editingteacher |
+      | teacher1 | CC1    | editingteacher |
     And the following config values are set as admin:
       | config              | value     | plugin            |
       | categorybreadcrumbs | yes       | theme_boost_union |
     And "Course C1" has been set to one page per section
-    And "Course C2" has been set to one page per section
     When I log in as "teacher1"
     And I am on section "1" page of "Course C1" course
     Then "Category E" "link" should exist in the ".breadcrumb" "css_element"
-    And "Enrolment options" "text" should exist in the ".breadcrumb" "css_element"
-    And "Enrolment options" "text" should appear after "Category E" "link" in the ".breadcrumb" "css_element"
-    And "Topic 1" "link" should not exist in the ".breadcrumb" "css_element"
-    And I am on section "1" page of "Course C2" course
-    And "Category E" "link" should exist in the ".breadcrumb" "css_element"
     And "Category ED" "link" should exist in the ".breadcrumb" "css_element"
     And "Topic 1" "link" should exist in the ".breadcrumb" "css_element"
     And "Category ED" "link" should appear after "Category E" "link" in the ".breadcrumb" "css_element"