diff --git a/app/controllers/aspect_memberships_controller.rb b/app/controllers/aspect_memberships_controller.rb
index 6e9f8edfce5ceee7e2d0f41e10bb2e2f7c0ee167..66cb7917d1f9649360b37e8ba7d79428cd7a6334 100644
--- a/app/controllers/aspect_memberships_controller.rb
+++ b/app/controllers/aspect_memberships_controller.rb
@@ -9,13 +9,13 @@ class AspectMembershipsController < ApplicationController
   def destroy
     #note :id is garbage
 
-      @person_id = params[:person_id]
-      @aspect_id = params[:id]
+    @person_id = params[:person_id]
+    @aspect_id = params[:aspect_id]
 
-      @contact = current_user.contact_for(Person.where(:id => @person_id).first)
-      membership = @contact ? @contact.aspect_memberships.where(:aspect_id => @aspect_id).first : nil
+    @contact = current_user.contact_for(Person.where(:id => @person_id).first)
+    membership = @contact ? @contact.aspect_memberships.where(:aspect_id => @aspect_id).first : nil
 
-      if membership && membership.destroy
+    if membership && membership.destroy
         flash.now[:notice] = I18n.t 'aspect_memberships.destroy.success'
 
         respond_to do |format|