From df8c858433c7a9f69a636373c4ed3cd72bf8d1f1 Mon Sep 17 00:00:00 2001
From: Brandt Lareau <brandt@lareaus.com>
Date: Mon, 22 Apr 2024 10:11:57 -0700
Subject: [PATCH] Fixed bug with Not finding disk info in DB & Made the GUI a
 bit nicer when it comes to what is going on

commit-id:a50256ab
---
 .../movie_title_table_component.html.erb      | 70 ++++++++++---------
 app/components/movie_title_table_component.rb |  2 +-
 app/services/list_drives_service.rb           |  8 +--
 app/views/movies/show.html.erb                |  3 +
 app/workers/rip_worker.rb                     |  2 +-
 5 files changed, 46 insertions(+), 39 deletions(-)

diff --git a/app/components/movie_title_table_component.html.erb b/app/components/movie_title_table_component.html.erb
index 03399f7a..62dcac90 100644
--- a/app/components/movie_title_table_component.html.erb
+++ b/app/components/movie_title_table_component.html.erb
@@ -1,34 +1,40 @@
-<table class="table" id="<%= dom_id %>">
-  <thead>
-    <tr>
-      <th>#</th>
-      <th>Name</th>
-      <th>Duration</th>
-      <th>Size</th>
-      <th></th>
-    </tr>
-  </thead>
-  <tbody class="disk_titles">
-    <% disks.each do |disk| %>
-      <% disk.disk_titles.sort_by { |d| movie.runtime_range.include?(d.duration) ? 0 : 1  }.each do |disk_title| %>
-        <% text_class = 'text-primary-emphasis' if movie.runtime_range.include?(disk_title.duration) %>
 
-        <tr>
-          <th scope="row" class="<%= text_class %>"><%= disk_title.id %></th>
-          <td class="<%= text_class %>"><%= disk_title.name %></td>
-          <td class="<%= text_class %>"><%= distance_of_time_in_words(disk_title.duration.seconds) %></td>
-          <td class="<%= disk_title.size >= free_disk_space ? 'text-danger' : text_class %>">
-            <%= number_to_human_size(disk_title.size, precision: 3) %>
-            <% if disk_title.size >= free_disk_space %>
-              WARNING: Not enough space available to rip need another <%= number_to_human_size(disk_title.size - free_disk_space, precision: 3) %>
-            <% end %>
-          </td>
-          <td class="<%= text_class %>">
-            <%= link_to 'Rip', video_disk_title_path(movie, disk_title), data: { turbo_method: :patch } %>
-          </td>
-        </tr>
+<% if disks.any? %>
+  <table class="table" id="<%= dom_id %>">
+    <thead>
+      <tr>
+        <th>#</th>
+        <th>Name</th>
+        <th>Duration (<%= distance_of_time_in_words @movie.movie_runtime.seconds %>)</th>
+        <th>Size</th>
+        <th></th>
+      </tr>
+    </thead>
+    <tbody class="disk_titles">
+      <% disks.each do |disk| %>
+        <% disk.disk_titles.sort_by { |d| movie.runtime_range.include?(d.duration) ? 0 : 1  }.each do |disk_title| %>
+          <% text_class = 'text-primary-emphasis' if movie.runtime_range.include?(disk_title.duration) %>
+          <% text_class = 'text-success-emphasis' if movie.disk_title&.name == disk_title.name %>
+          <tr>
+            <th scope="row" class="<%= text_class %>"><%= disk_title.id %></th>
+            <td class="<%= text_class %>"><%= disk_title.name %></td>
+            <td class="<%= text_class %>"><%= distance_of_time_in_words(disk_title.duration.seconds) %></td>
+            <td class="<%= disk_title.size >= free_disk_space ? 'text-danger' : text_class %>">
+              <%= number_to_human_size(disk_title.size, precision: 3) %>
+              <% if disk_title.size >= free_disk_space %>
+                WARNING: Not enough space available to rip need another <%= number_to_human_size(disk_title.size - free_disk_space, precision: 3) %>
+              <% end %>
+            </td>
+            <td class="<%= text_class %>">
+              <%= link_to 'Rip', video_disk_title_path(movie, disk_title), data: { turbo_method: :patch } %>
+            </td>
+          </tr>
+        <% end %>
       <% end %>
-
-    <% end %>
-  </tbody>
-</table>
+    </tbody>
+  </table>
+<% elsif LoadDiskWorker.job.pending? %>
+  <p class="text-info">Loading Disk this might takes a while. Page will reload once ready. Fell free to refresh the page.</p>
+<% else %>
+  <p class="text-warn">Sorry no Disk is currently loaded or could be detected. Refresh page if your seeing this warning but know a disk is present.</p>
+<% end %>
diff --git a/app/components/movie_title_table_component.rb b/app/components/movie_title_table_component.rb
index 71390aee..7cb901ff 100644
--- a/app/components/movie_title_table_component.rb
+++ b/app/components/movie_title_table_component.rb
@@ -3,7 +3,7 @@
 class MovieTitleTableComponent < ViewComponent::Base
   extend Dry::Initializer
 
-  option :disks, Types::Array.of(Types.Instance(Disk))
+  option :disks, Types::Coercible::Array.of(Types.Instance(Disk))
   option :movie, Types.Instance(Movie)
 
   def dom_id
diff --git a/app/services/list_drives_service.rb b/app/services/list_drives_service.rb
index f3a69a63..41a3dd54 100644
--- a/app/services/list_drives_service.rb
+++ b/app/services/list_drives_service.rb
@@ -2,14 +2,12 @@
 
 class ListDrivesService
   extend Dry::Initializer
-  include Shell
-  include MkvParser
 
   option :noscan, Types::Bool, default: -> { false }
 
   class << self
-    def results(*)
-      new(*).results
+    def results(...)
+      new(...).results
     end
   end
 
@@ -32,7 +30,7 @@ def info
   end
 
   def drives
-    @drives ||= parse_mkv_string(info.stdout_str).select do |i|
+    @drives ||= info.parsed_mkv.select do |i|
       i.is_a?(MkvParser::DRV) && i.enabled.to_i.positive?
     end
   end
diff --git a/app/views/movies/show.html.erb b/app/views/movies/show.html.erb
index 0ea4eb60..fb756d25 100644
--- a/app/views/movies/show.html.erb
+++ b/app/views/movies/show.html.erb
@@ -17,6 +17,9 @@
               <%= link_to_movie_db_movie @movie.the_movie_db_id %>
             </small>
           </p>
+          <p class="text-primary-emphasis">Blue Means the disk title is within the runtime range of the movie.</p>
+          <p class="text-success-emphasis">Gream Means the disk has been ripped before.</p>
+
           <div id='disk-selector'>
             <%= render MovieTitleTableComponent.new(disks: @disks, movie: @movie) %>
           </div>
diff --git a/app/workers/rip_worker.rb b/app/workers/rip_worker.rb
index d05fdf7c..a79c4342 100644
--- a/app/workers/rip_worker.rb
+++ b/app/workers/rip_worker.rb
@@ -23,7 +23,7 @@ def create_mkv(disk_title)
 
   def upload_mkv(disk_title)
     sleep 1 while UploadWorker.job.pending?
-    UploadWorker.perform_async(disk_title)
+    UploadWorker.perform_async(disk_title:)
   end
 
   def disk_titles