From 41a9e83fb234a5a8250b06555ef887dca98a953a Mon Sep 17 00:00:00 2001
From: Bilal <bilal@saburly.com>
Date: Wed, 26 Aug 2020 14:08:40 +0300
Subject: [PATCH] fix MR comments

---
 app/models/broadcast_recording.rb                   | 4 ++++
 app/views/broadcasts/_broadcast_recordings.html.erb | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/app/models/broadcast_recording.rb b/app/models/broadcast_recording.rb
index a7add7d..679df7e 100644
--- a/app/models/broadcast_recording.rb
+++ b/app/models/broadcast_recording.rb
@@ -18,4 +18,8 @@ class BroadcastRecording < ApplicationRecord
   def download_file_name
     "#{broadcast_name}_Date_#{created_at.in_time_zone(broadcast.shoot_location_time_zone).strftime("%Y-%m-%d")}_Time_#{created_at.in_time_zone(broadcast.shoot_location_time_zone).strftime("%T")}".parameterize
   end
+
+  def thumbnail_url(width = 300)
+    "https://image.mux.com/#{asset_playback_uid}/thumbnail.jpg?width=#{width}"
+  end
 end
diff --git a/app/views/broadcasts/_broadcast_recordings.html.erb b/app/views/broadcasts/_broadcast_recordings.html.erb
index a49b36d..4041849 100644
--- a/app/views/broadcasts/_broadcast_recordings.html.erb
+++ b/app/views/broadcasts/_broadcast_recordings.html.erb
@@ -7,7 +7,7 @@
         <% if (controller.class.module_parent.to_s != "Public" && policy(BroadcastRecording).destroy?) %>
           <%= link_to "Hide", [broadcast.project, broadcast, recording], class: "btn-sm btn-primary ml-1 text-decoration-none", remote: true, method: :delete, data: { confirm: t('.confirm_hide') } %>
         <% end %>
-        <img class="img-thumbnail img-fluid mt-2 mb-2" src="<%= "https://image.mux.com/#{recording.asset_playback_uid}/thumbnail.jpg?width=300" %>" alt="Recording preview">
+        <%= image_tag(recording.thumbnail_url, class: 'img-thumbnail img-fluid mt-2 mb-2') %>
       </li>
     <% end %>
   </ul>