From 0e4a74ae832e39743c9f6a1bbedafe3986e804b5 Mon Sep 17 00:00:00 2001 From: 0x29a Date: Fri, 25 Oct 2024 14:36:24 +0200 Subject: [PATCH] fix: missing discussions provider --- cms/djangoapps/contentstore/utils.py | 1 + openedx/core/djangoapps/discussions/tasks.py | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/cms/djangoapps/contentstore/utils.py b/cms/djangoapps/contentstore/utils.py index 780e00fce3b9..c4b798322bf4 100644 --- a/cms/djangoapps/contentstore/utils.py +++ b/cms/djangoapps/contentstore/utils.py @@ -1085,6 +1085,7 @@ def update_course_discussions_settings(course): """ provider = DiscussionsConfiguration.get(context_key=course.id).provider_type store = modulestore() + course.discussions_settings['provider'] = provider course.discussions_settings['provider_type'] = provider store.update_item(course, course.published_by) diff --git a/openedx/core/djangoapps/discussions/tasks.py b/openedx/core/djangoapps/discussions/tasks.py index fea20dc59bd4..19c1f6b7ebc6 100644 --- a/openedx/core/djangoapps/discussions/tasks.py +++ b/openedx/core/djangoapps/discussions/tasks.py @@ -196,7 +196,7 @@ def update_unit_discussion_state_from_discussion_blocks(course_key: CourseKey, u """ store = modulestore() course = store.get_course(course_key) - provider = course.discussions_settings.get('provider', None) + provider = course.discussions_settings.get('provider', None) or course.discussions_settings.get('provider_type', None) # Only migrate to the new discussion provider if the current provider is the legacy provider. log.info(f"Current provider for {course_key} is {provider}") if provider is not None and provider != Provider.LEGACY and not force: