fix videos

Author: asadali145

src/ol_openedx_uai_content_customization/ol_openedx_uai_content_customization/management/commands/generate_uai_courses.py

@@ -65,6 +65,7 @@
     clone_course_in_modulestore,
     create_content_block,
     delete_course_sections,
+    save_video_block_with_edx_video_id,
 )
 
 log = logging.getLogger(__name__)
@@ -119,8 +120,6 @@ def handle(self, *args, **options):  # noqa: ARG002, PLR0915
         except User.DoesNotExist:
             msg = f"No user found with username {username!r}."
             raise CommandError(msg)  # noqa: B904
-        user_id = user.id
-
         customized_rows, customized_fieldnames = parse_csv(customized_csv)
         video_asset_rows, video_asset_fieldnames = parse_csv(video_assets_csv)
 
@@ -182,7 +181,12 @@ def handle(self, *args, **options):  # noqa: ARG002, PLR0915
 
             try:
                 self._create_course(
-                    source_key, new_key, display_name, videos, video_id_map, user_id
+                    source_key,
+                    new_key,
+                    display_name,
+                    videos,
+                    video_id_map,
+                    user,
                 )
                 created += 1
             except DuplicateCourseError:
@@ -235,7 +239,7 @@ def _validate_source_course_keys(self, course_groups):
             raise CommandError(msg)
 
     def _create_course(  # noqa: PLR0913
-        self, source_key, course_key_str, display_name, videos, video_id_map, user_id
+        self, source_key, course_key_str, display_name, videos, video_id_map, user
     ):
         """
         Clone the source course, strip its sections, then populate with UAI content.
@@ -248,6 +252,7 @@ def _create_course(  # noqa: PLR0913
         All modulestore writes are wrapped in bulk_operations for performance.
         """
         parsed_key = CourseKey.from_string(course_key_str)
+        user_id = user.id
         store = modulestore()
         with store.bulk_operations(parsed_key):
             course = clone_course_in_modulestore(
@@ -295,13 +300,13 @@ def _create_course(  # noqa: PLR0913
                     title,
                     user_id,
                 )
-                create_content_block(
+                video_block = create_content_block(
                     unit,
                     BLOCK_TYPE_VIDEO,
                     title,
                     user_id,
-                    edx_video_id=edx_video_id,
                 )
+                save_video_block_with_edx_video_id(video_block, user, edx_video_id)
 
         store.publish(course.location, user_id)
         if unmapped:

src/ol_openedx_uai_content_customization/ol_openedx_uai_content_customization/modulestore_utils.py

@@ -7,8 +7,12 @@
 
 import logging
 
+from cms.djangoapps.contentstore.xblock_storage_handlers.view_handlers import (
+    save_xblock_with_callback,
+)
 from xmodule.modulestore import ModuleStoreEnum
 from xmodule.modulestore.django import modulestore
+from xmodule.modulestore.inheritance import own_metadata
 
 from ol_openedx_uai_content_customization.constants import (
     BLOCK_TYPE_CHAPTER,
@@ -134,3 +138,41 @@ def create_content_block(parent, block_type, display_name, user_id, **extra_fiel
         )
 
     return block
+
+
+def save_video_block_with_edx_video_id(video_block, user, edx_video_id):
+    """
+    Save a newly-created video block using the CMS callback pipeline.
+
+    This mirrors Studio's save flow and normalizes legacy fallback fields so
+    an explicit ``edx_video_id`` is used as the canonical playback source.
+
+    Args:
+        video_block: Video XBlock descriptor created in modulestore.
+        user: Django user object performing the operation.
+        edx_video_id: VAL/Open edX video identifier to bind to the block.
+
+    Returns:
+        Updated video block descriptor returned by save callback utility.
+    """
+    old_metadata = own_metadata(video_block)
+    video_block.edx_video_id = edx_video_id.strip()
+
+    # Keep legacy fallback metadata empty to avoid default placeholder IDs.
+    for field_name, value in (
+        ("youtube_id_0_75", ""),
+        ("youtube_id_1_0", ""),
+        ("youtube_id_1_25", ""),
+        ("youtube_id_1_5", ""),
+        ("sub", ""),
+        ("source", ""),
+        ("html5_sources", []),
+    ):
+        if hasattr(video_block, field_name):
+            setattr(video_block, field_name, value)
+
+    return save_xblock_with_callback(
+        video_block,
+        user,
+        old_metadata=old_metadata,
+    )