[Ovirt-devel] [PATCH] fix for bug 464282: delete permissions

[Scott Seago]

Delete permissions now works -- had a conditional statement wrong in the controller action.

Signed-off-by: Scott Seago <sseago at redhat.com>
---
 src/app/controllers/permission_controller.rb |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/app/controllers/permission_controller.rb b/src/app/controllers/permission_controller.rb
index 5d3646d..813d9d9 100644
--- a/src/app/controllers/permission_controller.rb
+++ b/src/app/controllers/permission_controller.rb
@@ -102,7 +102,7 @@ class PermissionController < ApplicationController
       Permission.transaction do
         permissions = Permission.find(:all, :conditions => "id in (#{permission_ids.join(', ')})")
         permissions.each do |permission|
-          permission.destroy unless permission.is_primary?
+          permission.destroy if permission.is_primary?
         end
       end
       render :json => { :object => "permission", :success => true, 
-- 
1.6.0.4