feat: make downstream copied tags editable when upstream block is del…

…eted

cms/djangoapps/contentstore/signals/handlers.py

@@ -12,10 +12,17 @@
 from django.dispatch import receiver
 from edx_toggles.toggles import SettingToggle
 from opaque_keys.edx.keys import CourseKey
-from openedx_events.content_authoring.data import CourseCatalogData, CourseData, CourseScheduleData, XBlockData
+from openedx_events.content_authoring.data import (
+    CourseCatalogData,
+    CourseData,
+    CourseScheduleData,
+    LibraryBlockData,
+    XBlockData,
+)
 from openedx_events.content_authoring.signals import (
     COURSE_CATALOG_INFO_CHANGED,
     COURSE_IMPORT_COMPLETED,
+    LIBRARY_BLOCK_DELETED,
     XBLOCK_CREATED,
     XBLOCK_DELETED,
     XBLOCK_UPDATED,
@@ -38,7 +45,11 @@
 from xmodule.modulestore.exceptions import ItemNotFoundError
 
 from ..models import PublishableEntityLink
-from ..tasks import create_or_update_upstream_links, handle_create_or_update_xblock_upstream_link
+from ..tasks import (
+    create_or_update_upstream_links,
+    handle_create_or_update_xblock_upstream_link,
+    handle_unlink_upstream_block,
+)
 from .signals import GRADING_POLICY_CHANGED
 
 log = logging.getLogger(__name__)
@@ -287,3 +298,16 @@ def handle_new_course_import(**kwargs):
         force=True,
         replace=True
     )
+
+
+@receiver(LIBRARY_BLOCK_DELETED)
+def unlink_upstream_block_handler(**kwargs):
+    """
+    Handle unlinking the upstream (library) block from any downstream (course) blocks.
+    """
+    library_block = kwargs.get("library_block", None)
+    if not library_block or not isinstance(library_block, LibraryBlockData):
+        log.error("Received null or incorrect data for event")
+        return
+
+    handle_unlink_upstream_block.delay(str(library_block.usage_key))

cms/djangoapps/contentstore/tasks.py

@@ -67,6 +67,7 @@
 from common.djangoapps.util.monitoring import monitor_import_failure
 from openedx.core.djangoapps.content.learning_sequences.api import key_supports_outlines
 from openedx.core.djangoapps.content_libraries import api as v2contentlib_api
+from openedx.core.djangoapps.content_tagging.api import make_copied_tags_editable
 from openedx.core.djangoapps.course_apps.toggles import exams_ida_enabled
 from openedx.core.djangoapps.discussions.config.waffle import ENABLE_NEW_STRUCTURE_DISCUSSIONS
 from openedx.core.djangoapps.discussions.models import DiscussionsConfiguration, Provider
@@ -1466,3 +1467,23 @@ def create_or_update_upstream_links(
     for xblock in xblocks:
         create_or_update_xblock_upstream_link(xblock, course_key_str, created)
     course_status.update_status(LearningContextLinksStatusChoices.COMPLETED)
+
+
+@shared_task
+@set_code_owner_attribute
+def handle_unlink_upstream_block(upstream_usage_key_string: str) -> None:
+    """
+    Handle updates needed to downstream blocks when the upstream link is severed.
+    """
+    ensure_cms("handle_unlink_upstream_block may only be executed in a CMS context")
+
+    try:
+        upstream_usage_key = UsageKey.from_string(upstream_usage_key_string)
+    except (InvalidKeyError):
+        LOGGER.exception(f'Invalid upstream usage_key: {upstream_usage_key_string}')
+        return
+
+    for link in PublishableEntityLink.objects.filter(
+        upstream_usage_key=upstream_usage_key,
+    ):
+        make_copied_tags_editable(str(link.downstream_usage_key))

cms/djangoapps/contentstore/views/tests/test_clipboard_paste.py

@@ -477,6 +477,16 @@ def test_paste_from_library_read_only_tags(self):
             assert object_tag.value in self.lib_block_tags
             assert object_tag.is_copied
 
+        # If we delete the upstream library block...
+        library_api.delete_library_block(self.lib_block_key)
+
+        # ...the copied tags remain, but should no longer be marked as "copied"
+        object_tags = tagging_api.get_object_tags(new_block_key)
+        assert len(object_tags) == len(self.lib_block_tags)
+        for object_tag in object_tags:
+            assert object_tag.value in self.lib_block_tags
+            assert not object_tag.is_copied
+
     def test_paste_from_library_copies_asset(self):
         """
         Assets from a library component copied into a subdir of Files & Uploads.

openedx/core/djangoapps/content_tagging/api.py

@@ -441,3 +441,4 @@ def tag_object(
 get_object_tags = oel_tagging.get_object_tags
 add_tag_to_taxonomy = oel_tagging.add_tag_to_taxonomy
 copy_tags_as_read_only = oel_tagging.copy_tags
+make_copied_tags_editable = oel_tagging.unmark_copied_tags