Gradebook gb_grade_record_t's date_recorded is updated for lower scores when enabling gradebook.externalAssessments.updateSameScore=true

Description

https://sakaiproject.atlassian.net/browse/SAK-19953

gradebook.externalAssessments.updateSameScore=true

Which works great if the score is identical to the previous grade. However, if the grade is lower, the date will also be updated. In our case, for a certification that was re-taken, that was not desired (but I don’t know if other schools would disagree)

Activity

Show:

Austin February 18, 2022 at 1:16 AM
Edited

I’m not sure if this is specific to Samigo, but that’s the tool I was using when I found this issue. Maybe it could happen with other tools.

I think the problem is that when Samigo sends the gradebook the highest points (when “highest score” (default) is selected as the grading option), but not the date. Perhaps if Samigo, or other tools also sent the Date, then the gradebook could use that to compare and set it’s recorded date to that date instead of always setting it to “now” in updateExternalAssessmentScore()

agr.setDateRecorded(now);

or it could use “now” as a fallback in case the date wasn’t passed in.

Details

Priority

Affects versions

Components

Assignee

Reporter

Created February 18, 2022 at 1:10 AM
Updated February 21, 2022 at 8:38 PM