diff --git a/contentbank/classes/output/bankcontent.php b/contentbank/classes/output/bankcontent.php index c0826adc967..632550434c9 100644 --- a/contentbank/classes/output/bankcontent.php +++ b/contentbank/classes/output/bankcontent.php @@ -149,14 +149,15 @@ class bankcontent implements renderable, templatable { $allowedcontexts['courses'] = [get_string('courses') => $options]; } if (!empty($allowedcontexts)) { - $url = new \moodle_url('/contentbank/index.php'); + $strchoosecontext = get_string('choosecontext', 'core_contentbank'); $singleselect = new \single_select( - $url, + new \moodle_url('/contentbank/index.php'), 'contextid', $allowedcontexts, $this->context->id, - get_string('choosecontext', 'core_contentbank') + $strchoosecontext ); + $singleselect->set_label($strchoosecontext, ['class' => 'sr-only']); $data->allowedcontexts = $singleselect->export_for_template($output); } diff --git a/contentbank/tests/behat/navigate_content.feature b/contentbank/tests/behat/navigate_content.feature index 6faf18af435..2f2657bed3f 100644 --- a/contentbank/tests/behat/navigate_content.feature +++ b/contentbank/tests/behat/navigate_content.feature @@ -34,21 +34,20 @@ Feature: Navigate to different contexts in the content bank And I add the "Navigation" block if not present And I expand "Site pages" node When I click on "Content bank" "link" - And "contextid" "select" should exist - And the "contextid" select box should contain "System" - And the "contextid" select box should contain "Cat 1" - And the "contextid" select box should contain "Cat 2" - And the "contextid" select box should contain "C0" - And the "contextid" select box should contain "C1" - And the "contextid" select box should contain "C2" + And the "Choose course or category" select box should contain "System" + And the "Choose course or category" select box should contain "Cat 1" + And the "Choose course or category" select box should contain "Cat 2" + And the "Choose course or category" select box should contain "C0" + And the "Choose course or category" select box should contain "C1" + And the "Choose course or category" select box should contain "C2" And I should see "santjordi.h5p" And I should not see "santjordi_rose.h5p" And I should not see "Dragon.h5p" - And I set the field "contextid" to "Cat 1" + And I set the field "Choose course or category" to "Cat 1" Then I should not see "santjordi.h5p" And I should see "santjordi_rose.h5p" And I should not see "Dragon.h5p" - And I set the field "contextid" to "C0" + And I set the field "Choose course or category" to "C0" And I should not see "santjordi.h5p" And I should not see "santjordi_rose.h5p" And I should see "Dragon.h5p" @@ -69,18 +68,17 @@ Feature: Navigate to different contexts in the content bank And I add the "Navigation" block if not present And I expand "Site pages" node When I click on "Content bank" "link" - And "contextid" "select" should exist - And the "contextid" select box should contain "C0" - And the "contextid" select box should contain "C1" - And the "contextid" select box should not contain "System" - And the "contextid" select box should not contain "Cat 1" - And the "contextid" select box should not contain "Cat 2" - And the "contextid" select box should not contain "C2" + And the "Choose course or category" select box should contain "C0" + And the "Choose course or category" select box should contain "C1" + And the "Choose course or category" select box should not contain "System" + And the "Choose course or category" select box should not contain "Cat 1" + And the "Choose course or category" select box should not contain "Cat 2" + And the "Choose course or category" select box should not contain "C2" And I should see "Dragon.h5p" And I should not see "princess.h5p" And I should not see "santjordi.h5p" And I should not see "santjordi_rose.h5p" - And I set the field "contextid" to "C1" + And I set the field "Choose course or category" to "C1" Then I should not see "Dragon.h5p" And I should see "princess.h5p" And I should not see "santjordi.h5p" @@ -91,14 +89,13 @@ Feature: Navigate to different contexts in the content bank And I am on the "C0" "Course" page logged in as "teacher" And I expand "Site pages" node When I click on "Content bank" "link" - And "contextid" "select" should exist - And the "contextid" select box should contain "C0" - And the "contextid" select box should contain "C1" - And the "contextid" select box should contain "Cat 1" - And the "contextid" select box should not contain "System" - And the "contextid" select box should not contain "Cat 2" - And the "contextid" select box should not contain "C2" + And the "Choose course or category" select box should contain "C0" + And the "Choose course or category" select box should contain "C1" + And the "Choose course or category" select box should contain "Cat 1" + And the "Choose course or category" select box should not contain "System" + And the "Choose course or category" select box should not contain "Cat 2" + And the "Choose course or category" select box should not contain "C2" And I should see "Dragon.h5p" - And I set the field "contextid" to "Cat 1" + And I set the field "Choose course or category" to "Cat 1" And I should not see "Dragon.h5p" And I should see "santjordi_rose.h5p"