Moodle
  1. Moodle
  2. MDL-36283

Manager at category level can't view user details when assigning Cohort members

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.5, 2.3.2
    • Fix Version/s: 2.2.7, 2.3.4
    • Component/s: Cohorts
    • Labels:

      Description

      Steps to reproduce:

      Make sure the at least "email" is checked for setting "showuseridentity" and the manager role has capability "viewuseridentity" (which is the default).

      • Create a cohort at category level
      • Assign a user as a Manager at this category level
      • Login as this user
      • Go to thew above created category (where this user is manager)
      • Click on cohorts (from settings block)
      • Click on icon to assign members to the cohort

      Both in the potential users list and in the current users list, the manager can not view the user details (i.e. email). However, when assigning roles Category level manager is able to see user email in the selection boxes.

        Gliffy Diagrams

          Activity

          Hide
          Andrea Bicciolo added a comment - - edited

          Screen shot sample taken from user assignment to cohort

          Show
          Andrea Bicciolo added a comment - - edited Screen shot sample taken from user assignment to cohort
          Hide
          Petr Skoda added a comment -

          Thanks for the report.

          Show
          Petr Skoda added a comment - Thanks for the report.
          Hide
          Dan Poltawski added a comment -

          Petr. Please can you create 2.2 fixes too whilst we continue to support this branch.

          Show
          Dan Poltawski added a comment - Petr. Please can you create 2.2 fixes too whilst we continue to support this branch.
          Hide
          Dan Poltawski added a comment -

          Cherry-picked to 22 too.

          Integrated to master, 23 and 22.

          Show
          Dan Poltawski added a comment - Cherry-picked to 22 too. Integrated to master, 23 and 22.
          Hide
          Frédéric Massart added a comment -

          Test successful on 2.2, 2.3 and master. Thanks!

          Show
          Frédéric Massart added a comment - Test successful on 2.2, 2.3 and master. Thanks!
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Many, many thanks for your effort!

          Millions of people will enjoy the results of your work, yay!

          Closing as fixed. Ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Many, many thanks for your effort! Millions of people will enjoy the results of your work, yay! Closing as fixed. Ciao

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: