Unify format and indentation for sanitize, i18n and links on emails

In the mail section we have very different indentations and formatting in
 texts with sanitize, links and texts with interpolations. In my opinion it
helps a lot to have clearer indentations in these cases.

This may not be the best way to indent them, but at least I think it is
clearer than it was and at least relatively unified.

app/views/mailer/budget_investment_created.html.erb

@@ -5,20 +5,27 @@
   </h1>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.budget_investment_created.intro",
+    <%= sanitize(
-           author: @investment.author.name)) %>
+      t("mailers.budget_investment_created.intro", author: @investment.author.name)
+    ) %>
   </p>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.budget_investment_created.text",
+    <%= sanitize(
-           investment: @investment.title,
+      t("mailers.budget_investment_created.text",
-           budget: @investment.budget.name)) %>
+        investment: @investment.title,
+        budget: @investment.budget.name)
+    ) %>
   </p>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.budget_investment_created.follow",
+    <%= sanitize(
-           link: link_to(t("mailers.budget_investment_created.follow_link"), budgets_url,
+      t("mailers.budget_investment_created.follow",
-           style: css_for_mailer_link)), attributes: %w[href style]) %>
+        link: link_to(t("mailers.budget_investment_created.follow_link"), budgets_url,
+        style: css_for_mailer_link)
+      ),
+      attributes: %w[href style]
+    ) %>
   </p>
 
   <table style="width: 100%;">

app/views/mailer/budget_investment_unfeasible.html.erb

@@ -13,10 +13,14 @@
   </p>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.budget_investment_unfeasible.new",
+    <%= sanitize(
+      t("mailers.budget_investment_unfeasible.new",
         url: link_to(t("mailers.budget_investment_unfeasible.new_href"),
-        new_budget_investment_url(@investment.budget), style: css_for_mailer_link)),
+                     new_budget_investment_url(@investment.budget),
-        attributes: %w[href style]) %>
+                     style: css_for_mailer_link)
+      ),
+      attributes: %w[href style]
+    ) %>
   </p>
 
   <p style="<%= css_for_mailer_text %>">

app/views/mailer/comment.html.erb

@@ -18,13 +18,16 @@
   </div>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.config.unsubscribe_text",
+    <%= sanitize(
-      notifications: link_to(
+      t("mailers.config.unsubscribe_text",
-        t("mailers.config.notifications_link"),
+        notifications: link_to(
-        edit_subscriptions_url(token: @token),
+          t("mailers.config.notifications_link"),
-        style: css_for_mailer_link
+          edit_subscriptions_url(token: @token),
+          style: css_for_mailer_link
+        ),
+        notification: User.human_attribute_name(:email_on_comment)
       ),
-      notification: User.human_attribute_name(:email_on_comment)
+      attributes: %w[href style]
-    ), attributes: %w[href style]) %>
+    ) %>
   </p>
 </td>

app/views/mailer/direct_message_for_receiver.html.erb

@@ -21,10 +21,14 @@
   </table>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.direct_message_for_receiver.unsubscribe_text",
+    <%= sanitize(
-        notifications: link_to(t("mailers.config.notifications_link"),
+      t("mailers.direct_message_for_receiver.unsubscribe_text",
-                        edit_subscriptions_url(token: @token),
+        notifications: link_to(
-                        style: css_for_mailer_link)),
+          t("mailers.config.notifications_link"),
-                        attributes: %w[href style]) %>
+          edit_subscriptions_url(token: @token),
+          style: css_for_mailer_link)
+      ),
+      attributes: %w[href style]
+    ) %>
   </p>
 </td>

app/views/mailer/direct_message_for_sender.html.erb

@@ -5,8 +5,7 @@
   </h1>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.direct_message_for_sender.text",
+    <%= sanitize(t("mailers.direct_message_for_sender.text", receiver: @direct_message.receiver.name)) %>
-        receiver: @direct_message.receiver.name)) %>
   </p>
 
   <p style="<%= css_for_mailer_text %>">

app/views/mailer/email_verification.html.erb

@@ -9,17 +9,23 @@
   </p>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.email_verification.instructions",
+    <%= sanitize(
-          verification_link: link_to(
+      t("mailers.email_verification.instructions",
-            t("mailers.email_verification.click_here_to_verify"),
+        verification_link: link_to(
-            email_url(email_verification_token: @token), style: css_for_mailer_link)),
+          t("mailers.email_verification.click_here_to_verify"),
-            attributes: %w[href style]) %>
+          email_url(email_verification_token: @token),
+          style: css_for_mailer_link)
+      ),
+      attributes: %w[href style]
+    ) %>
   </p>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.email_verification.instructions_2",
+    <%= sanitize(
-          document_type: humanize_document_type(@document_type),
+      t("mailers.email_verification.instructions_2",
-          document_number: @document_number)) %>
+        document_type: humanize_document_type(@document_type),
+        document_number: @document_number)
+    ) %>
   </p>
 
   <p style="<%= css_for_mailer_text %>">

app/views/mailer/evaluation_comment.html.erb

@@ -9,11 +9,12 @@
   </p>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t(
+    <%= sanitize(
-      "mailers.evaluation_comment.new_comment_by",
+      t("mailers.evaluation_comment.new_comment_by",
-      commenter: @email.comment.author.name,
+        commenter: @email.comment.author.name,
-      investment: valuation_comments_link(@email.commentable)
+        investment: valuation_comments_link(@email.commentable)
-    )) %>
+      )
+    ) %>
   </p>
 
   <p style="<%= css_for_mailer_text %>">

app/views/mailer/machine_learning_error.html.erb

@@ -9,7 +9,8 @@
   </p>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= link_to t("mailers.machine_learning_error.link"), admin_machine_learning_url,
+    <%= link_to t("mailers.machine_learning_error.link"),
+                admin_machine_learning_url,
                 style: "color: #2895F1; text-decoration:none;" %>
   </p>
 </td>

app/views/mailer/machine_learning_success.html.erb

@@ -9,7 +9,8 @@
   </p>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= link_to t("mailers.machine_learning_success.link"), admin_machine_learning_url,
+    <%= link_to t("mailers.machine_learning_success.link"),
+                admin_machine_learning_url,
                 style: "color: #2895F1; text-decoration:none;" %>
   </p>
 </td>

app/views/mailer/newsletter.html.erb

@@ -5,13 +5,16 @@
   </p>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.config.unsubscribe_text",
+    <%= sanitize(
-      notifications: link_to(
+      t("mailers.config.unsubscribe_text",
-        t("mailers.config.notifications_link"),
+        notifications: link_to(
-        edit_subscriptions_url(token: @token),
+          t("mailers.config.notifications_link"),
-        style: css_for_mailer_link
+          edit_subscriptions_url(token: @token),
+          style: css_for_mailer_link
+        ),
+        notification: User.human_attribute_name(:newsletter)
       ),
-      notification: User.human_attribute_name(:newsletter)
+      attributes: %w[href style]
-    ), attributes: %w[href style]) %>
+    ) %>
   </p>
 </td>

app/views/mailer/proposal_notification_digest.html.erb

@@ -43,9 +43,14 @@
                 <tr>
                   <td>
                     <p style="<%= css_for_mailer_text %>">
-                      <%= sanitize(link_to(t("mailers.proposal_notification_digest.unfollow"),
+                      <%= sanitize(
+                        link_to(
+                          t("mailers.proposal_notification_digest.unfollow"),
                           proposal_url(notification.notifiable.proposal),
-                          style: css_for_mailer_link), attributes: %w[href style]) %>
+                          style: css_for_mailer_link
+                        ),
+                        attributes: %w[href style]
+                      ) %>
                     </p>
                   </td>
                 </tr>
@@ -58,10 +63,14 @@
   <% end %>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.proposal_notification_digest.unsubscribe_text",
+    <%= sanitize(
-        notifications: link_to(t("mailers.config.notifications_link"),
+      t("mailers.proposal_notification_digest.unsubscribe_text",
-                        edit_subscriptions_url(token: @token),
+        notifications: link_to(
-                        style: css_for_mailer_link)),
+          t("mailers.config.notifications_link"),
-                        attributes: %w[href style]) %>
+          edit_subscriptions_url(token: @token),
+          style: css_for_mailer_link)
+        ),
+      attributes: %w[href style]
+    ) %>
   </p>
 </td>

app/views/mailer/reply.html.erb

@@ -18,13 +18,16 @@
   </div>
 
   <p style="<%= css_for_mailer_text %>">
-    <%= sanitize(t("mailers.config.unsubscribe_text",
+    <%= sanitize(
-      notifications: link_to(
+      t("mailers.config.unsubscribe_text",
-        t("mailers.config.notifications_link"),
+        notifications: link_to(
-        edit_subscriptions_url(token: @token),
+          t("mailers.config.notifications_link"),
-        style: css_for_mailer_link
+          edit_subscriptions_url(token: @token),
+          style: css_for_mailer_link
+        ),
+        notification: User.human_attribute_name(:email_on_comment_reply)
       ),
-      notification: User.human_attribute_name(:email_on_comment_reply)
+      attributes: %w[href style]
-    ), attributes: %w[href style]) %>
+    ) %>
   </p>
 </td>