Refine data center controller: ensure actions only apply to active users by filtering team.users appropriately.

Author: ger619

app/controllers/data_center_controller.rb

@@ -512,11 +512,11 @@ def daily_report
                   tickets.where.not(statuses: { name: outstanding_statuses })
                 end.order('add_statuses.updated_at DESC')
 
-      hod_emails = team.users.select { |u| u.has_role?(:hod) }.map(&:email)
+      hod_emails = team.users.where(active: true).select { |u| u.has_role?(:hod) }.map(&:email)
       mail_options = {}
       mail_options[:cc] = hod_emails if hod_emails.any?
 
-      team.users.each do |user|
+      team.users.where(active: true).each do |user|
         tagged_tickets = tickets.select('tickets.*, add_statuses.updated_at')
           .joins(:taggings)
           .where(taggings: { user_id: user.id })
@@ -549,7 +549,7 @@ def send_team_ticket_emails
       .where(users: { id: user_ids })
       .where.not(statuses: { name: outstanding_statuses })
 
-    team.users.each do |user|
+    team.users.where(active: true).each do |user|
       user_tickets = base_scope.where(taggings: { user_id: user.id }).distinct
       UserMailer.morning_ticket_email(user, user_tickets.to_a).deliver_later if user_tickets.any?
     end