[Ovirt-devel] [PATCH server] Bug 478729 - Cannot add vm to smart pool after select one vm

Scott Seago sseago at redhat.com
Wed May 27 20:26:41 UTC 2009


There were two javascript functions with the same name, and the wrong one was being called. I renamed one to avoid conflicts.

Signed-off-by: Scott Seago <sseago at redhat.com>
---
 src/app/views/vm/add_to_smart_pool.rhtml |    3 ++-
 src/public/javascripts/ovirt.js          |    2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/app/views/vm/add_to_smart_pool.rhtml b/src/app/views/vm/add_to_smart_pool.rhtml
index cdfe92d..161496b 100644
--- a/src/app/views/vm/add_to_smart_pool.rhtml
+++ b/src/app/views/vm/add_to_smart_pool.rhtml
@@ -20,7 +20,8 @@
              :vms_per_page => 40} %>
 </div>
 
-<%= popup_footer("add_vms_to_smart_pool('#{url_for :controller => "smart_pools",
+<%= popup_footer("add_vms_to_current_smart_pool('#{url_for :controller =>
+                                                            "smart_pools",
                                                      :action => "add_vms",
                                                      :id => @pool}')",
                  "Add Vms") %>
diff --git a/src/public/javascripts/ovirt.js b/src/public/javascripts/ovirt.js
index 2754817..67dc455 100644
--- a/src/public/javascripts/ovirt.js
+++ b/src/public/javascripts/ovirt.js
@@ -115,7 +115,7 @@ function add_storage_to_smart_pool(url)
                }, 'json');
     }
 }
-function add_vms_to_smart_pool(url)
+function add_vms_to_current_smart_pool(url)
 {
     vms= get_selected_checkboxes("add_smart_vms_grid_form");
     if (validate_selected(vms, "vm")) {
-- 
1.6.0.6




More information about the ovirt-devel mailing list