fix(squash): rename get_container_by_ref to get_container_by_code

Looks up by container_code directly on the Container model instead of
going through publishable_entity__entity_ref.

BREAKING CHANGE: get_container_by_ref() has been renamed to
get_container_by_code() and its parameter changed from entity_ref
to container_code.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: kdmccormick

src/openedx_content/applets/containers/api.py

@@ -55,7 +55,7 @@
     "create_next_container_version",
     "get_container",
     "get_container_version",
-    "get_container_by_ref",
+    "get_container_by_code",
     "get_all_container_subclasses",
     "get_container_subclass",
     "get_container_type_code_of",
@@ -569,22 +569,22 @@ def get_container_version(container_version_pk: int) -> ContainerVersion:
     return ContainerVersion.objects.get(pk=container_version_pk)
 
 
-def get_container_by_ref(learning_package_id: LearningPackage.ID, /, entity_ref: str) -> Container:
+def get_container_by_code(learning_package_id: LearningPackage.ID, /, container_code: str) -> Container:
     """
     [ 🛑 UNSTABLE ]
-    Get a container by its learning package and entity ref.
+    Get a container by its learning package and container code.
 
     Args:
         learning_package_id: The ID of the learning package that contains the container.
-        entity_ref: The entity ref of the container.
+        container_code: The container code of the container.
 
     Returns:
-        The container with the given entity ref (as `Container`, not as its typed subclass).
+        The container with the given container code (as `Container`, not as its typed subclass).
     """
     try:
         return Container.objects.select_related("container_type").get(
-            publishable_entity__learning_package_id=learning_package_id,
-            publishable_entity__entity_ref=entity_ref,
+            learning_package_id=learning_package_id,
+            container_code=container_code,
         )
     except Container.DoesNotExist:
         # Check if it's the container or the learning package that does not exist:

tests/openedx_content/applets/containers/test_api.py

@@ -761,29 +761,29 @@ def test_get_container_version_nonexistent() -> None:
         containers_api.get_container_version(-500)
 
 
-# get_container_by_ref
+# get_container_by_code
 
 
-def test_get_container_by_ref(lp: LearningPackage, parent_of_two: TestContainer) -> None:
+def test_get_container_by_code(lp: LearningPackage, parent_of_two: TestContainer) -> None:
     """
-    Test getting a specific container by entity ref
+    Test getting a specific container by container code.
     """
-    result = containers_api.get_container_by_ref(lp.id, parent_of_two.entity_ref)
+    result = containers_api.get_container_by_code(lp.id, parent_of_two.container_code)
     assert result == parent_of_two.container
     # The API always returns "Container", not specific subclasses like TestContainer:
     assert result.__class__ is Container
 
 
-def test_get_container_by_ref_nonexistent(lp: LearningPackage) -> None:
+def test_get_container_by_code_nonexistent(lp: LearningPackage) -> None:
     """
-    Test getting a specific container by entity ref, where the ref and/or learning package is invalid
+    Test getting a specific container by container code, where the code and/or learning package is invalid.
     """
     FAKE_ID = cast(LearningPackage.ID, -500)
     with pytest.raises(LearningPackage.DoesNotExist):
-        containers_api.get_container_by_ref(FAKE_ID, "invalid-ref")
+        containers_api.get_container_by_code(FAKE_ID, "invalid-code")
 
     with pytest.raises(Container.DoesNotExist):
-        containers_api.get_container_by_ref(lp.id, "invalid-ref")
+        containers_api.get_container_by_code(lp.id, "invalid-code")
 
 
 # get_container_subclass