ActionItem726: new products begin selecting top level categories

Moises Machado
1 parent a70c44b1
Showing 3 changed files with 26 additions and 20 deletions Show diff stats
app/controllers/my_profile/manage_products_controller.rb
app/views/manage_products/_subcategories.rhtml
test/functional/manage_products_controller_test.rb
@@ -23,9 +23,8 @@ class ManageProductsController &lt; ApplicationController
   end
   def new
-    @current_category = ProductCategory.top_level_for(environment).first
     @object = Product.new
-    @categories = @current_category.nil? ? [] : @current_category.children
+    @categories = @current_category.nil? ? ProductCategory.top_level_for(environment) : @current_category.children
     @product = @profile.products.build(params[:product])
     @product.build_image unless @product.image
     if request.post?
@@ -75,9 +74,10 @@ class ManageProductsController &lt; ApplicationController
     end
     render :partial => 'shared/select_categories', :locals => {:object_name => 'product', :multiple => false}, :layout => false
   end
+
   def update_subcategories
-    @current_category = ProductCategory.find(params[:id])
-    @categories = @current_category.children
+    @current_category = ProductCategory.find(params[:id]) if params[:id]
+    @categories = @current_category ? @current_category.children : ProductCategory.top_level_for(environment)
     render :partial => 'subcategories'
   end
-<% if !@current_category.nil? and !@current_category.top_level? %>
+<% if !@current_category.nil? %>
   <p><%= _('Current category:') %></p>
   <%= hidden_field_tag 'product[product_category_id]', @current_category.id %>
-  <%
-    categories = [@current_category]
-    categories.push(@current_category) while @current_category = @current_category.parent
-    categories.last.name = _('Back')
+  <%= link_to_remote( _('Back'),
+      :update => "subcategories",
+      :url => { :action => 'update_subcategories', :id => nil, :loaded => visual_effect(:highlight, "subcategories") },
+      :class => 'select-current-category-link') + ' &rarr; '
+  %>
+  <%= ([@current_category] + @current_category.ancestors).reverse.map{|i|
+    link_to_remote(i.name,
+      :update => "subcategories",
+      :url => { :action => 'update_subcategories', :id => i, :loaded => visual_effect(:highlight, "subcategories") },
+      :class => 'select-current-category-link')}.join(' &rarr; ')
   %>
-  <% if categories.size > 1 %>
-    <%= categories.compact.reverse.map{|i|
-      link_to_remote(i.name, 
-        :update => "subcategories",
-        :url => { :action => 'update_subcategories', :id => i, :loaded => visual_effect(:highlight, "subcategories") },
-        :class => 'select-current-category-link')}.join(' &rarr; ')
-    %>
-  <% end %>
-
 <% end %>
-<% if @current_category %>
+<% if @current_category.nil? %>
   <p><%= _('Select a category:') %></p>
 <% elsif !@categories.empty?  %>
   <p><%= _('Select a subcategory:') %></p>
@@ -120,7 +120,7 @@ class ManageProductsControllerTest &lt; Test::Unit::TestCase
     category3 = ProductCategory.create!(:name => 'Category 3', :environment => environment, :parent => category2)
     get :new, :profile => @enterprise.identifier
     assert_tag :tag => 'p', :content => /Select a category:/
-    assert_tag :tag => 'a', :content => /#{category2.name}/
+    assert_tag :tag => 'a', :content => /#{category1.name}/
   end
   should 'show current category' do
@@ -265,5 +265,14 @@ class ManageProductsControllerTest &lt; Test::Unit::TestCase
     assert_template 'not_found.rhtml'
   end
+
+  should 'show top level product categories for the user to choose' do
+    pc1 = ProductCategory.create!(:name => 'test_category1', :environment => Environment.default)
+    pc2 = ProductCategory.create!(:name => 'test_category2', :environment => Environment.default)
+
+    get :new, :profile => @enterprise.identifier
+
+    assert_equivalent [pc1, pc2], assigns(:categories)
+  end
 end