openedx-learning 0.16.1__tar.gz → 0.16.3__tar.gz

{openedx_learning-0.16.1/openedx_learning.egg-info → openedx_learning-0.16.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: openedx-learning
-Version: 0.16.1
+Version: 0.16.3
 Summary: Open edX Learning Core and Tagging.
 Home-page: https://github.com/openedx/openedx-learning
 Author: David Ormsbee
@@ -18,12 +18,12 @@ Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Requires-Python: >=3.11
 License-File: LICENSE.txt
-Requires-Dist: attrs
-Requires-Dist: celery
-Requires-Dist: rules<4.0
-Requires-Dist: Django<5.0
 Requires-Dist: edx-drf-extensions
+Requires-Dist: Django<5.0
+Requires-Dist: celery
 Requires-Dist: djangorestframework<4.0
+Requires-Dist: attrs
+Requires-Dist: rules<4.0
 
 Open edX Learning Core (and Tagging)
 ====================================

{openedx_learning-0.16.1 → openedx_learning-0.16.3}/openedx_learning/__init__.py

@@ -2,4 +2,4 @@
 Open edX Learning ("Learning Core").
 """
 
-__version__ = "0.16.1"
+__version__ = "0.16.3"

{openedx_learning-0.16.1 → openedx_learning-0.16.3}/openedx_learning/apps/authoring/components/api.py

@@ -12,6 +12,7 @@ are stored in this app.
 """
 from __future__ import annotations
 
+import mimetypes
 from datetime import datetime, timezone
 from enum import StrEnum, auto
 from logging import getLogger
@@ -129,7 +130,7 @@ def create_component_version(
 def create_next_component_version(
     component_pk: int,
     /,
-    content_to_replace: dict[str, int | None],
+    content_to_replace: dict[str, int | None | bytes],
     created: datetime,
     title: str | None = None,
     created_by: int | None = None,
@@ -140,11 +141,14 @@ def create_next_component_version(
     A very common pattern for making a new ComponentVersion is going to be "make
     it just like the last version, except changing these one or two things".
     Before calling this, you should create any new contents via the contents
-    API, since ``content_to_replace`` needs Content IDs for the values.
+    API or send the content bytes as part of ``content_to_replace`` values.
 
     The ``content_to_replace`` dict is a mapping of strings representing the
-    local path/key for a file, to ``Content.id`` values. Using a `None` for
-    a value in this dict means to delete that key in the next version.
+    local path/key for a file, to ``Content.id`` or content bytes values. Using
+    `None` for a value in this dict means to delete that key in the next version.
+
+    Make sure to wrap the function call on a atomic statement:
+    ``with transaction.atomic():``
 
     It is okay to mark entries for deletion that don't exist. For instance, if a
     version has ``a.txt`` and ``b.txt``, sending a ``content_to_replace`` value
@@ -186,11 +190,27 @@ def create_next_component_version(
             component_id=component_pk,
         )
         # First copy the new stuff over...
-        for key, content_pk in content_to_replace.items():
+        for key, content_pk_or_bytes in content_to_replace.items():
             # If the content_pk is None, it means we want to remove the
             # content represented by our key from the next version. Otherwise,
             # we add our key->content_pk mapping to the next version.
-            if content_pk is not None:
+            if content_pk_or_bytes is not None:
+                if isinstance(content_pk_or_bytes, bytes):
+                    file_path, file_content = key, content_pk_or_bytes
+                    media_type_str, _encoding = mimetypes.guess_type(file_path)
+                    # We use "application/octet-stream" as a generic fallback media type, per
+                    # RFC 2046: https://datatracker.ietf.org/doc/html/rfc2046
+                    media_type_str = media_type_str or "application/octet-stream"
+                    media_type = contents_api.get_or_create_media_type(media_type_str)
+                    content = contents_api.get_or_create_file_content(
+                        component.learning_package.id,
+                        media_type.id,
+                        data=file_content,
+                        created=created,
+                    )
+                    content_pk = content.pk
+                else:
+                    content_pk = content_pk_or_bytes
                 ComponentVersionContent.objects.create(
                     content_id=content_pk,
                     component_version=component_version,

{openedx_learning-0.16.1 → openedx_learning-0.16.3}/openedx_learning/apps/authoring/components/management/commands/add_assets_to_component.py

@@ -4,14 +4,11 @@ Management command to add files to a Component.
 This is mostly meant to be a debugging tool to let us to easily load some test
 asset data into the system.
 """
-import mimetypes
 import pathlib
 from datetime import datetime, timezone
 
 from django.core.management.base import BaseCommand
 
-from ....components.api import create_component_version_content
-from ....contents.api import get_or_create_file_content, get_or_create_media_type
 from ....publishing.api import get_learning_package_by_key
 from ...api import create_next_component_version, get_component_by_key
 
@@ -69,39 +66,18 @@ class Command(BaseCommand):
         )
 
         created = datetime.now(tz=timezone.utc)
-        keys_to_remove = set()
-        local_keys_to_content = {}
+        local_keys_to_content_bytes = {}
 
         for file_mapping in file_mappings:
             local_key, file_path = file_mapping.split(":", 1)
 
-            # No file_path means to delete this entry from the next version.
-            if not file_path:
-                keys_to_remove.add(local_key)
-                continue
-
-            media_type_str, _encoding = mimetypes.guess_type(file_path)
-            media_type = get_or_create_media_type(media_type_str)
-            content = get_or_create_file_content(
-                learning_package.id,
-                media_type.id,
-                data=pathlib.Path(file_path).read_bytes(),
-                created=created,
-            )
-            local_keys_to_content[local_key] = content.id
+            local_keys_to_content_bytes[local_key] = pathlib.Path(file_path).read_bytes() if file_path else None
 
         next_version = create_next_component_version(
             component.pk,
-            content_to_replace={local_key: None for local_key in keys_to_remove},
+            content_to_replace=local_keys_to_content_bytes,
             created=created,
         )
-        for local_key, content_id in sorted(local_keys_to_content.items()):
-            create_component_version_content(
-                next_version.pk,
-                content_id,
-                key=local_key,
-                learner_downloadable=True,
-            )
 
         self.stdout.write(
             f"Created v{next_version.version_num} of "

{openedx_learning-0.16.1 → openedx_learning-0.16.3}/openedx_learning/apps/authoring/contents/models.py

@@ -6,6 +6,7 @@ more intelligent data models to be useful.
 from __future__ import annotations
 
 from functools import cache, cached_property
+from logging import getLogger
 
 from django.conf import settings
 from django.core.exceptions import ImproperlyConfigured, ValidationError
@@ -19,6 +20,8 @@ from ....lib.fields import MultiCollationTextField, case_insensitive_char_field,
 from ....lib.managers import WithRelationsManager
 from ..publishing.models import LearningPackage
 
+logger = getLogger()
+
 __all__ = [
     "MediaType",
     "Content",
@@ -316,8 +319,11 @@ class Content(models.Model):
 
         This will return ``None`` if there is no backing file (has_file=False).
         """
-        if self.has_file:
-            return get_storage().path(self.path)
+        try:
+            if self.has_file:
+                return get_storage().path(self.path)
+        except NotImplementedError:
+            logger.warning("Storage backend does not support path()")
         return None
 
     def read_file(self) -> File:

{openedx_learning-0.16.1 → openedx_learning-0.16.3/openedx_learning.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: openedx-learning
-Version: 0.16.1
+Version: 0.16.3
 Summary: Open edX Learning Core and Tagging.
 Home-page: https://github.com/openedx/openedx-learning
 Author: David Ormsbee
@@ -18,12 +18,12 @@ Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Requires-Python: >=3.11
 License-File: LICENSE.txt
-Requires-Dist: attrs
-Requires-Dist: celery
-Requires-Dist: rules<4.0
-Requires-Dist: Django<5.0
 Requires-Dist: edx-drf-extensions
+Requires-Dist: Django<5.0
+Requires-Dist: celery
 Requires-Dist: djangorestframework<4.0
+Requires-Dist: attrs
+Requires-Dist: rules<4.0
 
 Open edX Learning Core (and Tagging)
 ====================================

{openedx_learning-0.16.1 → openedx_learning-0.16.3}/openedx_learning.egg-info/requires.txt

@@ -1,6 +1,6 @@
-attrs
-celery
-rules<4.0
-Django<5.0
 edx-drf-extensions
+Django<5.0
+celery
 djangorestframework<4.0
+attrs
+rules<4.0