From 39ad7efcbff85f440e820d9891cde3184ba7ac6d Mon Sep 17 00:00:00 2001 From: bmedx Date: Tue, 26 Dec 2017 19:07:25 -0500 Subject: [PATCH] Add a fix to CourseOverviews::pre_requisite_courses --- .../djangoapps/content/course_overviews/models.py | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/openedx/core/djangoapps/content/course_overviews/models.py b/openedx/core/djangoapps/content/course_overviews/models.py index 7299ead4a6..53520611aa 100644 --- a/openedx/core/djangoapps/content/course_overviews/models.py +++ b/openedx/core/djangoapps/content/course_overviews/models.py @@ -492,6 +492,17 @@ class CourseOverview(TimeStampedModel): """ return json.loads(self._pre_requisite_courses_json) + @pre_requisite_courses.setter + def pre_requisite_courses(self, value): + """ + Django requires there be a setter for this, but it is not + necessary for the way we currently use it. Due to the way + CourseOverviews are constructed raising errors here will + cause a lot of issues. These should not be mutable after + construction, so for now we just eat this. + """ + pass + @classmethod def update_select_courses(cls, course_keys, force_update=False): """