fix: version_num error because pub_ent was not declared

openedx · Sep 4, 2024 · 7daf487 · 7daf487
1 parent 034e570
commit 7daf487
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/openedx_learning/apps/authoring/publishing/model_mixins.py b/openedx_learning/apps/authoring/publishing/model_mixins.py
@@ -272,6 +272,7 @@ def version_num(self, version_num):
             """
             Return a specific numbered version model.
             """
+            pub_ent = self.content_obj.publishable_entity
             return self.content_version_model_cls.objects.get(
                 publishable_entity_version__entity_id=pub_ent.id,
                 publishable_entity_version__version_num=version_num,

diff --git a/tests/openedx_learning/apps/authoring/components/test_models.py b/tests/openedx_learning/apps/authoring/components/test_models.py
@@ -61,6 +61,9 @@ def test_latest_version(self) -> None:
         # Grabbing the list of versions for this component
         assert list(component.versioning.versions) == [component_version]
 
+        # Grab a specific version by number
+        assert component.versioning.version_num(1) == component_version
+
     def test_last_publish_log(self):
         """
         Test last_publish_log versioning property for published Components.