From 2ec95262ab427a6cd07f264210aecd4cee727077 Mon Sep 17 00:00:00 2001 From: "J. Cliff Dyer" Date: Fri, 30 Sep 2016 10:21:35 -0400 Subject: [PATCH] Handle null weighted_possible values safely. Rather than failing, treat them as unscored blocks. [TNL-5689](https://openedx.atlassian.net/browse/TNL-5689): Server error when checking a problem. --- lms/djangoapps/grades/scores.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/lms/djangoapps/grades/scores.py b/lms/djangoapps/grades/scores.py index 86832ef156..28259b07de 100644 --- a/lms/djangoapps/grades/scores.py +++ b/lms/djangoapps/grades/scores.py @@ -108,8 +108,7 @@ def get_score(submissions_scores, csm_scores, persisted_block, block): _get_score_from_persisted_or_latest_block(persisted_block, block, weight) ) - assert weighted_possible is not None - has_valid_denominator = weighted_possible > 0.0 + has_valid_denominator = weighted_possible is not None and weighted_possible > 0.0 graded = _get_graded_from_block(persisted_block, block) if has_valid_denominator else False return ProblemScore(