Moodle

better dealing with groups on the course level

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9
  • Fix Version/s: 1.9.5
  • Component/s: Groups
  • Labels:
    None
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

the detection is groups are used is not the same at course level - review use of groups on course level
maybe add a switch to enable groups in course and remove all group features from UI if not enabled

Eloy might know more about these issues

Issue Links

Activity

Hide
Petr Škoda (skodak) added a comment -

I have reviewed the code base. The group related course settings are groupmode, groupmodeforce and default grouping - these settings specify default values but also in some places mean current setting at course level.
It is evident that these 3 settings are not enough and we should add some more switches in 2.0. For 1.9.x branch we can at least make sure these settings are respected both as defaults and as current course settings.

Show
Petr Škoda (skodak) added a comment - I have reviewed the code base. The group related course settings are groupmode, groupmodeforce and default grouping - these settings specify default values but also in some places mean current setting at course level. It is evident that these 3 settings are not enough and we should add some more switches in 2.0. For 1.9.x branch we can at least make sure these settings are respected both as defaults and as current course settings.
Hide
Petr Škoda (skodak) added a comment -

mostly finished

Show
Petr Škoda (skodak) added a comment - mostly finished
Hide
Helen Foster added a comment -

Petr, thanks for working on improving groups

Just wondering what needs testing before closing the issue?

Show
Helen Foster added a comment - Petr, thanks for working on improving groups Just wondering what needs testing before closing the issue?

People

Vote (0)
Watch (3)

Dates

  • Created:
    Updated:
    Resolved: