From 506a26ac86888cb56eeb59a336e61adb0e051b77 Mon Sep 17 00:00:00 2001
From: Ahmad Farhat <ahmad.af.farhat@gmail.com>
Date: Thu, 3 Jun 2021 17:25:26 -0400
Subject: [PATCH] Fixed issues with inaccessible recording (#2755)

---
 app/views/admins/components/_server_recording_row.html.erb | 4 ++--
 app/views/shared/components/_recording_row.html.erb        | 6 +++---
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/app/views/admins/components/_server_recording_row.html.erb b/app/views/admins/components/_server_recording_row.html.erb
index 6e2362f3..6041191b 100644
--- a/app/views/admins/components/_server_recording_row.html.erb
+++ b/app/views/admins/components/_server_recording_row.html.erb
@@ -42,7 +42,7 @@
     <div class="dropdown">
       <% if recording[:metadata][:"gl-listed"] == "true" %>
         <button class="btn btn-sm btn-secondary dropdown-toggle" data-toggle="dropdown"><i class="dropdown-icon fas fa-globe px-2"></i> <%= t("recording.visibility.public") %></button>
-      <% elsif recording[:state] == "unpublished" %>
+      <% elsif !recording[:published] %>
         <button class="btn btn-sm btn-secondary dropdown-toggle" data-toggle="dropdown"><i class="dropdown-icon fas fa-lock px-2"></i> <%= t("recording.visibility.inaccessible") %></button>
       <% else %>
         <button class="btn btn-sm btn-secondary dropdown-toggle" data-toggle="dropdown"><i class="dropdown-icon fas fa-link px-2"></i> <%= t("recording.visibility.unlisted") %></button>
@@ -61,7 +61,7 @@
     </div>
   </td>
   <td>
-    <% unless recording[:state] == "unpublished" %>
+    <% if recording[:published] %>
       <% sorted_formats = recording[:playbacks].sort_by! { |p| p[:type] } %>
       <% sorted_formats.each do |p| %>
         <%= link_to t("recording.format.#{p[:type]}"), p[:url], class: "btn btn-sm btn-primary", target: "_blank" %>
diff --git a/app/views/shared/components/_recording_row.html.erb b/app/views/shared/components/_recording_row.html.erb
index bd394c66..a6c4cb3c 100644
--- a/app/views/shared/components/_recording_row.html.erb
+++ b/app/views/shared/components/_recording_row.html.erb
@@ -32,7 +32,7 @@
   </td>
   <% if recording_thumbnails? %>
     <td class="overflow-hidden">
-      <% if recording[:state] != "unpublished" %>
+      <% if recording[:published] %>
         <% p = recording[:playbacks].find do |p| p.key?(:preview) end %>
         <% if p %>
           <% safe_recording_images(p[:preview][:images][:image]).each do |img| %>
@@ -52,7 +52,7 @@
     <div class="dropdown">
       <% if recording[:metadata][:"gl-listed"] == "true" %>
         <button class="btn btn-sm btn-secondary dropdown-toggle" data-toggle="dropdown"><i class="dropdown-icon fas fa-globe px-2"></i> <%= t("recording.visibility.public") %></button>
-      <% elsif recording[:state] == "unpublished" %>
+      <% elsif !recording[:published] %>
         <button class="btn btn-sm btn-secondary dropdown-toggle" data-toggle="dropdown"><i class="dropdown-icon fas fa-lock px-2"></i> <%= t("recording.visibility.inaccessible") %></button>
       <% else %>
         <button class="btn btn-sm btn-secondary dropdown-toggle" data-toggle="dropdown"><i class="dropdown-icon fas fa-link px-2"></i> <%= t("recording.visibility.unlisted") %></button>
@@ -71,7 +71,7 @@
     </div>
   </td>
   <td>
-    <% unless recording[:state] == "unpublished" %>
+    <% if recording[:published] %>
       <% sorted_formats = recording[:playbacks].sort_by! { |p| p[:type] } %>
       <% sorted_formats.each do |p| %>
         <%= link_to t("recording.format.#{p[:type]}"), p[:url], class: "btn btn-sm btn-primary", target: "_blank" %>