Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-3245

Grades for courses that a user is not enrolled in aren't displayed properly

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.7.2
    • Fix Version/s: 3.8.0
    • Component/s: Grades add-on
    • Labels:
    • Testing Instructions:
      Hide

      Test in a device of your choice:

      1. In a moodle site, create a course and enroll a student manually.
      2. Disable the manual enrolment method.
      3. Log into the application using the student account.
      4. Visit the Grades overview screen.
      5. All items in the list should be displayed correctly (before the fix, the name of the courses where the enrolment method was modified appeared empty).
      Show
      Test in a device of your choice: In a moodle site, create a course and enroll a student manually. Disable the manual enrolment method. Log into the application using the student account. Visit the Grades overview screen. All items in the list should be displayed correctly (before the fix, the name of the courses where the enrolment method was modified appeared empty).
    • Affected Branches:
      MOODLE_37_STABLE
    • Fixed Branches:
      MOODLE_38_STABLE
    • Pull Master Branch:
    • Sprint:
      Moodle App 3.8.0

      Description

      When a user has grades for a course they're no longer enrolled in, these won't be displayed properly on the grades screen.

      This is happening because two endpoints are being called to retrieve this information. The first one gets all of the user grades, whilst the second one is only retrieving information for enrolled courses.

      This can be fixed by calling the core_course_get_courses_by_field endpoint instead. But this may not be available for old versions. So it should fall back to using the current implementation but cleaning up missing grades.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              noeldemartin Noel De Martin
              Reporter:
              noeldemartin Noel De Martin
              Peer reviewer:
              Dani Palou
              Integrator:
              Juan Leyva
              Tester:
              Isabel Renedo Rouco
              Participants:
              Component watchers:
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                20/Dec/19