Skip to content

Commit f25cc96

Browse files
committed
fix: misses in modulestore_migrator
1 parent d44fb92 commit f25cc96

3 files changed

Lines changed: 7 additions & 7 deletions

File tree

cms/djangoapps/modulestore_migrator/api/read_api.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ def get_migrations(
135135
if source_key:
136136
migrations = migrations.filter(source__key=source_key)
137137
if target_key:
138-
migrations = migrations.filter(target__key=str(target_key))
138+
migrations = migrations.filter(target__package_ref=str(target_key))
139139
if target_collection_slug:
140140
migrations = migrations.filter(target_collection__collection_code=target_collection_slug)
141141
if task_uuid:

cms/djangoapps/modulestore_migrator/rest_api/v1/views.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -540,7 +540,7 @@ def get_queryset(self):
540540
self.request.user,
541541
lib_api.permissions.CAN_VIEW_THIS_CONTENT_LIBRARY
542542
)
543-
queryset = queryset.filter(target__key=library_key, source__key__startswith='course-v1')
543+
queryset = queryset.filter(target__package_ref=str(library_key), source__key__startswith='course-v1')
544544

545545
return queryset
546546

cms/djangoapps/modulestore_migrator/tests/test_tasks.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -89,12 +89,12 @@ def setUp(self):
8989
)
9090
self.collection = Collection.objects.create(
9191
learning_package=self.learning_package,
92-
key="test_collection",
92+
collection_code="test_collection",
9393
title="Test Collection",
9494
)
9595
self.collection2 = Collection.objects.create(
9696
learning_package=self.learning_package,
97-
key="test_collection2",
97+
collection_code="test_collection2",
9898
title="Test Collection 2",
9999
)
100100

@@ -426,7 +426,7 @@ def test_migrate_component_with_static_content(self):
426426
self.assertIsNone(reason) # noqa: PT009
427427

428428
component_media = result.componentversion.componentversionmedia_set.filter(
429-
key="static/test_image.png"
429+
path="static/test_image.png"
430430
).first()
431431
self.assertIsNotNone(component_media) # noqa: PT009
432432
self.assertEqual(component_media.media.id, test_media.id) # noqa: PT009
@@ -673,12 +673,12 @@ def test_migrate_component_content_filename_not_in_olx(self):
673673

674674
referenced_content_exists = (
675675
result.componentversion.componentversionmedia_set.filter(
676-
key="static/referenced.png"
676+
path="static/referenced.png"
677677
).exists()
678678
)
679679
unreferenced_content_exists = (
680680
result.componentversion.componentversionmedia_set.filter(
681-
key="static/unreferenced.png"
681+
path="static/unreferenced.png"
682682
).exists()
683683
)
684684

0 commit comments

Comments
 (0)