From 58201d57636ab9aaec58242fe1237e77fb1e5194 Mon Sep 17 00:00:00 2001
From: farhatahmad <35435341+farhatahmad@users.noreply.github.com>
Date: Wed, 31 Jul 2019 11:33:06 -0400
Subject: [PATCH] Email recording now includes all urls (#697)

---
 app/assets/javascripts/recording.js                        | 2 +-
 app/views/admins/components/_server_recording_row.html.erb | 5 +++--
 app/views/shared/components/_recording_row.html.erb        | 5 +++--
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/app/assets/javascripts/recording.js b/app/assets/javascripts/recording.js
index 213ca957..12b446dc 100644
--- a/app/assets/javascripts/recording.js
+++ b/app/assets/javascripts/recording.js
@@ -28,7 +28,7 @@ $(document).on('turbolinks:load', function(){
       $(this).click(function(){
         var subject = $(".username").text() + " " + t('room.mailer.subject');
         var body =  t('room.mailer.body') + "\n\n" + $(this).attr("data-pres-link");
-        var autogenerated = "\n\n" + t('room.mailer.autogenerated') + "\n";
+        var autogenerated = t('room.mailer.autogenerated') + "\n";
         var footer = t('room.mailer.footer');
 
         var url = "mailto:?subject=" + encodeURIComponent(subject) + "&body=" + encodeURIComponent(body) + encodeURIComponent(autogenerated) + encodeURIComponent(footer);
diff --git a/app/views/admins/components/_server_recording_row.html.erb b/app/views/admins/components/_server_recording_row.html.erb
index c6234755..1bfd3767 100644
--- a/app/views/admins/components/_server_recording_row.html.erb
+++ b/app/views/admins/components/_server_recording_row.html.erb
@@ -67,9 +67,10 @@
         <i class="fas fa-ellipsis-v px-4"></i>
       </a>
       <div class="dropdown-menu dropdown-menu-right">
-        <% p = recording[:playbacks].find do |p| p.key?(:length) end %>
+        <% recording_links = "" %>
+        <% p = recording[:playbacks].each do |p| recording_links.concat(p[:url] + "\n\n") if p.key?(:length) end %>
         <% if p %>
-          <a class="dropdown-item email-link" data-pres-link="<%= p[:url] %>"><i class="dropdown-icon far fa-envelope"></i> <%= t("recording.email") %></a>
+          <a class="dropdown-item email-link" data-pres-link="<%= recording_links %>"><i class="dropdown-icon far fa-envelope"></i> <%= t("recording.email") %></a>
           <div class="dropdown-divider"></div>
         <% end %>
         <%= button_to delete_recording_path(meetingID: recording[:meetingID], record_id: recording[:recordID]), method: :delete, class: "dropdown-item" do %>
diff --git a/app/views/shared/components/_recording_row.html.erb b/app/views/shared/components/_recording_row.html.erb
index 1d298e19..0a65bed2 100644
--- a/app/views/shared/components/_recording_row.html.erb
+++ b/app/views/shared/components/_recording_row.html.erb
@@ -74,9 +74,10 @@
         <i class="fas fa-ellipsis-v px-4"></i>
       </a>
       <div class="dropdown-menu dropdown-menu-right">
-        <% p = recording[:playbacks].find do |p| p.key?(:length) end %>
+        <% recording_links = "" %>
+        <% p = recording[:playbacks].each do |p| recording_links.concat(p[:url] + "\n\n") if p.key?(:length) end %>
         <% if p %>
-          <a class="dropdown-item email-link" data-pres-link="<%= p[:url] %>"><i class="dropdown-icon far fa-envelope"></i> <%= t("recording.email") %></a>
+          <a class="dropdown-item email-link" data-pres-link="<%= recording_links %>"><i class="dropdown-icon far fa-envelope"></i> <%= t("recording.email") %></a>
           <div class="dropdown-divider"></div>
         <% end %>
         <%= button_to delete_recording_path(meetingID: recording[:meetingID], record_id: recording[:recordID]), method: :delete, class: "dropdown-item" do %>