refactor taxonomy tree to use a function

Fixes #2622
spree · Feb 28, 2013 · bb4ca63 · bb4ca63
1 parent f789a81
commit bb4ca63
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/backend/app/assets/javascripts/admin/taxonomy.js b/backend/app/assets/javascripts/admin/taxonomy.js
@@ -76,9 +76,8 @@ var handle_delete = function(e, data){
 
 };
 
-var taxonomy_id; 
 
-$(document).ready(function(){
+var setup_taxonomy_tree = function(taxonomy_id) {
   if(taxonomy_id!=undefined){
     $.ajax({
       url: '/api/taxonomies/' + taxonomy_id + '/jstree', 
@@ -210,4 +209,4 @@ $(document).ready(function(){
       }
     });
   }
-});
+};
diff --git a/backend/app/views/spree/admin/taxonomies/_js_head.html.erb b/backend/app/views/spree/admin/taxonomies/_js_head.html.erb
@@ -3,6 +3,11 @@
     var loading = '#{escape_javascript t(:loading)}';
     var new_taxon = '#{escape_javascript t(:new_taxon)}';
     var server_error = '#{escape_javascript t(:server_error)}';
-    var taxonomy_tree_error = '#{escape_javascript t(:taxonomy_tree_error)}';"
+    var taxonomy_tree_error = '#{escape_javascript t(:taxonomy_tree_error)}';
+
+    $(document).ready(function(){
+      setup_taxonomy_tree(taxonomy_id);
+    });
+    "
   %>
 <% end %>