Moodle

Overview Report uses wrong rank for multiple courses

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Won't Fix
  • Affects Version/s: 1.9.2
  • Fix Version/s: None
  • Component/s: Gradebook
  • Labels:
    None
  • Database:
    MySQL
  • Affected Branches:
    MOODLE_19_STABLE

Description

If a student is enrolled in multiple courses, and there are different number of students in each course, then the rank max is chosen based on the course they came from for every course. For example, Bob is a student in Course A and Course B. There are 15 students in A and 20 in B. From A, he navigates to his Overview report which gives his rank out of 15 for A, and his rank out of 15 for B! For B, the max for both would be 20. I have provided a patch that can fix the rank bug.

Activity

Hide
Robert Russo added a comment -

Unfortunately, we found a bug in this fix where the overview report still counted students grades who had been removed from the course. This led to students having ranks of 89/78 (which makes no sense) and 2/78 where they were the highest grade for an enrolled student in the course, but a student who dropped had a higher grade. I changed the SQL query to account for moodle role assignments in the current course.

I also made the same fix for the user report and am creating that bug report as well.

Show
Robert Russo added a comment - Unfortunately, we found a bug in this fix where the overview report still counted students grades who had been removed from the course. This led to students having ranks of 89/78 (which makes no sense) and 2/78 where they were the highest grade for an enrolled student in the course, but a student who dropped had a higher grade. I changed the SQL query to account for moodle role assignments in the current course. I also made the same fix for the user report and am creating that bug report as well.
Hide
Robert Russo added a comment -

Patch for rank problem that is aware of enrolled students instead of all students.

Show
Robert Russo added a comment - Patch for rank problem that is aware of enrolled students instead of all students.
Hide
Michael de Raadt added a comment -

Thanks for reporting this issue.

We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported.

If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed.

Michael d;

lqjjLKA0p6

Show
Michael de Raadt added a comment - Thanks for reporting this issue. We have detected that this issue has been inactive for over a year has been recorded as affecting versions that are no longer supported. If you believe that this issue is still relevant to current versions (2.1 and beyond), please comment on the issue. Issues left inactive for a further month will be closed. Michael d; lqjjLKA0p6
Hide
Michael de Raadt added a comment -

I'm closing this issue as it appears to have become inactive and is probably not relevant to a current supported version. If you are encountering this problem or one similar, please launch a new issue.

Show
Michael de Raadt added a comment - I'm closing this issue as it appears to have become inactive and is probably not relevant to a current supported version. If you are encountering this problem or one similar, please launch a new issue.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: