[Ovirt-devel] [PATCH server] fixed a couple refactoring bugs: set success to 'true' in the json rather than no-longer-created success variable.

Scott Seago sseago at redhat.com
Thu May 14 16:28:51 UTC 2009


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

diff --git a/src/app/controllers/resources_controller.rb b/src/app/controllers/resources_controller.rb
index bf2a234..6990df7 100644
--- a/src/app/controllers/resources_controller.rb
+++ b/src/app/controllers/resources_controller.rb
@@ -90,7 +90,7 @@ class ResourcesController < PoolController
       raise PartialSuccessError.new("Delete failed for some VM Pools",
                                     failures, successes)
     end
-    render :json => { :object => "vm_resource_pool", :success => success,
+    render :json => { :object => "vm_resource_pool", :success => true,
                       :alert => "VM Pools were successfully deleted." }
   end
 
diff --git a/src/app/controllers/vm_controller.rb b/src/app/controllers/vm_controller.rb
index e918ae7..b51f4ae 100644
--- a/src/app/controllers/vm_controller.rb
+++ b/src/app/controllers/vm_controller.rb
@@ -104,7 +104,7 @@ class VmController < ApplicationController
       raise PartialSuccessError.new("Delete failed for some VMs",
                                     failures, successes)
     end
-    render :json => { :object => "vm", :success => success,
+    render :json => { :object => "vm", :success => true,
                       :alert => "VM Pools were successfully deleted." }
   end
 
-- 
1.6.0.6




More information about the ovirt-devel mailing list