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

Remove setting aggregateincludingsubcategories

    XMLWordPrintable

Details

    • Task
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.8
    • 2.8
    • Gradebook
    • MOODLE_28_STABLE
    • MOODLE_28_STABLE
    • MDL-47503-master
    • 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.

    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

          is duplicated by

          Sub-task - Issues are sometimes broken into multiple sub-tasks. MDL-46823 Remove option for "Aggregate including subcategories"

          • Minor - Minor loss of function where workaround is possible
          • Closed

          Activity

            People

              damyon Damyon Wiese
              damyon Damyon Wiese
              Andrew Lyons Andrew Lyons
              Marina Glancy Marina Glancy
              Jetha Chan Jetha Chan
              Adrian Greeve, Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                10/Nov/14