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

moodle/cohort:assign should allow adding/removing members to the cohort, this doesn't work (icon doesn't show up under edit column) until i allow moodle/cohort:manage

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1, 2.2
    • Fix Version/s: 2.1.2
    • Component/s: Enrolments
    • Labels:
      None
    • Testing Instructions:
      Hide

      1) turn off moodle/cohort:manage but allow moodle/cohort:assign in a role.
      2) that role should be able to edit the members in a cohort.

      Show
      1) turn off moodle/cohort:manage but allow moodle/cohort:assign in a role. 2) that role should be able to edit the members in a cohort.
    • Workaround:
      Hide

      you can still visit the assignment page directly.

      Show
      you can still visit the assignment page directly.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w31_MDL-28468_m22_cohortassign

      Description

      while testing MDL-28432 and the capabilities there, i saw that moodle/cohort:assign being allowed doesn't show the memebers assignment icon under edit column in cohorts management page.
      only turning on moodle/cohort:manage makes that show up.

        Gliffy Diagrams

          Activity

          Hide
          samhemelryk Sam Hemelryk added a comment -

          Good spotting Apu, and thanks for fixing Petr.
          This has been integrated now.

          Show
          samhemelryk Sam Hemelryk added a comment - Good spotting Apu, and thanks for fixing Petr. This has been integrated now.
          Hide
          rwijaya Rossiani Wijaya added a comment -

          Thanks for fixing the issue Petr.

          Test passed.

          Show
          rwijaya Rossiani Wijaya added a comment - Thanks for fixing the issue Petr. Test passed.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          Sent upstream and closing, many thanks!

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - Sent upstream and closing, many thanks!

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                10/Oct/11