fix: take combined_id for competence statistics
This commit is contained in:
parent
1b7b7770ea
commit
0c3be4d383
|
|
@ -57,24 +57,27 @@ def competences(
|
|||
learning_unit = learning_units.get(completion.page.id)
|
||||
circle = circles.get(learning_unit.id)
|
||||
|
||||
competence_records.setdefault(circle.id, {}).setdefault(
|
||||
combined_id = f"{circle.id}-{completion.course_session.id}"
|
||||
|
||||
competence_records.setdefault(combined_id, {}).setdefault(
|
||||
learning_unit,
|
||||
CompetenceRecordStatisticsType(
|
||||
_id=circle.id, # noqa
|
||||
_id=combined_id, # noqa
|
||||
title=learning_unit.title, # noqa
|
||||
course_session_id=completion.course_session.id, # noqa
|
||||
generation=completion.course_session.generation, # noqa
|
||||
circle_id=circle.id, # noqa
|
||||
success_count=0, # noqa
|
||||
fail_count=0, # noqa
|
||||
details_url=f"/course/{course_slug}/cockpit?courseSessionId={completion.course_session.id}", # noqa
|
||||
details_url=f"/course/{course_slug}/cockpit?courseSessionId={completion.course_session.id}",
|
||||
# noqa
|
||||
),
|
||||
)
|
||||
|
||||
if completion.completion_status == CourseCompletionStatus.SUCCESS.value:
|
||||
competence_records[circle.id][learning_unit].success_count += 1
|
||||
competence_records[combined_id][learning_unit].success_count += 1
|
||||
elif completion.completion_status == CourseCompletionStatus.FAIL.value:
|
||||
competence_records[circle.id][learning_unit].fail_count += 1
|
||||
competence_records[combined_id][learning_unit].fail_count += 1
|
||||
|
||||
values = [
|
||||
record
|
||||
|
|
|
|||
Loading…
Reference in New Issue