Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9
    • Fix Version/s: 2.0
    • Component/s: Gradebook
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      At present each module adds own grading settings into mod_edit form, solution of many problems is to standardize this by using new plugin_supports function.
      This will allow us to put all needed grade item settings without the need to synchronize module and gradebook settings.

      We also need a way to synchronise grade item hidden flag with module show option.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              andyjdavis Andrew Davis added a comment -
              Show
              andyjdavis Andrew Davis added a comment - First draft of spec is here. http://docs.moodle.org/en/Development:Grade_settings_modules
              Hide
              andyjdavis Andrew Davis added a comment -

              I've made a lot of additions to the document. I think it more or less spells out the implementation. The changes necessary to the assignment module are described fairly explicitly. The changes that will be required for the quiz module are less clear.

              Show
              andyjdavis Andrew Davis added a comment - I've made a lot of additions to the document. I think it more or less spells out the implementation. The changes necessary to the assignment module are described fairly explicitly. The changes that will be required for the quiz module are less clear.
              Hide
              andyjdavis Andrew Davis added a comment - - edited

              If I'm understanding the task I think the document is more or less done. I have previously not fully understood what I am to do so please speak up if the document isn't correct

              Show
              andyjdavis Andrew Davis added a comment - - edited If I'm understanding the task I think the document is more or less done. I have previously not fully understood what I am to do so please speak up if the document isn't correct
              Hide
              andyjdavis Andrew Davis added a comment - - edited

              I've reshuffled some forms code so that the grades fields are output by moodleform rather than by the modules themselves. A patch is attached for discussion and review.

              Note that I have not yet refactored the assignment.grade column as outlined at http://docs.moodle.org/en/Development:Grade_settings_modules#Module_Enhancements

              Show
              andyjdavis Andrew Davis added a comment - - edited I've reshuffled some forms code so that the grades fields are output by moodleform rather than by the modules themselves. A patch is attached for discussion and review. Note that I have not yet refactored the assignment.grade column as outlined at http://docs.moodle.org/en/Development:Grade_settings_modules#Module_Enhancements
              Hide
              andyjdavis Andrew Davis added a comment -

              Committed some changes. At one point I had standard_coursemodule_elements() call standard_grading_coursemodule_elements(). That meant that individual modules just had to say they supported grades and the grade element would appear automatically.

              The problem was that the module has no control of the order of the settings. If standard_coursemodule_elements() inserts the grade settings then they will always appear appear above "Common module settings". This was particularly a problem for the lesson module. It has some other kind of grade related settings like Display ongoing score and allow retakes that it has titled "Grade options". Having "Grade options" near the top and then "Grade" near the bottom just looked wrong. Alternatively I could rename that block something like "Attempt options"...

              Show
              andyjdavis Andrew Davis added a comment - Committed some changes. At one point I had standard_coursemodule_elements() call standard_grading_coursemodule_elements(). That meant that individual modules just had to say they supported grades and the grade element would appear automatically. The problem was that the module has no control of the order of the settings. If standard_coursemodule_elements() inserts the grade settings then they will always appear appear above "Common module settings". This was particularly a problem for the lesson module. It has some other kind of grade related settings like Display ongoing score and allow retakes that it has titled "Grade options". Having "Grade options" near the top and then "Grade" near the bottom just looked wrong. Alternatively I could rename that block something like "Attempt options"...
              Hide
              andyjdavis Andrew Davis added a comment -

              Splitting the shifting of assignment.grade into a separate tracker issue. MDL-24049

              Show
              andyjdavis Andrew Davis added a comment - Splitting the shifting of assignment.grade into a separate tracker issue. MDL-24049

                People

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

                  Dates

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