diff --git a/app/controllers/aspect_memberships_controller.rb b/app/controllers/aspect_memberships_controller.rb index 6bb565d1d..14e0ab907 100644 --- a/app/controllers/aspect_memberships_controller.rb +++ b/app/controllers/aspect_memberships_controller.rb @@ -22,11 +22,11 @@ class AspectMembershipsController < ApplicationController flash.now[:notice] = I18n.t 'aspect_memberships.destroy.success' respond_with do |format| - format.all{ } format.json{ render :json => { :person_id => @person_id, :aspect_ids => @contact.aspects.map{|a| a.id} } } + format.html{ redirect_to :back } end else diff --git a/spec/controllers/aspect_memberships_controller_spec.rb b/spec/controllers/aspect_memberships_controller_spec.rb index b190f728d..e94bad8dc 100644 --- a/spec/controllers/aspect_memberships_controller_spec.rb +++ b/spec/controllers/aspect_memberships_controller_spec.rb @@ -93,6 +93,16 @@ describe AspectMembershipsController do @aspect0.reload @aspect0.contacts.include?(@contact).should be false end + it 'does not 500 on an html request' do + alice.add_contact_to_aspect(@contact, @aspect1) + delete :destroy, + :id => 123, + :person_id => bob.person.id, + :aspect_id => @aspect0.id + response.should redirect_to :back + @aspect0.reload + @aspect0.contacts.include?(@contact).should be false + end context 'aspect membership does not exist' do it 'person does not exist' do