[Dashboard] optimization

Gemfile.base

@@ -166,6 +166,7 @@ gem 'slim-rails', '~> 3.2'
 gem 'ruby-openid'
 
 gem 'draper', '~> 3.0'
+gem 'activerecord-precounter'
 
 group :development do
   gem 'listen'

Gemfile.lock

@@ -153,6 +153,8 @@ GEM
       activerecord (~> 5.0.0)
       arel (~> 7.1.4)
       ruby-plsql (>= 0.5.0)
+    activerecord-precounter (0.3.3)
+      activerecord (>= 5)
     activesupport (5.0.7.2)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 0.7, < 2)
@@ -807,6 +809,7 @@ DEPENDENCIES
   activemerchant (~> 1.77.0)
   activemodel-serializers-xml
   activerecord-oracle_enhanced-adapter (~> 1.7.0)
+  activerecord-precounter
   acts-as-taggable-on (~> 4.0)
   acts_as_list (~> 0.9.17)
   acts_as_tree

Gemfile.prod.lock

@@ -154,6 +154,8 @@ GEM
       activerecord (~> 5.0.0)
       arel (~> 7.1.4)
       ruby-plsql (>= 0.5.0)
+    activerecord-precounter (0.3.3)
+      activerecord (>= 5)
     activesupport (5.0.7.2)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 0.7, < 2)
@@ -808,6 +810,7 @@ DEPENDENCIES
   activemerchant (~> 1.77.0)
   activemodel-serializers-xml
   activerecord-oracle_enhanced-adapter (~> 1.7.0)
+  activerecord-precounter
   acts-as-taggable-on (~> 4.0)
   acts_as_list (~> 0.9.17)
   acts_as_tree

app/controllers/provider/admin/dashboards_controller.rb

@@ -13,7 +13,9 @@ def show
     # .scoped(:include => [ :message => [ :sender ]])
     #
     # but 'Cannot eagerly load the polymorphic association :sender'
-    @services           = current_user.accessible_services
+    @services = current_user.accessible_services.includes(:proxy, :cinstances, :account)
+    ActiveRecord::Precounter.new(@services).precount(:unread_alerts, :api_docs_services)
+
     @messages_presenter = current_presenter
     @unread_messages_presenter = unread_messages_presenter
   end

app/helpers/menu_helper.rb

@@ -85,6 +85,8 @@ def forcibly_denied_switch?(switch)
   end
 
   def api_selector_services
+    return @api_selector_services if defined? @api_selector_services
+
     @api_selector_services ||= (site_account.provider? && logged_in? ? current_user.accessible_services : Service.none).decorate
     @api_selector_services.concat(current_user.account.backend_apis.decorate) if current_account.provider_can_use?(:api_as_product)
     @api_selector_services

app/helpers/provider/admin/dashboards_helper.rb

@@ -40,7 +40,8 @@ def dashboard_secondary_link(link_text, path, options = {})
   end
 
   def dashboard_collection_link(singular_name, collection, path, options = {})
-    link_text = pluralize(number_to_human(collection.size), singular_name, options.fetch(:plural, nil))
+    collection_size = collection.is_a?(Integer) ? collection : collection.size
+    link_text = pluralize(number_to_human(collection_size), singular_name, options.fetch(:plural, nil))
     dashboard_navigation_link(link_text, path, options)
   end
 

app/lib/logic/provider_settings.rb

@@ -29,8 +29,10 @@ def service_items_in_menu?
     end
 
     def multiservice?
+      return @multiservice_flag if defined? @multiservice_flag
+
       account = buyer? ? provider_account : self
-      account.multiple_accessible_services?
+      @multiservice_flag = account.multiple_accessible_services?
     end
 
     def multiple_accessible_services?(scope = nil)

app/models/alert.rb

@@ -12,6 +12,7 @@ class Alert < ApplicationRecord
 
   belongs_to :cinstance
   belongs_to :account
+  belongs_to :service
 
   has_one :user_account, through: :cinstance
 

app/models/service.rb

@@ -110,6 +110,8 @@ def cinstance
   has_many :top_level_metrics, -> { includes(:children).top_level }, class_name: 'Metric'
 
   has_many :service_tokens, inverse_of: :service, dependent: :destroy
+  has_many :alerts, inverse_of: :service
+  has_many :unread_alerts, -> { unread }, class_name: 'Alert', inverse_of: :service
 
   scope :accessible, -> { where.not(state: DELETE_STATE) }
   scope :deleted, -> { where(state: DELETE_STATE) }

app/views/provider/admin/dashboards/_service_navigation.html.slim

@@ -23,11 +23,10 @@ nav.DashboardNavigation
                                        admin_service_applications_path(service),
                                        icon_name: 'cubes'
 
-    - unread_alerts = current_account.buyer_alerts.by_service(service).unread
-    - if can?(:manage, :monitoring) && unread_alerts.any?
+    - if can?(:manage, :monitoring) && service.unread_alerts_count > 0
       li.DashboardNavigation-list-item
         = dashboard_collection_link 'Alert',
-                                    unread_alerts,
+                                    service.unread_alerts_count,
                                     admin_service_alerts_path(service),
                                     icon_name: 'exclamation-triangle',
                                     notice: true
@@ -37,15 +36,15 @@ nav.DashboardNavigation
       li.DashboardNavigation-list-item
         // Subscriptions (would be nice to have API specific links)
         => dashboard_collection_link 'Subscription',
-                                     service.service_contracts,
+                                     service.service_contracts_count,
                                      admin_buyers_service_contracts_path(search: {service_id: service.id}),
                                      icon_name: 'paperclip'
 
     // Active docs
     - if can?(:manage, :plans)
       li.DashboardNavigation-list-item
        = dashboard_collection_link 'ActiveDoc',
-                                    service.api_docs_services,
+                                    service.api_docs_services_count,
                                     admin_service_api_docs_path(service),
                                     icon_name: 'file-text'
     // Integration