Standup controller #show now accepts json request and exposes time zone in IANA format

Author: Andrew Leung and Dariusz Lorenc

app/controllers/standups_controller.rb

@@ -2,6 +2,7 @@ class StandupsController < ApplicationController
   before_filter :load_standup, only: [:edit, :show, :update, :destroy]
   around_filter :standup_timezone, only: [:edit, :show, :update, :destroy]
 
+
   def create
     @standup = Standup.create(params[:standup])
 
@@ -18,18 +19,33 @@ def new
   end
 
   def index
-    @standups = Standup.all.sort{ |a,b| a.title.downcase <=> b.title.downcase }
+    @standups = Standup.all.sort { |a, b| a.title.downcase <=> b.title.downcase }
   end
 
-  def edit; end
+  def edit;
+  end
 
   def show
     if @standup
       session[:last_visited_standup] = params[:id]
-      redirect_to standup_items_path(@standup)
+      respond_to do |format|
+        format.html {
+          redirect_to standup_items_path(@standup)
+        }
+        format.json {
+          render json: @standup.to_json(:methods => :time_zone_name_iana)
+        }
+      end
     else
-      flash[:error] = "A standup with the ID #{params[:id]} does not exist."
-      redirect_to standups_path
+      respond_to do |format|
+        format.html {
+          flash[:error] = "A standup with the ID #{params[:id]} does not exist."
+          redirect_to standups_path
+        }
+        format.json {
+          head :not_found
+        }
+      end
     end
   end
 

app/models/standup.rb

@@ -33,6 +33,10 @@ def next_standup_date
     standup_time
   end
 
+  def time_zone_name_iana
+    ActiveSupport::TimeZone.find_tzinfo(self.time_zone_name).identifier
+  end
+
   def finished_today
     standup_time_today < time_zone.now
   end

spec/controllers/standups_controller_spec.rb

@@ -79,6 +79,19 @@
       expect(response.body).to redirect_to standup_items_path(standup)
     end
 
+    it "renders json and time zone IANA field" do
+      request.accept = Mime::JSON.to_s
+      get :show, id: standup.id
+      expect(response.content_type).to eq(Mime::JSON.to_s)
+      expect(response.body).to include(standup.time_zone_name_iana)
+    end
+
+    it "returns 404 when standup not found" do
+      request.accept = Mime::JSON.to_s
+      get :show, id: 123
+      expect(response).to be_not_found
+    end
+
     it 'saves standup id to cookie' do
       get :show, id: standup.id
       expect(session[:last_visited_standup]).to eq(standup.id.to_s)