refactor: export function renamed

Author: marslanabdulrauf

cms/djangoapps/contentstore/git_export_utils.py

@@ -15,7 +15,7 @@
 from django.utils.translation import gettext_lazy as _
 from opaque_keys.edx.locator import LibraryLocator, LibraryLocatorV2
 
-from openedx.core.djangoapps.content_libraries.api import export_library_v2_to_zip
+from openedx.core.djangoapps.content_libraries.api import export_library_v2_to_dir
 from xmodule.contentstore.django import contentstore
 from xmodule.modulestore.django import modulestore
 from xmodule.modulestore.xml_exporter import export_course_to_xml, export_library_to_xml
@@ -84,16 +84,15 @@ def export_to_git(content_key, repo, user='', rdir=None):
     # pylint: disable=too-many-statements
 
     # Detect content type and select appropriate export function
+    content_type_label = "library"
     is_library_v2 = isinstance(content_key, LibraryLocatorV2)
     if is_library_v2:
         # V2 libraries use backup API with zip extraction
-        export_xml_func = export_library_v2_to_zip
-        content_type_label = "library"
+        content_export_func = export_library_v2_to_dir
     elif isinstance(content_key, LibraryLocator):
-        export_xml_func = export_library_to_xml
-        content_type_label = "library"
+        content_export_func = export_library_to_xml
     else:
-        export_xml_func = export_course_to_xml
+        content_export_func = export_course_to_xml
         content_type_label = "course"
 
     if not GIT_REPO_EXPORT_DIR:
@@ -160,10 +159,10 @@ def export_to_git(content_key, repo, user='', rdir=None):
 
     try:
         if is_library_v2:
-            export_xml_func(content_key, root_dir, content_dir, user)
+            content_export_func(content_key, root_dir, content_dir, user)
         else:
             # V1 libraries and courses: use XML export (no user parameter)
-            export_xml_func(modulestore(), contentstore(), content_key,
+            content_export_func(modulestore(), contentstore(), content_key,
                             root_dir, content_dir)
     except (OSError, AttributeError) as ex:
         log.exception('Failed to export %s', content_type_label)

openedx/core/djangoapps/content_libraries/api/backup.py

@@ -17,7 +17,7 @@
 
 from openedx_content.api import create_zip_file as create_lib_zip_file
 
-__all__ = ["create_library_v2_zip", "export_library_v2_to_zip"]
+__all__ = ["create_library_v2_zip", "export_library_v2_to_dir"]
 
 
 def create_library_v2_zip(library_key: LibraryLocatorV2, user) -> tuple:
@@ -44,17 +44,18 @@ def create_library_v2_zip(library_key: LibraryLocatorV2, user) -> tuple:
     return root_dir, file_path
 
 
-def export_library_v2_to_zip(library_key, root_dir, library_dir, user=None):
+def export_library_v2_to_dir(library_key, root_dir, library_dir, user=None):
     """
-    Export a v2 library using the backup API.
+    Export a v2 library to a directory by creating a zip backup and extracting it.
 
     V2 libraries are stored in Learning Core and use a zip-based backup mechanism.
-    This function creates a zip backup and extracts it to the specified directory.
+    This function creates a temporary zip backup, extracts its contents into
+    ``library_dir`` under ``root_dir``, then cleans up the temporary zip.
 
     Args:
         library_key: LibraryLocatorV2 for the library to export
         root_dir: Root directory where library_dir will be created
-        library_dir: Directory name for the exported library content
+        library_dir: Directory name under root_dir to extract the library into
         user: Username string for the backup API (optional)
 
     Raises:
@@ -65,19 +66,12 @@ def export_library_v2_to_zip(library_key, root_dir, library_dir, user=None):
     temp_dir, zip_path = create_library_v2_zip(library_key, user_obj)
 
     try:
-        # Target directory for extraction
         target_dir = os.path.join(root_dir, library_dir)
-
-        # Create target directory if it doesn't exist
         os.makedirs(target_dir, exist_ok=True)
-
         # Extract zip contents (will overwrite existing files)
         with zipfile.ZipFile(zip_path, 'r') as zip_ref:
             zip_ref.extractall(target_dir)
-
         log.info('Extracted library v2 backup to %s', target_dir)
-
     finally:
-        # Cleanup temporary files
         if temp_dir.exists():
             shutil.rmtree(temp_dir)