Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Component/s: moodle.org
    • Labels:
      None
    • Rank:
      43860

      Description

      This is similar to MDL-31437, but with some specifics for moodle.org.

      We need a plugin which will:

      • Sync cohorts with course groups across all courses
      • But only add the user to the course group if they are also enrolled in the course

      Example use case:

      • Eloy is a core developer, he is enrolled intwo courses English Using Moodle, and Spanish Using Moodle.
      • Eloy is a member of two system context Cohorts, 'Core Developers' and 'Moodle HQ'.
      • Eloy should be added to core developers and Moodle HQ groups on both the english and spanish course.
      • Eloy SHOULD NOT be enrolled into the Thai using moodle course
      • When eloy enrols onto the Thai using moodle course, he should be added to the 'Core Developers' and 'Moodle HQ' groups.

        Issue Links

          Activity

          Hide
          Dan Poltawski added a comment -

          Assigning this one to you Petr, as I think you've agreed to take this one.

          Show
          Dan Poltawski added a comment - Assigning this one to you Petr, as I think you've agreed to take this one.
          Hide
          Helen Foster added a comment -

          Regarding the example use case, I'm not sure whether we want Particularly Helpful Moodlers to be a cohort. I think it should be a group, as it is now, with the name translated into other languages e.g. Moodlers de gran ayuda in Moodle en Español. Otherwise, multi-talented people like Eloy and Mary will end up with tons of PHM group icons from different courses!

          Show
          Helen Foster added a comment - Regarding the example use case, I'm not sure whether we want Particularly Helpful Moodlers to be a cohort. I think it should be a group, as it is now, with the name translated into other languages e.g. Moodlers de gran ayuda in Moodle en Español. Otherwise, multi-talented people like Eloy and Mary will end up with tons of PHM group icons from different courses!
          Hide
          Dan Poltawski added a comment -

          Helen, they will all end up as groups in the course (in order to show the forum icon).

          But are you saying you don't think the PHM group should be global? I thought that we discussed wanting to encourage that, so that Mary will be identified as a helpful moodler when she first starts posting in the italian course, rather than just a 'nobody'?

          Show
          Dan Poltawski added a comment - Helen, they will all end up as groups in the course (in order to show the forum icon). But are you saying you don't think the PHM group should be global? I thought that we discussed wanting to encourage that, so that Mary will be identified as a helpful moodler when she first starts posting in the italian course, rather than just a 'nobody'?
          Hide
          Helen Foster added a comment -

          I remember we talked about it, but don't remember deciding upon it. I just think lots of PHM group icons will look weird. Also, if someone is a PHM in English say, then starts posting in another course, I don't think it will take them long to become a PHM there too.

          Show
          Helen Foster added a comment - I remember we talked about it, but don't remember deciding upon it. I just think lots of PHM group icons will look weird. Also, if someone is a PHM in English say, then starts posting in another course, I don't think it will take them long to become a PHM there too.
          Hide
          Dan Poltawski added a comment -

          Hi Helen, I updated the description to what I think we are agreed on..

          Show
          Dan Poltawski added a comment - Hi Helen, I updated the description to what I think we are agreed on..
          Hide
          Petr Škoda added a comment -

          Thanks for the description, it is clear to me what is requested. I am going to work on this before the hackfest.

          Show
          Petr Škoda added a comment - Thanks for the description, it is clear to me what is requested. I am going to work on this before the hackfest.
          Hide
          Petr Škoda added a comment -

          Here is the plugin https://github.com/skodak/moodle-enrol_groupsync, it will be probably moved to hq repo later. Thanks for the report.

          Show
          Petr Škoda added a comment - Here is the plugin https://github.com/skodak/moodle-enrol_groupsync , it will be probably moved to hq repo later. Thanks for the report.
          Hide
          Petr Škoda added a comment -

          Reopening and reassigning ,this need to be installed on moodle.org once we switch to 2.4 branch there.

          Show
          Petr Škoda added a comment - Reopening and reassigning ,this need to be installed on moodle.org once we switch to 2.4 branch there.
          Hide
          Martin Dougiamas added a comment -

          Thanks for writing the code, Petr.

          Dan, all yours to test and install on moodle.org.

          Show
          Martin Dougiamas added a comment - Thanks for writing the code, Petr. Dan, all yours to test and install on moodle.org.
          Hide
          Dan Poltawski added a comment -

          Upgrading clone to 2.4, and gonna test this out.

          Show
          Dan Poltawski added a comment - Upgrading clone to 2.4, and gonna test this out.
          Hide
          Dan Poltawski added a comment -

          Looking good right now.

          Show
          Dan Poltawski added a comment - Looking good right now.
          Hide
          Helen Foster added a comment -

          Dan, can we install it on moodle.org, or should I test it more on the clone first?

          Show
          Helen Foster added a comment - Dan, can we install it on moodle.org, or should I test it more on the clone first?
          Hide
          Dan Poltawski added a comment -

          Tested on the clone and now installed on moodle.org

          Show
          Dan Poltawski added a comment - Tested on the clone and now installed on moodle.org
          Hide
          Petr Škoda added a comment -

          Could you please clone my repo to the moodlehq organisation on github? Should this be submitted to the plugins database? Who would be the maintainer there?

          Show
          Petr Škoda added a comment - Could you please clone my repo to the moodlehq organisation on github? Should this be submitted to the plugins database? Who would be the maintainer there?
          Hide
          Helen Foster added a comment -

          Petr, thanks a lot for the cohorts-groups sync plugin which is working perfectly on moodle.org.

          Show
          Helen Foster added a comment - Petr, thanks a lot for the cohorts-groups sync plugin which is working perfectly on moodle.org.
          Hide
          Dan Poltawski added a comment -

          Petr: i've done that and given you access to it.

          Seems like a useful addition to the plugins DB to me - surely you should be the maintainer

          Show
          Dan Poltawski added a comment - Petr: i've done that and given you access to it. Seems like a useful addition to the plugins DB to me - surely you should be the maintainer
          Hide
          Petr Škoda added a comment -

          Here it is https://moodle.org/plugins/view.php?plugin=enrol_groupsync, I needed to bump up the version to make the plugins database happy, I have also created a new branch MOODLE_24_STABLE that should be used by moodle.org server, ciao.

          Show
          Petr Škoda added a comment - Here it is https://moodle.org/plugins/view.php?plugin=enrol_groupsync , I needed to bump up the version to make the plugins database happy, I have also created a new branch MOODLE_24_STABLE that should be used by moodle.org server, ciao.
          Hide
          Dan Poltawski added a comment -

          Thanks Petr, switched to the 24_STABLE branch.

          Show
          Dan Poltawski added a comment - Thanks Petr, switched to the 24_STABLE branch.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development