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

          Attachments

            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