Change function to read more clearly

app/models/scheme.rb

@@ -295,8 +295,8 @@ class Scheme < ApplicationRecord
     status == :active
   end
 
-  def all_locations_inactive?
+  def has_active_locations?
-    active? && locations.all? { |location| !location.active? }
+    active? && locations.active.exists?
   end
 
   def any_location_active_on_date?(date)

app/views/schemes/_scheme_list.html.erb

@@ -28,7 +28,7 @@
           <% row.with_cell(text: scheme.locations.visible&.count) %>
           <% row.with_cell do %>
             <%= status_tag_from_resource(scheme) %>
-            <% if scheme.all_locations_inactive? %>
+            <% unless scheme.has_active_locations? %>
               <%= content_tag(:div, "No currently active locations", class: "app-!-colour-muted", style: "margin-top: 10px") %>
             <% end %>
           <% end %>

app/views/schemes/show.html.erb

@@ -26,7 +26,7 @@
             <dt class="govuk-summary-list__key">Status</dt>
             <dd class="govuk-summary-list__value">
               <%= details_html({ name: "Status", value: status_tag_from_resource(@scheme), id: "status" }) %>
-              <% if @scheme.confirmed? && @scheme.all_locations_inactive? %>
+              <% if @scheme.confirmed? && !@scheme.has_active_locations? %>
                 <span class="app-!-colour-muted">No currently active locations</span>
               <% end %>
               <% if @scheme.confirmed? && @scheme.locations.confirmed.none? && LocationPolicy.new(current_user, @scheme.locations.new).create? %>