From 594d37bb40f3e3e0f15553383cf2a305d4d6742e Mon Sep 17 00:00:00 2001
From: mo khan <mo@mokhan.ca>
Date: Mon, 9 Feb 2015 19:43:03 -0700
Subject: extract api.

---
 app/controllers/agents_controller.rb | 58 ++++++++++--------------------------
 1 file changed, 16 insertions(+), 42 deletions(-)

(limited to 'app/controllers/agents_controller.rb')

diff --git a/app/controllers/agents_controller.rb b/app/controllers/agents_controller.rb
index a76a7ae..4acef9d 100644
--- a/app/controllers/agents_controller.rb
+++ b/app/controllers/agents_controller.rb
@@ -1,74 +1,48 @@
 class AgentsController < ApplicationController
-  before_action :set_agent, only: [:show, :edit, :update, :destroy]
-
-  # GET /agents
-  # GET /agents.json
   def index
     @agents = Agent.all
   end
 
-  # GET /agents/1
-  # GET /agents/1.json
   def show
+    @agent = Agent.find(params[:id])
   end
 
-  # GET /agents/new
   def new
     @agent = Agent.new
   end
 
-  # GET /agents/1/edit
   def edit
+    @agent = Agent.find(params[:id])
   end
 
-  # POST /agents
-  # POST /agents.json
   def create
     @agent = Agent.new(agent_params)
 
-    respond_to do |format|
-      if @agent.save
-        format.html { redirect_to @agent, notice: 'Agent was successfully created.' }
-        format.json { render :show, status: :created, location: @agent }
-      else
-        format.html { render :new }
-        format.json { render json: @agent.errors, status: :unprocessable_entity }
-      end
+    if @agent.save
+      redirect_to @agent, notice: 'Agent was successfully created.'
+    else
+      render :new
     end
   end
 
-  # PATCH/PUT /agents/1
-  # PATCH/PUT /agents/1.json
   def update
-    respond_to do |format|
-      if @agent.update(agent_params)
-        format.html { redirect_to @agent, notice: 'Agent was successfully updated.' }
-        format.json { render :show, status: :ok, location: @agent }
-      else
-        format.html { render :edit }
-        format.json { render json: @agent.errors, status: :unprocessable_entity }
-      end
+    @agent = Agent.find(params[:id])
+    if @agent.update(agent_params)
+      redirect_to @agent, notice: 'Agent was successfully updated.'
+    else
+      render :edit
     end
   end
 
-  # DELETE /agents/1
-  # DELETE /agents/1.json
   def destroy
+    @agent = Agent.find(params[:id])
     @agent.destroy
-    respond_to do |format|
-      format.html { redirect_to agents_url, notice: 'Agent was successfully destroyed.' }
-      format.json { head :no_content }
-    end
+    redirect_to agents_url, notice: 'Agent was successfully destroyed.'
   end
 
   private
-    # Use callbacks to share common setup or constraints between actions.
-    def set_agent
-      @agent = Agent.find(params[:id])
-    end
 
-    # Never trust parameters from the scary internet, only allow the white list through.
-    def agent_params
-      params.require(:agent).permit(:hostname)
-    end
+  def agent_params
+    params.require(:agent).permit(:hostname)
+  end
 end
-- 
cgit v1.2.3


From 64dd29d70ddda6f4927e9352e3e90c4f87b5041a Mon Sep 17 00:00:00 2001
From: mo khan <mo@mokhan.ca>
Date: Mon, 9 Feb 2015 20:00:21 -0700
Subject: display created at for agents and sort by created_at descending.

---
 app/controllers/agents_controller.rb | 2 +-
 app/views/agents/index.html.erb      | 7 ++++---
 2 files changed, 5 insertions(+), 4 deletions(-)

(limited to 'app/controllers/agents_controller.rb')

diff --git a/app/controllers/agents_controller.rb b/app/controllers/agents_controller.rb
index 4acef9d..dc2717a 100644
--- a/app/controllers/agents_controller.rb
+++ b/app/controllers/agents_controller.rb
@@ -1,6 +1,6 @@
 class AgentsController < ApplicationController
   def index
-    @agents = Agent.all
+    @agents = Agent.all.order(created_at: :desc)
   end
 
   def show
diff --git a/app/views/agents/index.html.erb b/app/views/agents/index.html.erb
index 3a6b738..75adfdb 100644
--- a/app/views/agents/index.html.erb
+++ b/app/views/agents/index.html.erb
@@ -9,15 +9,16 @@
       <thead>
         <tr>
           <th>Hostname</th>
-          <th colspan="4"></th>
+          <th>Created At</th>
+          <th colspan="3"></th>
         </tr>
       </thead>
       <tbody>
         <% @agents.each do |agent| %>
           <tr>
-            <td><%= agent.hostname %></td>
+            <td><%= link_to agent.hostname, agent_path(agent) %></td>
+            <td><%= agent.created_at %></td>
             <td><%= link_to 'Events', agent_events_path(agent) %></td>
-            <td><%= link_to 'Show', agent %></td>
             <td><%= link_to 'Edit', edit_agent_path(agent) %></td>
             <td><%= link_to 'Destroy', agent, method: :delete, data: { confirm: 'Are you sure?' } %></td>
           </tr>
-- 
cgit v1.2.3