Refactor organisation label

app/components/lettings_log_summary_component.html.erb

@@ -33,13 +33,13 @@
           <% if log.owning_organisation %>
             <div class="app-metadata__item">
               <dt class="app-metadata__term">Owned by</dt>
-              <dd class="app-metadata__definition"><%= organisation_label(log.owning_organisation) %></dd>
+              <dd class="app-metadata__definition"><%= log.owning_organisation&.label %></dd>
             </div>
           <% end %>
           <% if log.managing_organisation %>
             <div class="app-metadata__item">
               <dt class="app-metadata__term">Managed by</dt>
-              <dd class="app-metadata__definition"><%= organisation_label(log.managing_organisation) %></dd>
+              <dd class="app-metadata__definition"><%= log.managing_organisation&.label %></dd>
             </div>
           <% end %>
         </dl>

app/components/lettings_log_summary_component.rb

@@ -10,10 +10,4 @@ class LettingsLogSummaryComponent < ViewComponent::Base
   def log_status
     helpers.status_tag(log.status)
   end
-
-  def organisation_label(organisation)
-    return unless organisation
-
-    organisation.status == :deleted ? "#{organisation.name} (deleted)" : organisation.name
-  end
 end

app/components/sales_log_summary_component.html.erb

@@ -26,13 +26,13 @@
           <% if log.owning_organisation %>
             <div class="app-metadata__item">
               <dt class="app-metadata__term">Owned by</dt>
-              <dd class="app-metadata__definition"><%= organisation_label(log.owning_organisation) %></dd>
+              <dd class="app-metadata__definition"><%= log.owning_organisation&.label %></dd>
             </div>
           <% end %>
           <% if log.managing_organisation %>
             <div class="app-metadata__item">
               <dt class="app-metadata__term">Reported by</dt>
-              <dd class="app-metadata__definition"><%= organisation_label(log.managing_organisation) %></dd>
+              <dd class="app-metadata__definition"><%= log.managing_organisation&.label %></dd>
             </div>
           <% end %>
         </dl>

app/components/sales_log_summary_component.rb

@@ -10,10 +10,4 @@ class SalesLogSummaryComponent < ViewComponent::Base
   def log_status
     helpers.status_tag(log.status)
   end
-
-  def organisation_label(organisation)
-    return unless organisation
-
-    organisation.status == :deleted ? "#{organisation.name} (deleted)" : organisation.name
-  end
 end

app/models/organisation.rb

@@ -186,4 +186,8 @@ class Organisation < ApplicationRecord
     users.each(&:discard!)
     update!(discarded_at: Time.zone.now)
   end
+
+  def label
+    status == :deleted ? "#{name} (deleted)" : name
+  end
 end