openedx-learning 0.14.0__tar.gz → 0.16.0__tar.gz

{openedx_learning-0.14.0/openedx_learning.egg-info → openedx_learning-0.16.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: openedx-learning
-Version: 0.14.0
+Version: 0.16.0
 Summary: Open edX Learning Core and Tagging.
 Home-page: https://github.com/openedx/openedx-learning
 Author: David Ormsbee
@@ -18,11 +18,11 @@ 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: djangorestframework<4.0
-Requires-Dist: rules<4.0
+Requires-Dist: attrs
 Requires-Dist: Django<5.0
+Requires-Dist: rules<4.0
 Requires-Dist: edx-drf-extensions
 
 Open edX Learning Core (and Tagging)

{openedx_learning-0.14.0 → openedx_learning-0.16.0}/openedx_learning/__init__.py

@@ -1,4 +1,4 @@
 """
 Open edX Learning ("Learning Core").
 """
-__version__ = "0.14.0"
+__version__ = "0.16.0"

{openedx_learning-0.14.0 → openedx_learning-0.16.0}/openedx_learning/apps/authoring/components/api.py

@@ -12,16 +12,18 @@ are stored in this app.
 """
 from __future__ import annotations
 
-from datetime import datetime
+from datetime import datetime, timezone
 from enum import StrEnum, auto
 from logging import getLogger
 from pathlib import Path
 from uuid import UUID
 
+from django.core.exceptions import ValidationError
 from django.db.models import Q, QuerySet
 from django.db.transaction import atomic
 from django.http.response import HttpResponse, HttpResponseNotFound
 
+from ..collections.models import Collection, CollectionPublishableEntity
 from ..contents import api as contents_api
 from ..publishing import api as publishing_api
 from .models import Component, ComponentType, ComponentVersion, ComponentVersionContent
@@ -48,6 +50,7 @@ __all__ = [
     "look_up_component_version_content",
     "AssetError",
     "get_redirect_response_for_component_asset",
+    "set_collections",
 ]
 
 
@@ -603,3 +606,54 @@ def get_redirect_response_for_component_asset(
     )
 
     return HttpResponse(headers={**info_headers, **redirect_headers})
+
+
+def set_collections(
+    learning_package_id: int,
+    component: Component,
+    collection_qset: QuerySet[Collection],
+    created_by: int | None = None,
+) -> set[Collection]:
+    """
+    Set collections for a given component.
+
+    These Collections must belong to the same LearningPackage as the Component, or a ValidationError will be raised.
+
+    Modified date of all collections related to component is updated.
+
+    Returns the updated collections.
+    """
+    # Disallow adding entities outside the collection's learning package
+    invalid_collection = collection_qset.exclude(learning_package_id=learning_package_id).first()
+    if invalid_collection:
+        raise ValidationError(
+            f"Cannot add collection {invalid_collection.pk} in learning package  "
+            f"{invalid_collection.learning_package_id} to component {component} in "
+            f"learning package {learning_package_id}."
+        )
+    current_relations = CollectionPublishableEntity.objects.filter(
+        entity=component.publishable_entity
+    ).select_related('collection')
+    # Clear other collections for given component and add only new collections from collection_qset
+    removed_collections = set(
+        r.collection for r in current_relations.exclude(collection__in=collection_qset)
+    )
+    new_collections = set(collection_qset.exclude(
+        id__in=current_relations.values_list('collection', flat=True)
+    ))
+    # Use `remove` instead of `CollectionPublishableEntity.delete()` to trigger m2m_changed signal which will handle
+    # updating component index.
+    component.publishable_entity.collections.remove(*removed_collections)
+    component.publishable_entity.collections.add(
+        *new_collections,
+        through_defaults={"created_by_id": created_by},
+    )
+    # Update modified date via update to avoid triggering post_save signal for collections
+    # The signal triggers index update for each collection synchronously which will be very slow in this case.
+    # Instead trigger the index update in the caller function asynchronously.
+    affected_collection = removed_collections | new_collections
+    Collection.objects.filter(
+        id__in=[collection.id for collection in affected_collection]
+    ).update(modified=datetime.now(tz=timezone.utc))
+
+    return affected_collection

{openedx_learning-0.14.0 → openedx_learning-0.16.0/openedx_learning.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: openedx-learning
-Version: 0.14.0
+Version: 0.16.0
 Summary: Open edX Learning Core and Tagging.
 Home-page: https://github.com/openedx/openedx-learning
 Author: David Ormsbee
@@ -18,11 +18,11 @@ 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: djangorestframework<4.0
-Requires-Dist: rules<4.0
+Requires-Dist: attrs
 Requires-Dist: Django<5.0
+Requires-Dist: rules<4.0
 Requires-Dist: edx-drf-extensions
 
 Open edX Learning Core (and Tagging)

{openedx_learning-0.14.0 → openedx_learning-0.16.0}/openedx_learning.egg-info/SOURCES.txt

@@ -106,6 +106,7 @@ openedx_tagging/core/tagging/migrations/0014_minor_fixes.py
 openedx_tagging/core/tagging/migrations/0015_taxonomy_export_id.py
 openedx_tagging/core/tagging/migrations/0016_object_tag_export_id.py
 openedx_tagging/core/tagging/migrations/0017_alter_tagimporttask_status.py
+openedx_tagging/core/tagging/migrations/0018_objecttag_is_copied.py
 openedx_tagging/core/tagging/migrations/__init__.py
 openedx_tagging/core/tagging/models/__init__.py
 openedx_tagging/core/tagging/models/base.py

{openedx_learning-0.14.0 → openedx_learning-0.16.0}/openedx_tagging/core/tagging/api.py

@@ -484,3 +484,35 @@ def delete_tags_from_taxonomy(
     """
     taxonomy = taxonomy.cast()
     taxonomy.delete_tags(tags, with_subtags)
+
+
+def copy_tags(source_object_id: str, dest_object_id: str):
+    """
+    Copy all tags from one object to another.
+
+    This keeps all not-copied tags and delete all
+    previous copied tags of the dest object.
+    If there are not-copied tags that also are in 'source_object_id',
+    then they become copied.
+    """
+    source_object_tags = get_object_tags(
+        source_object_id,
+    )
+    copied_tags = ObjectTag.objects.filter(
+        object_id=dest_object_id,
+        is_copied=True,
+    )
+
+    with transaction.atomic():
+        # Delete all copied tags of destination
+        copied_tags.delete()
+
+        # Copy an create object_tags in destination
+        for object_tag in source_object_tags:
+            ObjectTag.objects.update_or_create(
+                object_id=dest_object_id,
+                taxonomy_id=object_tag.taxonomy_id,
+                tag_id=object_tag.tag_id,
+                defaults={"is_copied": True},
+                # Note: _value and _export_id are set automatically
+            )

openedx_learning-0.16.0/openedx_tagging/core/tagging/migrations/0018_objecttag_is_copied.py

@@ -0,0 +1,18 @@
+# Generated by Django 4.2.16 on 2024-10-04 19:21
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('oel_tagging', '0017_alter_tagimporttask_status'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='objecttag',
+            name='is_copied',
+            field=models.BooleanField(default=False, help_text="True if this object tag has been copied from one object to another using 'copy_tags' api function"),
+        ),
+    ]

{openedx_learning-0.14.0 → openedx_learning-0.16.0}/openedx_tagging/core/tagging/models/base.py

@@ -795,6 +795,13 @@ class ObjectTag(models.Model):
             "Tag associated with this object tag. Provides the tag's 'value' if set."
         ),
     )
+    is_copied = models.BooleanField(
+        default=False,
+        help_text=_(
+            "True if this object tag has been copied from one object to another"
+            " using 'copy_tags' api function"
+        ),
+    )
     _export_id = case_insensitive_char_field(
         max_length=255,
         help_text=_(
@@ -981,6 +988,7 @@ class ObjectTag(models.Model):
         self.tag = object_tag.tag
         self.taxonomy = object_tag.taxonomy
         self.object_id = object_tag.object_id
+        self.is_copied = object_tag.is_copied
         self._value = object_tag._value  # pylint: disable=protected-access
         self._export_id = object_tag._export_id  # pylint: disable=protected-access
         return self

{openedx_learning-0.14.0 → openedx_learning-0.16.0}/openedx_tagging/core/tagging/rest_api/v1/serializers.py

@@ -176,6 +176,9 @@ class ObjectTagsByTaxonomySerializer(UserPermissionsSerializerMixin, serializers
         """
         Convert this list of ObjectTags to the serialized dictionary, grouped by Taxonomy
         """
+        # Allows consumers like edx-platform to override this
+        ObjectTagViewMinimalSerializer = self.context["view"].minimal_serializer_class
+
         can_tag_object_perm = f"{self.app_label}.can_tag_object"
         by_object: dict[str, dict[str, Any]] = {}
         for obj_tag in instance:
@@ -194,7 +197,7 @@ class ObjectTagsByTaxonomySerializer(UserPermissionsSerializerMixin, serializers
                     "export_id": obj_tag.export_id,
                 }
                 taxonomies.append(tax_entry)
-            tax_entry["tags"].append(ObjectTagMinimalSerializer(obj_tag, context=self.context).data)
+            tax_entry["tags"].append(ObjectTagViewMinimalSerializer(obj_tag, context=self.context).data)
         return by_object
 
 

{openedx_learning-0.14.0 → openedx_learning-0.16.0}/openedx_tagging/core/tagging/rest_api/v1/views.py

@@ -37,6 +37,7 @@ from ..utils import view_auth_classes
 from .permissions import ObjectTagObjectPermissions, TaxonomyObjectPermissions, TaxonomyTagsObjectPermissions
 from .serializers import (
     ObjectTagListQueryParamsSerializer,
+    ObjectTagMinimalSerializer,
     ObjectTagsByTaxonomySerializer,
     ObjectTagSerializer,
     ObjectTagUpdateBodySerializer,
@@ -443,7 +444,10 @@ class ObjectTagView(
         * 405 - Method not allowed
     """
 
+    # Serializer used in `get_queryset` when getting tags per taxonomy
     serializer_class = ObjectTagSerializer
+    # Serializer used in the result in `to_representation` in `ObjectTagsByTaxonomySerializer`
+    minimal_serializer_class = ObjectTagMinimalSerializer
     permission_classes = [ObjectTagObjectPermissions]
     lookup_field = "object_id"
 

{openedx_learning-0.14.0 → openedx_learning-0.16.0}/openedx_learning.egg-info/requires.txt

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