From 438f900bfc904c098c43b435f71074a59fe10bda Mon Sep 17 00:00:00 2001 From: Paulo Viadanna Date: Wed, 4 Sep 2024 09:24:34 -0300 Subject: [PATCH] refactor: improve code performance --- openedx/core/djangoapps/discussions/serializers.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/openedx/core/djangoapps/discussions/serializers.py b/openedx/core/djangoapps/discussions/serializers.py index b5fc828a08ea..6cc18605df9f 100644 --- a/openedx/core/djangoapps/discussions/serializers.py +++ b/openedx/core/djangoapps/discussions/serializers.py @@ -265,14 +265,6 @@ def update(self, instance: DiscussionsConfiguration, validated_data: dict) -> Di course_overview_id=instance.context_key, type='discussion' ).update(is_hidden=not instance.enabled) - # do the same for the modulestore representation - store = modulestore() - course = store.get_course(instance.context_key) - for tab in course.tabs: - if tab.tab_id == 'discussion': - tab.is_hidden = not instance.enabled - store.update_item(course, self.context['user_id']) - break update_discussions_settings_from_course_task.delay(str(instance.context_key)) return instance