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

Remove setting aggregateincludingsubcategories

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.8
    • Fix Version/s: 2.8
    • Component/s: Gradebook
    • Labels:
    • Testing Instructions:
      Hide

      On a site without this patch, create a course with aggregateincluding subcats enabled.
      Back it up.
      Upgrade to this version.
      Go to the course, verify you are warned about the setting being removed.
      Restore the backup.
      Verify you get the warning in the restored course.
      Verify clicking 'OK' in the warning makes it go away.
      grep the code for aggregatesubcats - the only results should be related to the upgrade warnings.

      Show
      On a site without this patch, create a course with aggregateincluding subcats enabled. Back it up. Upgrade to this version. Go to the course, verify you are warned about the setting being removed. Restore the backup. Verify you get the warning in the restored course. Verify clicking 'OK' in the warning makes it go away. grep the code for aggregatesubcats - the only results should be related to the upgrade warnings.
    • Affected Branches:
      MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_28_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-47503-master

      Description

      This feature is 100% incompatible with natural weighting.

      This was never mentioned in the spec - it was assumed. Note - it can break things when set on a parent grade category if any of the sub categories is natural.

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  10/Nov/14