Moved from Ticket::ScreenOptions.agents to generic User.of_role('Agent').

app/controllers/ticket_overviews_controller.rb

@@ -61,7 +61,7 @@ class TicketOverviewsController < ApplicationController
       group_ids.push group.id
     }
     agents = {}
-    Ticket::ScreenOptions.agents.each { |user|
+    User.of_role('Agent').each { |user|
       agents[ user.id ] = 1
     }
     groups_users = {}

app/models/online_notification.rb

@@ -202,8 +202,7 @@ cleanup old online notifications
     OnlineNotification.where('seen = ? AND created_at < ?', true, Time.zone.now - 1.days).delete_all
 
     # notify all agents
-    users = Ticket::ScreenOptions.agents
+    User.of_role('Agent').each {|user|
-    users.each {|user|
       Sessions.send_to(
         user.id,
         {

app/models/ticket/screen_options.rb

@@ -3,22 +3,6 @@ module Ticket::ScreenOptions
 
 =begin
 
-list of active agents
-
-  result = Ticket::ScreenOptions.agents
-
-returns
-
-  result = [user1, user2]
-
-=end
-
-  def self.agents
-    User.where( active: true ).joins(:roles).where( 'roles.name' => 'Agent', 'roles.active' => true ).uniq()
-  end
-
-=begin
-
 list attributes
 
   result = Ticket::ScreenOptions.attributes_to_change(
@@ -98,7 +82,7 @@ returns
 
     # get group / user relations
     agents = {}
-    Ticket::ScreenOptions.agents.each { |user|
+    User.of_role('Agent').each { |user|
       agents[ user.id ] = 1
     }
 

app/models/user.rb

@@ -416,6 +416,22 @@ returns
     save
   end
 
+=begin
+
+list of active users in role
+
+  result = User.of_role('Agent')
+
+returns
+
+  result = [user1, user2]
+
+=end
+
+  def self.of_role(role)
+    User.where(active: true).joins(:roles).where( 'roles.name' => role, 'roles.active' => true ).uniq()
+  end
+
   private
 
   def cache_delete

lib/sessions/backend/ticket_overview_list.rb

@@ -72,11 +72,11 @@ class Sessions::Backend::TicketOverviewList
 
       # get groups
       group_ids = []
-      Group.where( active: true ).each { |group|
+      Group.where(active: true).each { |group|
         group_ids.push group.id
       }
       agents = {}
-      Ticket::ScreenOptions.agents.each { |user|
+      User.of_role('Agent').each { |user|
         agents[ user.id ] = 1
       }
       users = {}