From 52e954cbf256f78dbc314896cabf9bad99a02277 Mon Sep 17 00:00:00 2001 From: "adeel.tajamul" Date: Thu, 27 Jul 2023 15:41:48 +0500 Subject: [PATCH] fix: fixed provider not setting in course discussion settings --- cms/djangoapps/contentstore/utils.py | 5 ++--- cms/djangoapps/contentstore/views/course.py | 2 +- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/cms/djangoapps/contentstore/utils.py b/cms/djangoapps/contentstore/utils.py index 20c6237ed704..6f2835887d11 100644 --- a/cms/djangoapps/contentstore/utils.py +++ b/cms/djangoapps/contentstore/utils.py @@ -972,13 +972,12 @@ def get_subsections_by_assignment_type(course_key): return subsections_by_assignment_type -def update_course_discussions_settings(course_key): +def update_course_discussions_settings(course): """ Updates course provider_type when new course is created """ - provider = DiscussionsConfiguration.get(context_key=course_key).provider_type + provider = DiscussionsConfiguration.get(context_key=course.id).provider_type store = modulestore() - course = store.get_course(course_key) course.discussions_settings['provider_type'] = provider store.update_item(course, course.published_by) diff --git a/cms/djangoapps/contentstore/views/course.py b/cms/djangoapps/contentstore/views/course.py index 7bd2088fbea5..96743daf5116 100644 --- a/cms/djangoapps/contentstore/views/course.py +++ b/cms/djangoapps/contentstore/views/course.py @@ -996,7 +996,7 @@ def create_new_course(user, org, number, run, fields): store_for_new_course = modulestore().default_modulestore.get_modulestore_type() new_course = create_new_course_in_store(store_for_new_course, user, org, number, run, fields) add_organization_course(org_data, new_course.id) - update_course_discussions_settings(new_course.id) + update_course_discussions_settings(new_course) # Enable certain fields rolling forward, where configured if default_enable_flexible_peer_openassessments(new_course.id):