Merge pull request #3440 from DMPRoadmap/aaron/bullet-fixes

Address Some Bullet Warnings / Optimise Mean Request Times

Author: aaronskiba

app/controllers/org_admin/plans_controller.rb

@@ -17,7 +17,11 @@ def index
 
       @super_admin = current_user.can_super_admin?
       @clicked_through = params[:click_through].present?
-      @plans = @super_admin ? Plan.all.page(1) : current_user.org.org_admin_plans.page(1)
+      @plans = if @super_admin
+                 Plan.all.page(1).includes(:template, roles: { user: :org })
+               else
+                 current_user.org.org_admin_plans.page(1)
+               end
     end
     # rubocop:enable Metrics/AbcSize
 

app/controllers/public_pages_controller.rb

@@ -85,7 +85,7 @@ def template_export
   # GET /plans_index
   # ------------------------------------------------------------------------------------
   def plan_index
-    @plans = Plan.publicly_visible.includes(:template)
+    @plans = Plan.publicly_visible.includes(:template, roles: { user: :org })
     render 'plan_index', locals: {
       query_params: {
         page: paginable_params.fetch(:page, 1),

app/views/org_admin/plans/index.html.erb

@@ -34,7 +34,7 @@
         </div>
       </div>
     <% end %>
-    <% if @plans.length > 0  %>
+    <% if @plans.any?  %>
       <% unless @super_admin %>
         <%= link_to sanitize(_('Download plans <em class="sr-only">(new window)</em><span class="new-window-popup-info">%{open_in_new_window_text}</span>') %
                            { open_in_new_window_text: _('Opens in new window') },

app/views/super_admin/api_clients/_form.html.erb

@@ -1,7 +1,7 @@
 <%
 url = @api_client.new_record? ? super_admin_api_clients_path : super_admin_api_client_path(@api_client)
 method = @api_client.new_record? ? :post : :put
-orgs = Org.where(is_other: false)
+orgs = Org.where(is_other: false).includes(identifiers: :identifier_scheme)
 %>
 
 <%= form_for @api_client, url: url, method: method,