Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-50191

Cohort show/hide button is displayed without manage capability

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 2.8.6, 2.9, 3.0
    • 2.8.7, 2.9.1
    • Cohorts
    • MOODLE_28_STABLE, MOODLE_29_STABLE, MOODLE_30_STABLE
    • MOODLE_28_STABLE, MOODLE_29_STABLE
    • wip-MDL-50191-master
    • Hide
      • Create some cohorts
      • Give user capability cohort/view but do not give him cohort/manage (for example, give this cap to the authenticated user role)
      • login as this user, go to "Cohorts" and make sure you can browse cohorts but you do not see show/hide buttons
      • login as admin or manager, go to Cohorts and make sure you can see show/hide icons for the cohorts
      Show
      Create some cohorts Give user capability cohort/view but do not give him cohort/manage (for example, give this cap to the authenticated user role) login as this user, go to "Cohorts" and make sure you can browse cohorts but you do not see show/hide buttons login as admin or manager, go to Cohorts and make sure you can see show/hide icons for the cohorts

    Description

      If a user is able to only view cohorts but not edit, the show/hide button is still displayed.
      This is not a security issue, rather the visual bug. It is still impossible to change cohort visibility.

      Attachments

        Issue Links

          Activity

            People

              marina Marina Glancy
              marina Marina Glancy
              Simey Lameze Simey Lameze
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Rajesh Taneja Rajesh Taneja
              David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                6/Jul/15