From d76357981b4817f492345beccbbe4c33b77c267f Mon Sep 17 00:00:00 2001 From: 0xflotus <0xflotus@gmail.com> Date: Tue, 29 Oct 2024 19:28:14 +0100 Subject: [PATCH] fix: small bug in course_progress_summary.py This is a small logical fix. Otherwise if `row.progress == 10 or row.progress == 40 or row.progress == 70` wouldn't have an effect. --- .../course_progress_summary/course_progress_summary.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lms/lms/report/course_progress_summary/course_progress_summary.py b/lms/lms/report/course_progress_summary/course_progress_summary.py index cb39e924..604d773b 100644 --- a/lms/lms/report/course_progress_summary/course_progress_summary.py +++ b/lms/lms/report/course_progress_summary/course_progress_summary.py @@ -93,11 +93,11 @@ def get_charts(data): for row in data: if row.progress == 100: completed += 1 - elif row.progress < 100 and row.progress > 70: + elif row.progress < 100 and row.progress >= 70: less_than_hundred += 1 - elif row.progress < 70 and row.progress > 40: + elif row.progress < 70 and row.progress >= 40: less_than_seventy += 1 - elif row.progress < 40 and row.progress > 10: + elif row.progress < 40 and row.progress >= 10: less_than_forty += 1 elif row.progress < 10: less_than_ten += 1