Add a route for a shared agents table

Conflicts: app/controllers/agents_controller.rb app/views/agents/_table.html.erb app/views/agents/all.html.erb
huginn · Aug 31, 2016 · 5b157cf · 5b157cf
1 parent ce9395d
commit 5b157cf
Show file tree

Hide file tree

Showing 8 changed files with 152 additions and 5 deletions.
diff --git a/app/controllers/agents_controller.rb b/app/controllers/agents_controller.rb
@@ -7,6 +7,22 @@ def index
     set_table_sort sorts: %w[name created_at last_check_at last_event_at last_receive_at], default: { created_at: :desc }
 
     @agents = current_user.agents.preload(:scenarios, :controllers).reorder(table_sort).page(params[:page])
+    @shared_agents = Agent.shared - @agents
+
+    if show_only_enabled_agents?
+      @agents = @agents.where(disabled: false)
+    end
+
+    respond_to do |format|
+      format.html
+      format.json { render json: @agents }
+    end
+  end
+
+  def shared
+    set_table_sort sorts: %w[name users.username created_at last_check_at last_event_at last_receive_at], default: { created_at: :desc }
+
+    @agents = Agent.shared.preload(:scenarios, :controllers).reorder(table_sort).page(params[:page])
 
     if show_only_enabled_agents?
       @agents = @agents.where(disabled: false)

diff --git a/app/views/agents/_shared_table.html.erb b/app/views/agents/_shared_table.html.erb
@@ -0,0 +1,103 @@
+<div class='table-responsive'>
+  <table class='table table-striped'>
+    <tr>
+      <th></th>
+      <th><%= sortable_column 'name', 'asc' %></th>
+      <th><%= sortable_column 'created_at', 'desc', name: 'Age' %></th>
+      <th>Schedule</th>
+      <th><%= sortable_column 'last_check_at', name: 'Last Check' %></th>
+      <th><%= sortable_column 'last_event_at', name: 'Last Event Out' %></th>
+      <th><%= sortable_column 'last_receive_at', name: 'Last Event In' %></th>
+      <th>Events Created</th>
+      <th>Working?</th>
+      <th><%= sortable_column 'users.username', 'asc', name: 'User' %></th>
+      <th></th>
+    </tr>
+
+    <% agents.each do |agent| %>
+      <tr>
+        <td class='<%= "agent-unavailable" if agent.unavailable? %>'>
+          <%= agent_type_icon(agent, agents) %>
+        </td>
+        <td class='<%= "agent-unavailable" if agent.unavailable? %>'>
+          <% if current_user == agent.user %>
+          <%= link_to agent.name, agent_path(agent, return: (defined?(return_to) && return_to) || request.path) %>
+          <% else %>
+          <span><%= agent.name %></span>
+          <% end %>
+          <br/>
+          <span class='text-muted'><%= agent.short_type.titleize %></span>
+          <% if current_user == agent.user %>
+          <% if agent.scenarios.present? %>
+            <span>
+              <%= scenario_links(agent) %>
+            </span>
+          <% end %>
+          <% end %>
+        </td>
+        <td class='<%= "agent-unavailable" if agent.unavailable? %>'>
+          <%= time_ago_in_words agent.created_at %>
+        </td>
+        <td class='<%= "agent-unavailable" if agent.unavailable? %>'>
+          <% if agent.can_be_scheduled? %>
+            <%= agent_schedule(agent, ',<br/>') %>
+          <% else %>
+            <span class='not-applicable'></span>
+          <% end %>
+        </td>
+        <td class='<%= "agent-unavailable" if agent.unavailable? %>'>
+          <% if agent.can_be_scheduled? %>
+            <%= agent.last_check_at ? time_ago_in_words(agent.last_check_at) + " ago" : "never" %>
+          <% else %>
+            <span class='not-applicable'></span>
+          <% end %>
+        </td>
+        <td class='<%= "agent-unavailable" if agent.unavailable? %>'>
+          <% if agent.can_create_events? %>
+            <%= agent.last_event_at ? time_ago_in_words(agent.last_event_at) + " ago" : "never" %>
+          <% else %>
+            <span class='not-applicable'></span>
+          <% end %>
+        </td>
+        <td class='<%= "agent-unavailable" if agent.unavailable? %>'>
+          <% if agent.can_receive_events? %>
+            <%= agent.last_receive_at ? time_ago_in_words(agent.last_receive_at) + " ago" : "never" %>
+          <% else %>
+            <span class='not-applicable'></span>
+          <% end %>
+        </td>
+        <td class='<%= "agent-unavailable" if agent.unavailable? %>'>
+          <% if agent.can_create_events? %>
+            <%= link_to(agent.events_count || 0, agent_events_path(agent, return: (defined?(return_to) && return_to) || request.path)) %>
+          <% else %>
+            <span class='not-applicable'></span>
+          <% end %>
+        </td>
+        <td><%= working(agent) %></td>
+        <td class='<%= "agent-unavailable" if agent.unavailable? %>'>
+          <% if current_user == agent.user %>
+            <span><%= agent.user.username %></span>
+          <% else %>
+            <%= link_to agent.user.username, edit_admin_user_path(agent.user, return: (defined?(return_to) && return_to) || request.path) %>
+          <% end %>
+        </td>
+        <td>
+          <% if current_user == agent.user %>
+            <div class="btn-group">
+              <button type="button" class="btn btn-default btn-sm dropdown-toggle" data-toggle="dropdown">
+                <span class="glyphicon glyphicon-th-list"></span> Actions <span class="caret"></span>
+              </button>
+              <%= render 'agents/action_menu', agent: agent, return_to: (defined?(return_to) && return_to) || request.path %>
+            </div>
+          <% else %>
+            <%#= link_to 'Become User', switch_user_admin_user_path(agent.user), class: "btn btn-sm btn-default btn-info" %>
+          <% end %>
+        </td>
+      </tr>
+    <% end %>
+  </table>
+</div>
+
+<% if defined? agents.preload %>
+<%= paginate agents, :theme => 'twitter-bootstrap-3' %>
+<% end %>
diff --git a/app/views/agents/_table.html.erb b/app/views/agents/_table.html.erb
@@ -13,10 +13,10 @@
       <th></th>
     </tr>
 
-    <% @agents.each do |agent| %>
+    <% agents.each do |agent| %>
       <tr>
         <td class='<%= "agent-unavailable" if agent.unavailable? %>'>
-          <%= agent_type_icon(agent, @agents) %>
+          <%= agent_type_icon(agent, agents) %>
         </td>
         <td class='<%= "agent-unavailable" if agent.unavailable? %>'>
           <%= link_to agent.name, agent_path(agent, return: (defined?(return_to) && return_to) || request.path) %>
@@ -80,4 +80,6 @@
   </table>
 </div>
 
-<%= paginate @agents, :theme => 'twitter-bootstrap-3' %>
+<% if defined? agents.preload %>
+<%= paginate agents, :theme => 'twitter-bootstrap-3' %>
+<% end %>
diff --git a/app/views/agents/index.html.erb b/app/views/agents/index.html.erb
@@ -5,7 +5,7 @@
         <h2>Your Agents</h2>
       </div>
 
-      <%= render 'agents/table' %>
+      <%= render partial: 'agents/table', locals: {agents: @agents} %>
 
       <br/>
 
@@ -15,6 +15,10 @@
         <%= link_to icon_tag('glyphicon-random') + ' View diagram', diagram_path, class: "btn btn-default" %>
         <%= link_to icon_tag('glyphicon-adjust') + toggle_disabled_text, toggle_visibility_agents_path, method: :put, class: "btn btn-default visibility-enabler" %>
       </div>
+
+      <div class="btn-group">
+        <%= link_to 'Shared Agents', shared_agents_path, class: "btn btn-default" %>
+      </div>
     </div>
   </div>
 </div>
diff --git a/app/views/agents/shared.html.erb b/app/views/agents/shared.html.erb
@@ -0,0 +1,17 @@
+<div class='container'>
+  <div class='row'>
+    <div class='col-md-12'>
+      <div class="page-header">
+        <h2>Shared Agents</h2>
+      </div>
+
+      <%= render partial: 'agents/shared_table', locals: {agents: @agents} %>
+
+      <br/>
+
+      <div class="btn-group">
+        <%= link_to 'My Agents', agents_path, class: "btn btn-default" %>
+      </div>
+    </div>
+  </div>
+</div>
diff --git a/app/views/scenarios/show.html.erb b/app/views/scenarios/show.html.erb
@@ -10,7 +10,7 @@
         <blockquote><%= markdown(@scenario.description) %></blockquote>
       <% end %>
 
-      <%= render 'agents/table', :return_to => scenario_path(@scenario) %>
+      <%= render partial: 'agents/table', locals: {agents: @agents}, :return_to => scenario_path(@scenario) %>
 
       <br/>
 

diff --git a/config/routes.rb b/config/routes.rb
@@ -9,6 +9,7 @@
     end
 
     collection do
+      get :shared
       put :toggle_visibility
       post :propagate
       get :type_details

diff --git a/spec/controllers/agents_controller_spec.rb b/spec/controllers/agents_controller_spec.rb
@@ -26,6 +26,10 @@ def valid_attributes(options = {})
     end
   end
 
+  describe "GET all" do
+    pending("it returns all Agents if the current user is an admin")
+  end
+
   describe "POST handle_details_post" do
     it "passes control to handle_details_post on the agent" do
       sign_in users(:bob)