From c3f672a39968a72ae67dc6ee18caaa5816f891d0 Mon Sep 17 00:00:00 2001 From: 0x29a Date: Fri, 30 Dec 2022 11:13:30 +0100 Subject: [PATCH] refactor: rename module -> block within remaining openedx/features --- openedx/features/course_experience/course_updates.py | 8 ++++---- .../features/course_experience/views/course_updates.py | 6 +++--- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/openedx/features/course_experience/course_updates.py b/openedx/features/course_experience/course_updates.py index 744e090d2b..f25bb864df 100644 --- a/openedx/features/course_experience/course_updates.py +++ b/openedx/features/course_experience/course_updates.py @@ -59,12 +59,12 @@ def get_ordered_updates(request, course): """ Returns all public course updates in reverse chronological order, including dismissed ones. """ - info_module = get_course_info_section_block(request, request.user, course, 'updates') - if not info_module: + info_block = get_course_info_section_block(request, request.user, course, 'updates') + if not info_block: return [] - info_block = getattr(info_module, '_xmodule', info_module) - ordered_updates = [update for update in info_module.items if update.get('status') == STATUS_VISIBLE] + info_block = getattr(info_block, '_xmodule', info_block) + ordered_updates = [update for update in info_block.items if update.get('status') == STATUS_VISIBLE] ordered_updates.sort( key=lambda item: (_safe_parse_date(item['date']), item['id']), reverse=True diff --git a/openedx/features/course_experience/views/course_updates.py b/openedx/features/course_experience/views/course_updates.py index 0f00ccaef4..d0fd50e850 100644 --- a/openedx/features/course_experience/views/course_updates.py +++ b/openedx/features/course_experience/views/course_updates.py @@ -76,8 +76,8 @@ class CourseUpdatesFragmentView(EdxFragmentView): for older implementations and a few tests that store a single html object representing all the updates. """ - info_module = get_course_info_section_block(request, request.user, course, 'updates') - info_block = getattr(info_module, '_xmodule', info_module) + info_block = get_course_info_section_block(request, request.user, course, 'updates') + info_block = getattr(info_block, '_xmodule', info_block) return info_block.system.service( info_block, "replace_urls" - ).replace_urls(info_module.data) if info_module else '' + ).replace_urls(info_block.data) if info_block else ''