refactor: Improve video metadata handling and cluster ID validation in UI components

Author: duartebarbosadev

src/core/metadata_processor.py

@@ -337,6 +337,15 @@ def _init_result_dict() -> Dict[str, Any]:
                     os.path.basename(operational_path),
                 )
                 video_metadata = _build_basic_video_metadata(operational_path)
+                if isinstance(video_metadata, dict):
+                    if (
+                        video_metadata.get("file_path") != cache_key_path
+                        and "operational_path" not in video_metadata
+                    ):
+                        video_metadata["operational_path"] = video_metadata.get(
+                            "file_path"
+                        )
+                    video_metadata["file_path"] = cache_key_path
                 results[cache_key_path]["raw_metadata"] = video_metadata
                 if exif_disk_cache:
                     exif_disk_cache.set(cache_key_path, video_metadata)

src/core/similarity_utils.py

@@ -35,7 +35,7 @@ def _get_raw_dimensions(image_path: str) -> Tuple[int, int] | None:
                 getattr(sizes, "height", 0) or getattr(sizes, "iheight", 0) or 0
             )
             flip = int(getattr(sizes, "flip", 0) or 0)
-            if flip in {5, 6}:
+            if flip in {5, 6, 7, 8}:
                 width, height = height, width
 
             if width > 0 and height > 0:

src/ui/ui_components.py

@@ -120,7 +120,10 @@ def _is_cluster_drop_valid(self, event) -> bool:
             return False
 
         item_data = item.data(Qt.ItemDataRole.UserRole)
-        return isinstance(item_data, str) and item_data.startswith("cluster_header_")
+        if not (isinstance(item_data, str) and item_data.startswith("cluster_header_")):
+            return False
+
+        return self._get_cluster_id_from_index(proxy_index) is not None
 
     def _get_cluster_id_from_index(self, proxy_index) -> Optional[int]:
         """Extract cluster ID from a cluster header item."""