caboose-store 0.0.21 → 0.0.22

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NmYwMGM3MzgxNTEyNjlkYWNkNTI5MzU5NGFjM2Q2M2IzODM2N2FhMw==
+    OTE4N2M4ZWE4NjYwZDEzYzM2MmY3YTA5Y2ViNjc5ZjgwNWU5Yjc2MA==
   data.tar.gz: !binary |-
-    Yzc3Zjg0MzQ4ZmQ3YzVjYTBkMTJmMjE4ZjgxOGNmMjhhY2NkNGM1NA==
+    MWZmZmYzNzMyMjJjYjA4MWRjOTk2YTI3ZWNhYzllNDU4NzZlNGI0Yw==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    NGU2YjIxZTU4ZTE1Mzg1NTYxMjZmYTkyOTgxYWIwZWMzNDRjYzFhOWYxY2Ey
-    ODkwM2U0ZWRiNzc5NWVmMjgyYWU0ZTA0OGM1ZjQxZTg0YjlkZWQxOGJkMzdk
-    Y2MzN2FkNTliNTJkODllOTVjYjk0MjY1MzA0OGExMTI0YTdiMTQ=
+    Yjg2MzI2OThmMWM1OTk3MzkxMThiM2ZlODk4M2Y4OGVlOWIxMjE5YTM1YjQ2
+    YWNhNzc0YWYzMmE3OTkzZmQzN2M2NWUyNzRhNTA5MjU4NGVkODM2MTk2MzM0
+    MjY4YTk3MzNlOTE1YmY2Y2U0ZWQyM2JmYWY2MDU0N2RhZDZmY2Y=
   data.tar.gz: !binary |-
-    MmMzYTM0YmM0MzYyODA5MDA4ZmRmNGJjOWZkMzgyM2RhNmRlNTNhOTI1MTlk
-    YzNhM2QyMDBiNWEzNWM0N2E2MzdjNzI3Y2QxY2RkZjY0M2Y2NDkwZTQ2MTA3
-    OGVjNTZmOWU1OWVhMDcxMWFjMjVmNzgxZGMwZTJkMzRhMjY5ZDc=
+    NDg1OTllZjE0MWJhMDY4MDAzMzA0MmU3MmI3NTFhYTcwMjRjNGQ4YzU4YTY4
+    M2FkYmY0MTkzZmU3MGM0ZjVlZjIxNDE3YTc0MzdhY2FkMTI0MTg0NGE4NjU0
+    NjZlZTI3ZTg4ZDliZjg4YzgyOTc0OWJlOTkzNDNjYjJiOWIwMDU=

data/app/assets/javascripts/caboose_store/modules/product.js

@@ -21,6 +21,9 @@ Caboose.Store.Modules.Product = (function() {
     
     $.get('/products/' + self.$product.data('id') + '/info', function(response) {
       self.product = response.product;
+      self.option1_values = response.option1_values;
+      self.option2_values = response.option2_values;
+      self.option3_values = response.option3_values;
       self.render();
       self.bindEvents();
       self.setVariant(self.getInitialVariant());
@@ -173,14 +176,20 @@ Caboose.Store.Modules.Product = (function() {
   };
   
   self.getOptionsWithAllValues = function() {
-    return _.map(self.getOptionsFromProduct(), function(optionName) {
-      return {
-        name: optionName,
-        values: _.uniq(_.map(self.product.variants, function(variant) {
-          return variant[self.getOptionAttribute(optionName)];
-        }))
-      };
-    });
+    //return _.map(self.getOptionsFromProduct(), function(optionName) {
+    //  return {
+    //    name: optionName,
+    //    values: _.uniq(_.map(self.product.variants, function(variant) {
+    //      return variant[self.getOptionAttribute(optionName)];
+    //    }))
+    //  };
+    //});
+    
+    var options = [];
+    if (self.product.option1) options.push({ name: self.product.option1, values: self.option1_values });
+    if (self.product.option2) options.push({ name: self.product.option2, values: self.option2_values });
+    if (self.product.option3) options.push({ name: self.product.option3, values: self.option3_values });
+    return options;          
   };
   
   self.getOptionAttribute = function(option) {

data/app/controllers/caboose_store/checkout_controller.rb

@@ -37,6 +37,15 @@ module CabooseStore
       redirect_to '/checkout/step-two'   and return if @order.shipping_address.nil? || @order.billing_address.nil?
       redirect_to '/checkout/step-three' and return if @order.shipping_code.nil?
       
+      # Make sure all the variants still exist      
+      @order.line_items.each do |li|
+        v = Variant.where(:id => li.variant_id).first
+        if v.nil? || v.status == 'Deleted'
+          render :file => 'caboose_store/checkout/deleted_variant'
+          return
+        end
+      end
+      
       case CabooseStore::payment_processor
         when 'authorize.net'
           @sim_transaction = AuthorizeNet::SIM::Transaction.new(

data/app/controllers/caboose_store/products_controller.rb

@@ -97,7 +97,13 @@ module CabooseStore
     
     # GET /product/info
     def info
-      render :json => { :product => Product.find(params[:id]) }
+      p = Product.find(params[:id])
+      render :json => { 
+        :product => p,
+        :option1_values => p.option1_values,
+        :option2_values => p.option2_values,
+        :option3_values => p.option3_values
+      }
     end
     
     # GET /api/products
@@ -458,6 +464,13 @@ module CabooseStore
       render :layout => 'caboose/admin'
     end
     
+    # GET /admin/products/:id/variants/sort-order  
+    def admin_edit_variant_sort_order
+      return if !user_is_allowed('products', 'edit')    
+      @product = Product.find(params[:id])      
+      render :layout => 'caboose/admin'
+    end
+    
     # GET /admin/products/:id/delete
     def admin_delete_form
       return if !user_is_allowed('products', 'edit')    
@@ -638,8 +651,40 @@ module CabooseStore
     def admin_update_sort_order
       params[:product_ids].each_with_index do |product_id, index|
         Product.find(product_id.to_i).update_attribute(:sort_order, index)
-      end
-      
+      end      
+      render :json => { :success => true }
+    end
+    
+    # PUT /admin/products/:id/variants/option1-sort-order
+    def admin_update_variant_option1_sort_order
+      product_id = params[:id]
+      params[:values].each_with_index do |value, i|
+        Variant.where(:product_id => product_id, :option1 => value).all.each do |v|
+          v.update_attribute(:option1_sort_order, i)
+        end
+      end            
+      render :json => { :success => true }
+    end
+    
+    # PUT /admin/products/:id/variants/option1-sort-order
+    def admin_update_variant_option2_sort_order            
+      product_id = params[:id]
+      params[:values].each_with_index do |value, i|
+        Variant.where(:product_id => product_id, :option2 => value).all.each do |v|
+          v.update_attribute(:option2_sort_order, i)
+        end
+      end            
+      render :json => { :success => true }
+    end
+    
+    # PUT /admin/products/:id/variants/option1-sort-order
+    def admin_update_variant_option3_sort_order      
+      product_id = params[:id]
+      params[:values].each_with_index do |value, i|
+        Variant.where(:product_id => product_id, :option3 => value).all.each do |v|
+          v.update_attribute(:option3_sort_order, i)
+        end
+      end            
       render :json => { :success => true }
     end
   end

data/app/models/caboose_store/product.rb

@@ -8,7 +8,7 @@ module CabooseStore
     has_many :customization_memberships, :class_name => 'CabooseStore::CustomizationMembership'
     has_many :categories, :class_name => 'CabooseStore::Category', :through => :category_memberships
     has_many :category_memberships, :class_name => 'CabooseStore::CategoryMembership'
-    has_many :variants, :class_name => 'CabooseStore::Variant'
+    has_many :variants, :class_name => 'CabooseStore::Variant', :order => 'sort_order'
     has_many :product_images, :class_name => 'CabooseStore::ProductImage'
     has_many :proudct_inputs, :class_name => 'CabooseStore::ProductInput'
     has_many :reviews, :class_name => 'CabooseStore::Review'
@@ -130,5 +130,17 @@ module CabooseStore
       # Return variants that haven't been "deleted"
       self.variants.where(:status => ['Active', 'Inactive'])
     end
+    
+    def option1_values
+      arr = self.variants.reorder(:option1_sort_order).pluck(:option1).uniq.reject { |x| x.nil? || x.empty? }
+    end
+    
+    def option2_values
+      self.variants.reorder(:option2_sort_order).pluck(:option2).uniq.reject { |x| x.nil? || x.empty? }
+    end
+    
+    def option3_values
+      self.variants.reorder(:option3_sort_order).pluck(:option3).uniq.reject { |x| x.nil? || x.empty? }
+    end
   end
 end

data/app/models/caboose_store/schema.rb

@@ -197,7 +197,10 @@ class CabooseStore::Schema < Caboose::Utilities::Schema
         [ :product_id            , :integer  ],
         [ :shipping_unit_value   , :numeric  ],
         [ :alternate_id          , :string   ],
-        [ :status                , :string   ]
+        [ :status                , :string   ],
+        [ :option1_sort_order    , :integer  , { :defult => 0 }],
+        [ :option2_sort_order    , :integer  , { :defult => 0 }],
+        [ :option3_sort_order    , :integer  , { :defult => 0 }]
       ],
       
       CabooseStore::Vendor => [

data/app/views/caboose_store/products/_admin_header.html.erb

@@ -15,11 +15,11 @@ tabs = {
   "/admin/products/#{@product.id}/categories"   => 'Categories'
 }
 
-tabs["/admin/products/#{@product.id}/options"        ] = 'Options <span style="font-size: 65%">(size, color, style, etc.)</span>' if @product.options && @product.options.count > 0
-tabs["/admin/products/#{@product.id}/variants"       ] = 'Inventory & variants'
-# tabs["/admin/products/#{@product.id}/variants/group" ] = 'Group Variants'
-tabs["/admin/products/#{@product.id}/images"         ] = 'Images'
-tabs["/admin/products/#{@product.id}/delete"         ] = 'Delete Product'
+tabs["/admin/products/#{@product.id}/options"             ] = 'Options <span style="font-size: 65%">(size, color, style, etc.)</span>' if @product.options && @product.options.count > 0
+tabs["/admin/products/#{@product.id}/variants/sort-order" ] = 'Sort Option Values' if @product.options && @product.options.count > 0
+tabs["/admin/products/#{@product.id}/variants"            ] = 'Inventory & Variants'
+tabs["/admin/products/#{@product.id}/images"              ] = 'Images'
+tabs["/admin/products/#{@product.id}/delete"              ] = 'Delete Product'
 
 #tabs["/admin/products/#{@product.id}/seo"          ] = 'SEO'
 

data/app/views/caboose_store/products/admin_edit_variant_sort_order.html.erb

@@ -0,0 +1,63 @@
+<%
+p = @product
+%>
+
+<%= render :partial => 'caboose_store/products/admin_header' %>
+
+<div id='options'>
+<% if p.option1 %><div id='option1_container'><h2><%= p.option1 %></h2><ul id='option1'><% p.option1_values.each do |option| %><li id='values_<%= option %>'><%= option %></li><% end %></ul></div><% end %>
+<% if p.option2 %><div id='option2_container'><h2><%= p.option2 %></h2><ul id='option2'><% p.option2_values.each do |option| %><li id='values_<%= option %>'><%= option %></li><% end %></ul></div><% end %>
+<% if p.option3 %><div id='option3_container'><h2><%= p.option3 %></h2><ul id='option3'><% p.option3_values.each do |option| %><li id='values_<%= option %>'><%= option %></li><% end %></ul></div><% end %>
+</div>                                                               
+
+<div id='message'></div>
+
+<%= render :partial => 'caboose_store/products/admin_footer' %>
+
+<% content_for :caboose_css do %>
+<style type='text/css'>
+#options div { width: 33%; float: left; }
+#options ul { margin: 0 10px 0 0; padding: 0; list-style: none; }
+#options li { margin: 0; padding: 4px 8px; list-style: none; border: #ccc 1px solid; }
+#message { clear: left; }
+</style>
+<% end %>
+
+<% content_for :caboose_js do %>
+<script type='text/javascript'>
+
+$(document).ready(function() {  
+  $('#option1').sortable({    
+    update: function(event, ui) {
+      $.ajax({
+        url: '/admin/products/<%= p.id %>/variants/option1-sort-order',
+        type: 'put',
+        data: $('#option1').sortable('serialize'),
+        success: function(resp) {}
+      });        
+    }    
+  });
+  $('#option2').sortable({    
+    update: function(event, ui) {
+      $.ajax({
+        url: '/admin/products/<%= p.id %>/variants/option2-sort-order',
+        type: 'put',
+        data: $('#option2').sortable('serialize'),
+        success: function(resp) {}
+      });        
+    }    
+  });
+  $('#option3').sortable({    
+    update: function(event, ui) {
+      $.ajax({
+        url: '/admin/products/<%= p.id %>/variants/option3-sort-order',
+        type: 'put',
+        data: $('#option3').sortable('serialize'),
+        success: function(resp) {}
+      });        
+    }    
+  });
+});
+
+</script>
+<% end %>

data/app/views/caboose_store/products/admin_edit_variants.html.erb

@@ -30,60 +30,64 @@ end
 %></div>
 
 <div id='variants_wrapper'>
-<table id='variants'>
-  <tr>
-    <% if @cols['option1'           ] && p.option1 %><th valign='bottom'><%= p.option1    %></th><% end %>
-    <% if @cols['option2'           ] && p.option2 %><th valign='bottom'><%= p.option2    %></th><% end %>
-    <% if @cols['option3'           ] && p.option3 %><th valign='bottom'><%= p.option3    %></th><% end %>
-    <% if @cols['status'            ] == true      %><th valign='bottom'>Status             </th><% end %>
-    <% if @cols['alternate_id'      ] == true      %><th valign='bottom'>Alternate ID       </th><% end %>
-    <% if @cols['barcode'           ] == true      %><th valign='bottom'>Barcode            </th><% end %>
-    <% if @cols['price'             ] == true      %><th valign='bottom'>Price              </th><% end %>
-    <% if @cols['quantity_in_stock' ] == true      %><th valign='bottom'>Quantity           </th><% end %>
-    <% if @cols['weight'            ] == true      %><th valign='bottom'>Weight             </th><% end %>
-    <% if @cols['length'            ] == true      %><th valign='bottom'>Length             </th><% end %>
-    <% if @cols['width'             ] == true      %><th valign='bottom'>Width              </th><% end %>
-    <% if @cols['height'            ] == true      %><th valign='bottom'>Height             </th><% end %>
-    <% if @cols['cylinder'          ] == true      %><th valign='bottom'>Cylinder           </th><% end %>
-    <% if @cols['requires_shipping' ] == true      %><th valign='bottom'>Requires Shipping  </th><% end %>
-    <% if @cols['allow_backorder'   ] == true      %><th valign='bottom'>Allow Backorder    </th><% end %>
-    <% if @cols['taxable'           ] == true      %><th valign='bottom'>Taxable            </th><% end %>
-    <th>&nbsp;</th>
-    <th>&nbsp;</th>
-  </tr>
-<% Caboose.log p.variants.where(:status => ['Active', 'Inactive']).first %>
-<% p.variants.where(:status => ['Active', 'Inactive']).each do |v| %>
-  <tr <% if @highlight_variant_id && v.id == @highlight_variant_id %>class='highlight'<% end %>>
-    <% if @cols['option1'           ] && p.option1 %><td><div id='variant_<%= v.id %>_option1'           ></div></td><% end %>
-    <% if @cols['option2'           ] && p.option2 %><td><div id='variant_<%= v.id %>_option2'           ></div></td><% end %>
-    <% if @cols['option3'           ] && p.option3 %><td><div id='variant_<%= v.id %>_option3'           ></div></td><% end %>
-    <% if @cols['status'            ] == true      %><td><div id='variant_<%= v.id %>_status'            ></div></td><% end %>
-    <% if @cols['alternate_id'      ] == true      %><td><div id='variant_<%= v.id %>_alternate_id'      ></div></td><% end %>
-    <% if @cols['barcode'           ] == true      %><td><div id='variant_<%= v.id %>_barcode'           ></div></td><% end %> 
-    <% if @cols['price'             ] == true      %><td><div id='variant_<%= v.id %>_price'             ></div></td><% end %> 
-    <% if @cols['quantity_in_stock' ] == true      %><td><div id='variant_<%= v.id %>_quantity_in_stock' ></div></td><% end %> 
-    <% if @cols['weight'            ] == true      %><td><div id='variant_<%= v.id %>_weight'            ></div></td><% end %> 
-    <% if @cols['length'            ] == true      %><td><div id='variant_<%= v.id %>_length'            ></div></td><% end %> 
-    <% if @cols['width'             ] == true      %><td><div id='variant_<%= v.id %>_width'             ></div></td><% end %> 
-    <% if @cols['height'            ] == true      %><td><div id='variant_<%= v.id %>_height'            ></div></td><% end %> 
-    <% if @cols['cylinder'          ] == true      %><td><div id='variant_<%= v.id %>_cylinder'          ></div></td><% end %> 
-    <% if @cols['requires_shipping' ] == true      %><td><div id='variant_<%= v.id %>_requires_shipping' ></div></td><% end %>
-    <% if @cols['allow_backorder'   ] == true      %><td><div id='variant_<%= v.id %>_allow_backorder'   ></div></td><% end %>
-    <% if @cols['taxable'           ] == true      %><td><div id='variant_<%= v.id %>_taxable'           ></div></td><% end %>
-    <td class='edit'><a href='/admin/products/<%= p.id %>/variants/<%= v.id %>/edit'>Edit</a></td>
-    <td class='delete'><a href='#' onclick='delete_variant(<%= v.id %>);'>Delete</a></td>
-  </tr>
-<% end %>
-</table>
-
+  <table id='variants'>
+    <tr>
+      <th>&nbsp;</th>
+      <% if @cols['option1'           ] && p.option1 %><th valign='bottom'><%= p.option1    %></th><% end %>
+      <% if @cols['option2'           ] && p.option2 %><th valign='bottom'><%= p.option2    %></th><% end %>
+      <% if @cols['option3'           ] && p.option3 %><th valign='bottom'><%= p.option3    %></th><% end %>
+      <% if @cols['status'            ] == true      %><th valign='bottom'>Status             </th><% end %>
+      <% if @cols['alternate_id'      ] == true      %><th valign='bottom'>Alternate ID       </th><% end %>
+      <% if @cols['barcode'           ] == true      %><th valign='bottom'>Barcode            </th><% end %>
+      <% if @cols['price'             ] == true      %><th valign='bottom'>Price              </th><% end %>
+      <% if @cols['quantity_in_stock' ] == true      %><th valign='bottom'>Quantity           </th><% end %>
+      <% if @cols['weight'            ] == true      %><th valign='bottom'>Weight             </th><% end %>
+      <% if @cols['length'            ] == true      %><th valign='bottom'>Length             </th><% end %>
+      <% if @cols['width'             ] == true      %><th valign='bottom'>Width              </th><% end %>
+      <% if @cols['height'            ] == true      %><th valign='bottom'>Height             </th><% end %>
+      <% if @cols['cylinder'          ] == true      %><th valign='bottom'>Cylinder           </th><% end %>
+      <% if @cols['requires_shipping' ] == true      %><th valign='bottom'>Requires Shipping  </th><% end %>
+      <% if @cols['allow_backorder'   ] == true      %><th valign='bottom'>Allow Backorder    </th><% end %>
+      <% if @cols['taxable'           ] == true      %><th valign='bottom'>Taxable            </th><% end %>
+      <th>&nbsp;</th>
+      <th>&nbsp;</th>
+    </tr>
+    <tbody id='variants_tbody'>
+      <% p.variants.where(:status => ['Active', 'Inactive']).each do |v| %>
+        <tr id='variant_<%= v.id %>' <% if @highlight_variant_id && v.id == @highlight_variant_id %>class='highlight'<% end %>>
+          <td class='sort_handle'>&nbsp;</td>
+          <% if @cols['option1'           ] && p.option1 %><td><div id='variant_<%= v.id %>_option1'           ></div></td><% end %>
+          <% if @cols['option2'           ] && p.option2 %><td><div id='variant_<%= v.id %>_option2'           ></div></td><% end %>
+          <% if @cols['option3'           ] && p.option3 %><td><div id='variant_<%= v.id %>_option3'           ></div></td><% end %>
+          <% if @cols['status'            ] == true      %><td><div id='variant_<%= v.id %>_status'            ></div></td><% end %>
+          <% if @cols['alternate_id'      ] == true      %><td><div id='variant_<%= v.id %>_alternate_id'      ></div></td><% end %>
+          <% if @cols['barcode'           ] == true      %><td><div id='variant_<%= v.id %>_barcode'           ></div></td><% end %> 
+          <% if @cols['price'             ] == true      %><td><div id='variant_<%= v.id %>_price'             ></div></td><% end %> 
+          <% if @cols['quantity_in_stock' ] == true      %><td><div id='variant_<%= v.id %>_quantity_in_stock' ></div></td><% end %> 
+          <% if @cols['weight'            ] == true      %><td><div id='variant_<%= v.id %>_weight'            ></div></td><% end %> 
+          <% if @cols['length'            ] == true      %><td><div id='variant_<%= v.id %>_length'            ></div></td><% end %> 
+          <% if @cols['width'             ] == true      %><td><div id='variant_<%= v.id %>_width'             ></div></td><% end %> 
+          <% if @cols['height'            ] == true      %><td><div id='variant_<%= v.id %>_height'            ></div></td><% end %> 
+          <% if @cols['cylinder'          ] == true      %><td><div id='variant_<%= v.id %>_cylinder'          ></div></td><% end %> 
+          <% if @cols['requires_shipping' ] == true      %><td><div id='variant_<%= v.id %>_requires_shipping' ></div></td><% end %>
+          <% if @cols['allow_backorder'   ] == true      %><td><div id='variant_<%= v.id %>_allow_backorder'   ></div></td><% end %>
+          <% if @cols['taxable'           ] == true      %><td><div id='variant_<%= v.id %>_taxable'           ></div></td><% end %>
+          <td class='edit'><a href='/admin/products/<%= p.id %>/variants/<%= v.id %>/edit'>Edit</a></td>
+          <td class='delete'><a href='#' onclick='delete_variant(<%= v.id %>);'>Delete</a></td>
+        </tr>
+      <% end %>
+    </tbody>
+  </table>
 </div>
+
 <p><input type='button' value='New Variant' onclick="add_variant(<%= p.id %>);" /></p>
 
 <%= render :partial => 'caboose_store/products/admin_footer' %>
 
 <% content_for :caboose_css do %>
 <style type='text/css'>
-tr.highlight td { background: #ffff99 } 
+tr.highlight td { background: #ffff99 }
+td.sort_handle { background: #ccc; width: 20px; }
 </style>
 <% end %>
 
@@ -117,7 +121,19 @@ $(document).ready(function() {
       <% if @cols['allow_backorder'   ] == true      %>{ name: 'allow_backorder'    , nice_name: 'Allow backorder'                  , type: 'checkbox' , align: 'center', value: <%= raw Caboose.json(v.allow_backorder     ) %>, width: 50, fixed_placeholder: false } <% end %>     
     ]
   });
-  <% end %>    
+  <% end %>
+  
+  $('#variants_tbody').sortable({
+    handle: '.sort_handle',
+    update: function(event, ui) {
+      $.ajax({
+        url: '/admin/products/<%= p.id %>/variants/sort-order',
+        type: 'put',
+        data: $('#variants_tbody').sortable('serialize', { key: 'variant_ids[]' }),
+        success: function(resp) {}
+      });        
+    }    
+  });
 });
 
 var modal = false;

data/config/routes.rb

@@ -66,7 +66,7 @@ CabooseStore::Engine.routes.draw do
 
   get     "/admin/products"                             => "products#admin_index"
   get     '/admin/products/sort'                        => 'products#admin_sort'
-  put     '/admin/products/update-sort-order'           => 'products#admin_update_sort_order'
+  put     '/admin/products/update-sort-order'           => 'products#admin_update_sort_order'  
   put     "/admin/products/update-vendor-status/:id"    => "products#admin_update_vendor_status"
   get     "/admin/products/new"                         => "products#admin_new"
   get     "/admin/products/status-options"              => "products#admin_status_options"  
@@ -78,7 +78,11 @@ CabooseStore::Engine.routes.draw do
   get     "/admin/products/:id/variants"                => "products#admin_edit_variants"
   get     "/admin/products/:id/variants/json"           => "products#admin_variants_json"
   get     "/admin/products/:id/variant-cols"            => "products#admin_edit_variant_columns"
-  put     "/admin/products/:id/variant-cols"            => "products#admin_update_variant_columns"    
+  put     "/admin/products/:id/variant-cols"            => "products#admin_update_variant_columns"
+  get     "/admin/products/:id/variants/sort-order"     => "products#admin_edit_variant_sort_order"
+  put     '/admin/products/:id/variants/option1-sort-order' => 'products#admin_update_variant_option1_sort_order'
+  put     '/admin/products/:id/variants/option2-sort-order' => 'products#admin_update_variant_option2_sort_order'
+  put     '/admin/products/:id/variants/option3-sort-order' => 'products#admin_update_variant_option3_sort_order'
   get     "/admin/products/:id/images"                  => "products#admin_edit_images"
   post    "/admin/products/:id/images"                  => "products#admin_add_image"
   get     "/admin/products/:id/collections"             => "products#admin_edit_collections"

data/lib/caboose-store/version.rb

@@ -1,3 +1,3 @@
 module CabooseStore
-  VERSION = '0.0.21'
+  VERSION = '0.0.22'
 end

data/lib/tasks/caboose-store.rake

@@ -12,4 +12,16 @@ namespace :caboose_store do
   
   desc "Loads data into caboose tables"
   task :load_data => :environment do CabooseStore::Schema.load_data end
+  
+  desc "Fix variant sort order"
+  task :set_variant_sort_order => :environment do
+    CabooseStore::Product.all.each do |p|
+      puts "Setting sort order for product #{p.id}..."
+      i = 1
+      CabooseStore::Variant.where(:product_id => p.id).reorder(:id).all.each do |v|
+        v.update_attribute('sort_order', i)
+        i = i + 1
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: caboose-store
 version: !ruby/object:Gem::Version
-  version: 0.0.21
+  version: 0.0.22
 platform: ruby
 authors:
 - William Barry
@@ -200,6 +200,7 @@ files:
 - app/views/caboose_store/products/admin_edit_options.html.erb
 - app/views/caboose_store/products/admin_edit_seo.html.erb
 - app/views/caboose_store/products/admin_edit_variant_columns.html.erb
+- app/views/caboose_store/products/admin_edit_variant_sort_order.html.erb
 - app/views/caboose_store/products/admin_edit_variants.html.erb
 - app/views/caboose_store/products/admin_edit_variants_single.html.erb
 - app/views/caboose_store/products/admin_group_variants.html.erb