Return 406 on people/guid/contacts, format json

app/controllers/people_controller.rb

@@ -135,24 +135,30 @@ class PeopleController < ApplicationController
   end
 
   def contacts
-    @person = Person.find_by_guid(params[:person_id])
+    respond_to do |format|
+      format.json { render nothing: true, status: 406 }
 
-    if @person
+      format.any do
-      @contact = current_user.contact_for(@person)
+        @person = Person.find_by_guid(params[:person_id])
-      @contacts_of_contact = Contact.contact_contacts_for(current_user, @person)
+
-      gon.preloads[:person] = PersonPresenter.new(@person, current_user).full_hash_with_profile
+        if @person
-      gon.preloads[:photos] = {
+          @contact = current_user.contact_for(@person)
-        count: photos_from(@person, :all).count(:all)
+          @contacts_of_contact = Contact.contact_contacts_for(current_user, @person)
-      }
+          gon.preloads[:person] = PersonPresenter.new(@person, current_user).full_hash_with_profile
-      gon.preloads[:contacts] = {
+          gon.preloads[:photos] = {
-        count: @contacts_of_contact.count(:all),
+            count: photos_from(@person, :all).count(:all)
-      }
+          }
-      @contacts_of_contact = @contacts_of_contact.paginate(:page => params[:page], :per_page => (params[:limit] || 15))
+          gon.preloads[:contacts] = {
-      @hashes = hashes_for_people @contacts_of_contact, @aspects
+            count: @contacts_of_contact.count(:all),
-      respond_with @person, layout: "with_header"
+          }
-    else
+          @contacts_of_contact = @contacts_of_contact.paginate(page: params[:page], per_page: (params[:limit] || 15))
-      flash[:error] = I18n.t 'people.show.does_not_exist'
+          @hashes = hashes_for_people @contacts_of_contact, @aspects
-      redirect_to people_path
+          respond_with @person, layout: "with_header"
+        else
+          flash[:error] = I18n.t "people.show.does_not_exist"
+          redirect_to people_path
+        end
+      end
     end
   end
 

spec/controllers/people_controller_spec.rb

@@ -497,6 +497,11 @@ describe PeopleController, :type => :controller do
       get :contacts, :person_id => eve.person.to_param
       expect(response.body).to include '"photos":{"count":16}' # eve is not sharing with alice
     end
+
+    it "returns a 406 for json format" do
+      get :contacts, person_id: "foo", format: :json
+      expect(response.code).to eq("406")
+    end
   end
 
   describe '#diaspora_id?' do