Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-28270 Cohort improvements META
  3. MDL-22404

Missing cohorts filter/search criteria from Bulk user action

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1, 2.2
    • Fix Version/s: 2.1.1
    • Component/s: Administration
    • Labels:
      None

      Description

      Fresh install Moodle 2.0 Preview 1 (Build: 20100510)

      Bulk user action does not show any filter settings for Choorts, either for selecting or assigning users.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            neogic B Johnson added a comment - - edited

            Big +1 for this - very strange that Cohorts aren't a filterable field, given that this is Moodle's site-wide grouping mechanism. This issue needs votes. Here's related discussion: MDL-26612. I'd also suggest adding "Add to Cohort" to the actions list (and "Enrol on course" seems an obvious usability improvement).

            BTW Andrea - I'd suggest including the word "filter" in your title (e.g., ".. filter/search criteria ..") or summary, to aid people searching for this issue, as it's the "New filter" section of the Bulk User Actions / List Users screens.

            Show
            neogic B Johnson added a comment - - edited Big +1 for this - very strange that Cohorts aren't a filterable field, given that this is Moodle's site-wide grouping mechanism. This issue needs votes . Here's related discussion: MDL-26612 . I'd also suggest adding " Add to Cohort " to the actions list (and "Enrol on course" seems an obvious usability improvement). BTW Andrea - I'd suggest including the word "filter" in your title (e.g., ".. filter/search criteria ..") or summary, to aid people searching for this issue, as it's the "New filter" section of the Bulk User Actions / List Users screens.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Halting this until:

            1) missing © header.
            2) should it be backported to 2.0? I think 2.1 is ok, but 2.0?

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Halting this until: 1) missing © header. 2) should it be backported to 2.0? I think 2.1 is ok, but 2.0? Ciao
            Hide
            skodak Petr Skoda added a comment -

            1/ the other filters do not have headers too, so I did not bother
            2/ I think there is no need to backport to 2.0

            Show
            skodak Petr Skoda added a comment - 1/ the other filters do not have headers too, so I did not bother 2/ I think there is no need to backport to 2.0
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated (master and 21). Thanks!

            Side note: I'm not going to reject this because of the copyright missing but I think any new file should have it.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (master and 21). Thanks! Side note: I'm not going to reject this because of the copyright missing but I think any new file should have it.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            (grrr, sorry for the spam, had to restart the whole loop to arrive to the correct transition). Integrated!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - (grrr, sorry for the spam, had to restart the whole loop to arrive to the correct transition). Integrated!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Searching for system / category cohorts return members as expected. Passing.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Searching for system / category cohorts return members as expected. Passing.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Congratulations - this fix has just been released in the weeklies.

            Show
            samhemelryk Sam Hemelryk added a comment - Congratulations - this fix has just been released in the weeklies.

              People

              • Votes:
                4 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  1/Aug/11