MDL-69145 core_user: Set default filter to ALL
authorJun Pataleta <jun@moodle.com>
Tue, 24 Nov 2020 14:22:18 +0000 (22:22 +0800)
committerJun Pataleta <jun@moodle.com>
Wed, 9 Dec 2020 02:51:12 +0000 (10:51 +0800)
user/amd/build/participantsfilter.min.js
user/amd/build/participantsfilter.min.js.map
user/amd/src/participantsfilter.js
user/templates/participantsfilter.mustache
user/tests/behat/filter_participants.feature

index 1f5b3f0..d99bc1b 100644 (file)
Binary files a/user/amd/build/participantsfilter.min.js and b/user/amd/build/participantsfilter.min.js differ
index 05781b4..63fcd31 100644 (file)
Binary files a/user/amd/build/participantsfilter.min.js.map and b/user/amd/build/participantsfilter.min.js.map differ
index 3f17241..c084f58 100644 (file)
@@ -337,8 +337,8 @@ export const init = participantsRegionId => {
 
         if (filters.length === 1) {
             filterSet.querySelector(Selectors.filterset.regions.filtermatch).classList.add('hidden');
-            filterSet.querySelector(Selectors.filterset.fields.join).value = 1;
-            filterSet.dataset.filterverb = 1;
+            filterSet.querySelector(Selectors.filterset.fields.join).value = 2;
+            filterSet.dataset.filterverb = 2;
         } else {
             filterSet.querySelector(Selectors.filterset.regions.filtermatch).classList.remove('hidden');
         }
index e9315f8..fc9a8b8 100644 (file)
@@ -40,7 +40,7 @@
     }
 }}
 
-<div id="core_user-participantsfilter-{{uniqid}}" class="filter-group my-2 p-2 bg-light border-radius border" data-table-region="{{tableregionid}}" data-table-course-id="{{courseid}}" data-filterverb="1">
+<div id="core_user-participantsfilter-{{uniqid}}" class="filter-group my-2 p-2 bg-light border-radius border" data-table-region="{{tableregionid}}" data-table-course-id="{{courseid}}" data-filterverb="2">
     <div data-filterregion="filtermatch" class="hidden">
         <label for="core_user-local-participantsfilter-jointype-{{uniqid}}" class="my-0" aria-hidden="true">
             {{#str}}match, core_user{{/str}}
@@ -48,8 +48,8 @@
         <select class="custom-select" data-filterfield="join" id="core_user-local-participantsfilter-jointype-{{uniqid}}"
                 aria-label="{{#str}}filtersetmatchdescription, core_user{{/str}}">
             <option value="0">{{#str}}none{{/str}}</option>
-            <option selected=selected value="1">{{#str}}any{{/str}}</option>
-            <option value="2">{{#str}}all{{/str}}</option>
+            <option value="1">{{#str}}any{{/str}}</option>
+            <option value="2" selected>{{#str}}all{{/str}}</option>
         </select>
         <span aria-hidden="true">{{#str}}matchofthefollowing, core_user{{/str}}</span>
     </div>
index e962dfc..b957693 100644 (file)
@@ -707,7 +707,7 @@ Feature: Course participants can be filtered
     And I should not see "Student 4" in the "participants" "table"
     And I should not see "Teacher 1" in the "participants" "table"
     And I click on "Add condition" "button"
-    # Re-add a second filter and ensure the default (any) filterset match type is set.
+    # Re-add a second filter and ensure the default (All) filterset match type is set.
     And I set the field "Match" in the "Filter 2" "fieldset" to "All"
     And I set the field "type" in the "Filter 2" "fieldset" to "Role"
     And I click on ".form-autocomplete-downarrow" "css_element" in the "Filter 2" "fieldset"
@@ -716,7 +716,7 @@ Feature: Course participants can be filtered
     And I should see "Student 1" in the "participants" "table"
     And I should see "Student 2" in the "participants" "table"
     And I should see "Student 3" in the "participants" "table"
-    And I should see "Student 4" in the "participants" "table"
+    And I should not see "Student 4" in the "participants" "table"
     And I should not see "Teacher 1" in the "participants" "table"
 
   @javascript