From f5c893e34dd1f5c488c1cb09b00090421f6d843b Mon Sep 17 00:00:00 2001 From: Dillon Dumesnil Date: Tue, 4 Aug 2020 11:03:56 -0700 Subject: [PATCH] Fix for enabled_for to not assume the self_paced attr exists Also being defensive on category just in case --- .../show_answer/show_answer_field_override.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/openedx/features/personalized_learner_schedules/show_answer/show_answer_field_override.py b/openedx/features/personalized_learner_schedules/show_answer/show_answer_field_override.py index f216269b3e..f4bb34ab19 100644 --- a/openedx/features/personalized_learner_schedules/show_answer/show_answer_field_override.py +++ b/openedx/features/personalized_learner_schedules/show_answer/show_answer_field_override.py @@ -34,7 +34,7 @@ class ShowAnswerFieldOverride(FieldOverrideProvider): # SHOWANSWER.FINISHED is left on a Course. In that case, we continue # to follow the same mapping of FINISHED -> AFTER_ALL_ATTEMPTS_OR_CORRECT. # This value will then be inherited throughout the rest of the Course. - if not has_showanswer and block.category == 'course': + if not has_showanswer and block and block.category == 'course': return SHOWANSWER.AFTER_ALL_ATTEMPTS_OR_CORRECT elif not has_showanswer: return default @@ -53,4 +53,4 @@ class ShowAnswerFieldOverride(FieldOverrideProvider): @classmethod def enabled_for(cls, course): """ Enabled only for Self-Paced courses using Personalized User Schedules. """ - return course.self_paced and RELATIVE_DATES_FLAG.is_enabled(course.id) + return course and course.self_paced and RELATIVE_DATES_FLAG.is_enabled(course.id)