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

web service function core_grading_get_definitions has an invalid return value

    XMLWordPrintable

Details

    • MOODLE_27_STABLE, MOODLE_28_STABLE
    • MOODLE_28_STABLE
    • Hide

      The unit tests now include validation of the return value and so this can be tested by commenting out the areaname (line 104 in lib/classes/grading_external.php). The unit tests will fail.

      Line 104 should then be restored and the unit tests run again. The unit tests will now pass.

      Show
      The unit tests now include validation of the return value and so this can be tested by commenting out the areaname (line 104 in lib/classes/grading_external.php). The unit tests will fail. Line 104 should then be restored and the unit tests run again. The unit tests will now pass.

    Description

      The web service function core_grading_get_definitions returns an array of grading_areas and warnings.

      The structure of the grading_area array is defined in the function grading_area(). This defines the field areaname as a required field. However, this field is missing from the array that is created and returned in the function get_definitions().

      Attachments

        Issue Links

          has a non-specific relationship to

          Sub-task - Issues are sometimes broken into multiple sub-tasks. MDL-31681 core_grade_get_definitions

          • Major - Major loss of function, incorrect output
          • Closed

          Activity

            People

              pcharsle Paul Charsley
              pcharsle Paul Charsley
              Damyon Wiese Damyon Wiese
              Marina Glancy Marina Glancy
              Mark Nelson Mark Nelson
              Ilya Tregubov, Kevin Percy, Mathew May, Mihail Geshoski, Shamim Rezaie, Juan Leyva, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                10/Nov/14