Use field partial in admin account show view (#35503)

app/helpers/home_helper.rb

@@ -49,8 +49,8 @@ module HomeHelper
     end
   end
 
-  def custom_field_classes(field)
+  def field_verified_class(verified)
-    if field.verified?
+    if verified
       'verified'
     else
       'emojify'

app/views/admin/accounts/_field.html.haml

@@ -0,0 +1,9 @@
+-# locals: (field:, account:)
+%dl
+  %dt.emojify{ title: field.name }
+    = prerender_custom_emojis(h(field.name), account.emojis)
+  %dd{ title: field.value, class: field_verified_class(field.verified?) }
+    - if field.verified?
+      %span.verified__mark{ title: t('accounts.link_verified_on', date: l(field.verified_at)) }
+        = material_symbol 'check'
+    = prerender_custom_emojis(account_field_value_format(field, with_rel_me: false), account.emojis)

app/views/admin/accounts/show.html.haml

@@ -7,25 +7,17 @@
 
 = render 'application/card', account: @account
 
-- account = @account
+- if @account.fields? || @account.note?
-- fields = account.fields
-- unless fields.empty? && account.note.blank?
   .admin-account-bio
-    - unless fields.empty?
+    - if @account.fields?
       %div
         .account__header__fields
-          - fields.each do |field|
+          = render partial: 'field', collection: @account.fields, locals: { account: @account }
-            %dl
-              %dt.emojify{ title: field.name }= prerender_custom_emojis(h(field.name), account.emojis)
-              %dd{ title: field.value, class: custom_field_classes(field) }
-                - if field.verified?
-                  %span.verified__mark{ title: t('accounts.link_verified_on', date: l(field.verified_at)) }
-                    = material_symbol 'check'
-                = prerender_custom_emojis(account_field_value_format(field, with_rel_me: false), account.emojis)
 
-    - if account.note.present?
+    - if @account.note?
       %div
-        .account__header__content.emojify= prerender_custom_emojis(account_bio_format(account), account.emojis)
+        .account__header__content.emojify
+          = prerender_custom_emojis(account_bio_format(@account), @account.emojis)
 
 = render 'admin/accounts/counters', account: @account
 

spec/helpers/home_helper_spec.rb

@@ -75,23 +75,19 @@ RSpec.describe HomeHelper do
     end
   end
 
-  describe 'custom_field_classes' do
+  describe 'field_verified_class' do
-    context 'with a verified field' do
+    subject { helper.field_verified_class(verified) }
-      let(:field) { instance_double(Account::Field, verified?: true) }
 
-      it 'returns verified string' do
+    context 'with a verified field' do
-        result = helper.custom_field_classes(field)
+      let(:verified) { true }
-        expect(result).to eq 'verified'
+
-      end
+      it { is_expected.to eq('verified') }
     end
 
     context 'with a non-verified field' do
-      let(:field) { instance_double(Account::Field, verified?: false) }
+      let(:verified) { false }
 
-      it 'returns verified string' do
+      it { is_expected.to eq('emojify') }
-        result = helper.custom_field_classes(field)
-        expect(result).to eq 'emojify'
-      end
     end
   end