From e301a24a39f911bc94cd337f28d37b8312867732 Mon Sep 17 00:00:00 2001
From: Matt Duran <taosgx2km8fm@opayq.com>
Date: Fri, 17 Oct 2025 15:06:04 -0700
Subject: [PATCH] Fix: Prevent crash when playing non-TubeArchivist media

- Fixed incorrect variable ordering in OnPlaybackProgress causing duplicate
  declaration and wrong hierarchy traversal
- Added type check to reject non-Episode items early
- Prevents 'Guid can't be empty' exception when playing movies

Fixes #60
---
 Jellyfin.Plugin.TubeArchivistMetadata/Plugin.cs | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/Jellyfin.Plugin.TubeArchivistMetadata/Plugin.cs b/Jellyfin.Plugin.TubeArchivistMetadata/Plugin.cs
index 256d3d6..da6929f 100644
--- a/Jellyfin.Plugin.TubeArchivistMetadata/Plugin.cs
+++ b/Jellyfin.Plugin.TubeArchivistMetadata/Plugin.cs
@@ -161,6 +161,12 @@ namespace Jellyfin.Plugin.TubeArchivistMetadata
 
             if (Instance!.Configuration.JFTASync && eventArgs.Users.Any(u => Instance!.Configuration.JFUsernameFrom.Equals(u.Username, StringComparison.Ordinal)))
             {
+                // Only process Episodes (YouTube videos), not Movies or other media types
+                if (eventArgs.Item is not Episode)
+                {
+                    return;
+                }
+
                 BaseItem? season = LibraryManager.GetItemById(eventArgs.Item.ParentId);
                 BaseItem? channel = LibraryManager.GetItemById(season!.ParentId);
                 BaseItem? collection = LibraryManager.GetItemById(channel!.ParentId);