Add metadata change detection to optimize video update process,

To keep things simple, we first create a non-reactive copy of the event object. We then compare this copy with the current event to detect whether any metadata-related fields have changed. Based on this comparison, we pass a flag to the backend, which decides whether the update metadata call needs to be triggered.

Author: ferishili

lib/Routes/Video/VideoUpdate.php

@@ -91,6 +91,13 @@ public function __invoke(Request $request, Response $response, $args)
             'text' => _('Das Video wurde erfolgreich aktualisiert.')
         ];
 
+        // If changes are only related to local event properties, we don't need to update the metadata on the Opencast server.
+        if (isset($event['has_metadata_changes']) && $event['has_metadata_changes'] === false) {
+            return $this->createResponse([
+                'message' => $message,
+            ], $response->withStatus(200));
+        }
+
         $update = $video->updateMetadata($event);
 
         if ($update !== true) {

vueapp/components/Videos/Actions/VideoEdit.vue

@@ -138,7 +138,8 @@ export default {
             showEditDialog: false,
             visibility: null,
             visible_timestamp: null,
-            use_timestamp: false
+            use_timestamp: false,
+            originalEvent: null
         }
     },
 
@@ -181,6 +182,24 @@ export default {
                 };
             }
 
+            // Decide whether the actual metadata has been changed or not, in order to check if the update metadata against opencast server is needed.
+            let has_metadata_changes = false;
+            let non_metadata_fields = ['tags', 'seminar_visibility', 'cid', 'playlist_token']; // These fields are not part of the metadata.
+            for (const field in this.event) {
+                if (non_metadata_fields.includes(field)) {
+                    continue;
+                }
+
+                let event_value = JSON.stringify(this.event[field]);
+                let original_event_value = JSON.stringify(this.originalEvent[field]);
+                if (event_value !== original_event_value) {
+                    has_metadata_changes = true;
+                    break;
+                }
+            }
+
+            this.event.has_metadata_changes = has_metadata_changes;
+
             await this.$store.dispatch('updateVideo', this.event)
             .then(({ data }) => {
                 this.$store.dispatch('addMessage', data.message);
@@ -238,6 +257,8 @@ export default {
         this.visibility = this.event.seminar_visibility?.visibility;
         this.visible_timestamp = this.event.seminar_visibility?.visible_timestamp;
         this.checkVisibility();
+        // Make sure originalEvent is a non-reactive copy of the event.
+        this.originalEvent = JSON.parse(JSON.stringify(this.event));
     }
 }
-</script>
+</script>