Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-29108 META: Advanced grading methods in Moodle 2.2
  3. MDL-29679

DISCUSSION: Method to inject advanced grading into the existing grading form

    XMLWordPrintable

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.2
    • Component/s: Grading methods
    • Labels:
      None
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE

      Description

      Dear collegues

      we will very much appreciate your opinion on the following question.

      Advanced grading provides API for developers to include their grading method into the existing Grade form (moodleform).

      David thinks that this injection should be done as a widget, and each plugin should return a widget (inheriting from the base class)

      I think that each advanced grading plugin should develop their moodleform element.

      Pros & Cons that we see at the moment:
      WIDGET:

      PROS:

      • use renderers

      CONS:

      • without Javascript enabled, widget must open a new window

      FORM ELEMENT:

      PROS:

      • works as well without JavaScript

      CONS:

      • forms do not support renderers
      • when there is an error, the whole element is made red and one error is printed on the top (even if element actually consists of 30 simple elements)

        Attachments

          Activity

            People

            Assignee:
            mudrd8mz David Mudrák (@mudrd8mz)
            Reporter:
            marina Marina Glancy
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              5/Dec/11