From 2a86067f5e738f5ff40913bc9924135314079307 Mon Sep 17 00:00:00 2001 From: Matt Jankowski Date: Tue, 18 Apr 2017 11:15:30 -0400 Subject: [PATCH] Simplify renders in views (#2084) * Simplify render in mailer views * Simplify render in about/more view * Simplify render in accounts follow views * Simplify render in recovery code views * Simplify render in stream entry embed * Simplify render in stream entries status partial * Simplify render in authorize follow new view --- app/views/about/more.html.haml | 2 +- app/views/accounts/followers.html.haml | 2 +- app/views/accounts/following.html.haml | 2 +- app/views/accounts/show.html.haml | 2 +- app/views/authorize_follow/new.html.haml | 2 +- app/views/notification_mailer/favourite.text.erb | 2 +- app/views/notification_mailer/mention.text.erb | 2 +- app/views/notification_mailer/reblog.text.erb | 2 +- app/views/settings/two_factor_auths/create.html.haml | 2 +- app/views/settings/two_factor_auths/recovery_codes.html.haml | 2 +- app/views/stream_entries/_status.html.haml | 2 +- app/views/stream_entries/embed.html.haml | 2 +- 12 files changed, 12 insertions(+), 12 deletions(-) diff --git a/app/views/about/more.html.haml b/app/views/about/more.html.haml index 84b29912cf..8f9cd2f6a0 100644 --- a/app/views/about/more.html.haml +++ b/app/views/about/more.html.haml @@ -28,5 +28,5 @@ .panel= @instance_presenter.site_extended_description.html_safe .sidebar - = render partial: 'contact', object: @instance_presenter + = render 'contact', contact: @instance_presenter = render 'links' diff --git a/app/views/accounts/followers.html.haml b/app/views/accounts/followers.html.haml index 4b53aef0cc..9e1643b238 100644 --- a/app/views/accounts/followers.html.haml +++ b/app/views/accounts/followers.html.haml @@ -5,7 +5,7 @@ .accounts-grid - if @followers.empty? - = render partial: 'nothing_here' + = render 'nothing_here' - else = render partial: 'grid_card', collection: @followers, as: :account, cached: true diff --git a/app/views/accounts/following.html.haml b/app/views/accounts/following.html.haml index 4711997d9d..19138b0a4e 100644 --- a/app/views/accounts/following.html.haml +++ b/app/views/accounts/following.html.haml @@ -5,7 +5,7 @@ .accounts-grid - if @following.empty? - = render partial: 'nothing_here' + = render 'nothing_here' - else = render partial: 'grid_card', collection: @following, as: :account, cached: true diff --git a/app/views/accounts/show.html.haml b/app/views/accounts/show.html.haml index b01f3c4e3e..44d66d4716 100644 --- a/app/views/accounts/show.html.haml +++ b/app/views/accounts/show.html.haml @@ -24,7 +24,7 @@ - if @statuses.empty? .accounts-grid - = render partial: 'nothing_here' + = render 'nothing_here' - else .activity-stream = render partial: 'stream_entries/status', collection: @statuses, as: :status diff --git a/app/views/authorize_follow/new.html.haml b/app/views/authorize_follow/new.html.haml index 95601253e6..3b60df0580 100644 --- a/app/views/authorize_follow/new.html.haml +++ b/app/views/authorize_follow/new.html.haml @@ -5,7 +5,7 @@ .follow-prompt %h2= t('authorize_follow.prompt_html', self: current_account.username) - = render partial: 'card', locals: { account: @account } + = render 'card', account: @account = form_tag authorize_follow_path, method: :post, class: 'simple_form' do = hidden_field_tag :acct, @account.acct diff --git a/app/views/notification_mailer/favourite.text.erb b/app/views/notification_mailer/favourite.text.erb index 99852592f6..795045307f 100644 --- a/app/views/notification_mailer/favourite.text.erb +++ b/app/views/notification_mailer/favourite.text.erb @@ -2,4 +2,4 @@ <%= raw t('notification_mailer.favourite.body', name: @account.acct) %> -<%= render partial: 'status', locals: { status: @status } %> +<%= render 'status', status: @status %> diff --git a/app/views/notification_mailer/mention.text.erb b/app/views/notification_mailer/mention.text.erb index c0d4be1d86..b38c5a4d06 100644 --- a/app/views/notification_mailer/mention.text.erb +++ b/app/views/notification_mailer/mention.text.erb @@ -2,4 +2,4 @@ <%= raw t('notification_mailer.mention.body', name: @status.account.acct) %> -<%= render partial: 'status', locals: { status: @status } %> +<%= render 'status', status: @status %> diff --git a/app/views/notification_mailer/reblog.text.erb b/app/views/notification_mailer/reblog.text.erb index c32b486500..fd85437a7d 100644 --- a/app/views/notification_mailer/reblog.text.erb +++ b/app/views/notification_mailer/reblog.text.erb @@ -2,4 +2,4 @@ <%= raw t('notification_mailer.reblog.body', name: @account.acct) %> -<%= render partial: 'status', locals: { status: @status } %> +<%= render 'status', status: @status %> diff --git a/app/views/settings/two_factor_auths/create.html.haml b/app/views/settings/two_factor_auths/create.html.haml index 138a930fd6..aaeebaf2e0 100644 --- a/app/views/settings/two_factor_auths/create.html.haml +++ b/app/views/settings/two_factor_auths/create.html.haml @@ -1,4 +1,4 @@ - content_for :page_title do = t('settings.two_factor_auth') -= render partial: 'recovery_codes', object: @codes += render 'recovery_codes', recovery_codes: @codes diff --git a/app/views/settings/two_factor_auths/recovery_codes.html.haml b/app/views/settings/two_factor_auths/recovery_codes.html.haml index 138a930fd6..aaeebaf2e0 100644 --- a/app/views/settings/two_factor_auths/recovery_codes.html.haml +++ b/app/views/settings/two_factor_auths/recovery_codes.html.haml @@ -1,4 +1,4 @@ - content_for :page_title do = t('settings.two_factor_auth') -= render partial: 'recovery_codes', object: @codes += render 'recovery_codes', recovery_codes: @codes diff --git a/app/views/stream_entries/_status.html.haml b/app/views/stream_entries/_status.html.haml index f389a8dfe7..6c763bde26 100644 --- a/app/views/stream_entries/_status.html.haml +++ b/app/views/stream_entries/_status.html.haml @@ -26,7 +26,7 @@ %strong.emojify= display_name(status.account) = t('stream_entries.reblogged') - = render partial: centered ? 'stream_entries/detailed_status' : 'stream_entries/simple_status', locals: { status: status.proper } + = render (centered ? 'stream_entries/detailed_status' : 'stream_entries/simple_status'), status: status.proper - if include_threads = render partial: 'stream_entries/status', collection: @descendants, as: :status, locals: { is_successor: true, parent_id: status.id} diff --git a/app/views/stream_entries/embed.html.haml b/app/views/stream_entries/embed.html.haml index fd07fdd915..5df82528bd 100644 --- a/app/views/stream_entries/embed.html.haml +++ b/app/views/stream_entries/embed.html.haml @@ -1,2 +1,2 @@ .activity-stream.activity-stream-headless - = render partial: @type, locals: { @type.to_sym => @stream_entry.activity, centered: true } + = render @type, @type.to_sym => @stream_entry.activity, centered: true