[Ovirt-devel] [PATCH] fix for bug 464282: delete permissions
Jason Guiditta
jguiditt at redhat.com
Wed Dec 17 19:13:08 UTC 2008
On Fri, 2008-12-05 at 20:19 +0000, Scott Seago wrote:
> 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,
ACK, this works for me.
More information about the ovirt-devel
mailing list