From 94ac24a7c61a6c95be7a70d5de84c2fd1d9f4df1 Mon Sep 17 00:00:00 2001 From: Adam Stankiewicz Date: Tue, 3 Sep 2024 16:34:06 -0400 Subject: [PATCH] chore: pr feedback --- enterprise_access/apps/content_assignments/admin.py | 1 + enterprise_access/apps/content_assignments/api.py | 12 ++++++------ 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/enterprise_access/apps/content_assignments/admin.py b/enterprise_access/apps/content_assignments/admin.py index a5b4a4e0..fb7f3f65 100644 --- a/enterprise_access/apps/content_assignments/admin.py +++ b/enterprise_access/apps/content_assignments/admin.py @@ -90,6 +90,7 @@ class LearnerContentAssignmentAdmin(DjangoQLSearchMixin, SimpleHistoryAdmin): 'lms_user_id', 'get_enterprise_customer_uuid', 'preferred_course_run_key', + 'parent_content_key', 'is_assigned_course_run', ) autocomplete_fields = ['assignment_configuration'] diff --git a/enterprise_access/apps/content_assignments/api.py b/enterprise_access/apps/content_assignments/api.py index dfc8b385..0de0dfc8 100644 --- a/enterprise_access/apps/content_assignments/api.py +++ b/enterprise_access/apps/content_assignments/api.py @@ -521,8 +521,8 @@ def _get_content_title(assignment_configuration, content_key): """ Helper to retrieve (from cache) the title of a content_key'ed content_metadata """ - content_metadata = _get_content_summary(assignment_configuration, content_key) - return content_metadata.get('content_title') + course_content_metadata = _get_content_summary(assignment_configuration, content_key) + return course_content_metadata.get('content_title') def _get_parent_content_key(assignment_configuration, content_key): @@ -534,8 +534,8 @@ def _get_parent_content_key(assignment_configuration, content_key): If content_key is for a course key, this will return the same key. Otherwise, the content_key represents a course run, and this will return the run's parent course key. """ - content_metadata = _get_content_summary(assignment_configuration, content_key) - metadata_content_key = content_metadata.get('content_key') + course_content_metadata = _get_content_summary(assignment_configuration, content_key) + metadata_content_key = course_content_metadata.get('content_key') # Check if the assignment's content_key matches the returned content_key. If so, this is a course key # which has no parent key. @@ -555,8 +555,8 @@ def _get_preferred_course_run_key(assignment_configuration, content_key): Returns: The preferred course run key (from cache) of a content_key'ed content_metadata """ - content_metadata = _get_content_summary(assignment_configuration, content_key) - return content_metadata.get('course_run_key') + course_content_metadata = _get_content_summary(assignment_configuration, content_key) + return course_content_metadata.get('course_run_key') def _create_new_assignments(