Merge pull request #13655 from edx/hotfix-2016-10-06

Break early if max_score is not defined
2016-10-06 14:22:19 -04:00
parent 6043fcb195 7be2093edb
commit 80cbb59fce
1 changed files with 5 additions and 1 deletions
--- a/lms/djangoapps/instructor/enrollment.py
+++ b/lms/djangoapps/instructor/enrollment.py
@@ -317,7 +317,11 @@ def _fire_score_changed_for_block(course_id, student, block, module_state_key):
            field_data_cache=cache,
            course_key=course_id
        )
-        points_earned, points_possible = weighted_score(0, module.max_score(), getattr(module, 'weight', None))
+        max_score = module.max_score()
+        if max_score is None:
+            return
+        else:
+            points_earned, points_possible = weighted_score(0, max_score, getattr(module, 'weight', None))
    else:
        points_earned, points_possible = 0, 0
    SCORE_CHANGED.send(