solidus_product_bundle 1.0.0

data/config/locales/fr.yml

@@ -0,0 +1,12 @@
+---
+fr:
+  spree:
+    available_parts: Parties disponibles
+    can_be_part: Peut faire partie d'un package
+    individual_sale: Vente individuelle
+    no_variants: Pas de variants
+    parts: Parties
+    assembly_cannot_be_part: ensemble ne peut pas être partie
+    product_bundles: Ensembles du produit
+    parts_included: Les pièces comprises
+    part_of_bundle: 'Une partie du %{sku}'

data/config/locales/ru.yml

@@ -0,0 +1,12 @@
+---
+ru:
+  spree:
+    available_parts: Доступные составные части
+    can_be_part: Может входить в состав других продуктов
+    individual_sale: Может продаваться отдельно
+    no_variants: Нет вариантов
+    parts: Составные части
+    assembly_cannot_be_part: сборка не может быть частью
+    product_bundles: Продукт cвязки
+    parts_included: 'Детали, входящие'
+    part_of_bundle: 'Часть пучка %{sku}'

data/config/locales/sv.yml

@@ -0,0 +1,12 @@
+---
+sv:
+  spree:
+    available_parts: Tillgängliga delar
+    can_be_part: Kan vara del
+    individual_sale: Individuell försäljning
+    no_variants: Inga varianter
+    parts: Delar
+    assembly_cannot_be_part: enheten kan inte vara en del
+    product_bundles: Produktpaket
+    parts_included: Delar som ingår
+    part_of_bundle: 'Del av %{sku}'

data/config/routes.rb

@@ -0,0 +1,19 @@
+Spree::Core::Engine.add_routes do
+
+  namespace :admin do
+    resources :products do
+      resources :parts do
+        member do
+          post :select
+          post :remove
+          post :set_count
+        end
+        collection do
+          post :available
+          get  :selected
+        end
+      end
+    end
+  end
+
+end

data/db/migrate/20091028152124_add_many_to_many_relation_to_products.rb

@@ -0,0 +1,13 @@
+class AddManyToManyRelationToProducts < ActiveRecord::Migration
+  def self.up
+    create_table :assemblies_parts, :id => false do |t|
+      t.integer "assembly_id", :null => false
+      t.integer "part_id", :null => false
+      t.integer "count", :null => false, :default => 1
+    end
+  end
+
+  def self.down
+    drop_table :assemblies_parts
+  end
+end

data/db/migrate/20091029165620_add_parts_fields_to_products.rb

@@ -0,0 +1,27 @@
+class AddPartsFieldsToProducts < ActiveRecord::Migration
+  def self.up
+    table = if table_exists?(:products)
+      'products'
+    elsif table_exists?(:spree_products)
+      'spree_products'
+    end 
+    
+    change_table(table) do |t|
+      t.column :can_be_part, :boolean, :default => false, :null => false
+      t.column :individual_sale, :boolean, :default => true, :null => false
+    end  
+  end
+
+  def self.down
+    table = if table_exists?(:products)
+      'products'
+    elsif table_exists?(:spree_products)
+      'spree_products'
+    end 
+    
+    change_table(table) do |t|
+      t.remove :can_be_part
+      t.remove :individual_sale
+    end
+  end
+end

data/db/migrate/20120316141830_namespace_product_assembly_for_spree_one.rb

@@ -0,0 +1,9 @@
+class NamespaceProductAssemblyForSpreeOne < ActiveRecord::Migration
+  def up
+    rename_table :assemblies_parts, :spree_assemblies_parts
+  end
+
+  def down
+    rename_table :spree_assemblies_parts, :assemblies_parts
+  end
+end

data/db/migrate/20140620223938_add_id_to_spree_assemblies_parts.rb

@@ -0,0 +1,9 @@
+class AddIdToSpreeAssembliesParts < ActiveRecord::Migration
+  def up
+    add_column :spree_assemblies_parts, :id, :primary_key
+  end
+
+  def down
+    add_column :spree_assemblies_parts, :id
+  end
+end

data/db/migrate/20150219192418_add_variant_selection_deferred_to_assemblies_parts.rb

@@ -0,0 +1,5 @@
+class AddVariantSelectionDeferredToAssembliesParts < ActiveRecord::Migration
+  def change
+    add_column :spree_assemblies_parts, :variant_selection_deferred, :boolean
+  end
+end

data/db/migrate/20150303105615_create_part_line_items.rb

@@ -0,0 +1,9 @@
+class CreatePartLineItems < ActiveRecord::Migration
+  def change
+    create_table :spree_part_line_items do |t|
+      t.belongs_to :line_item, null: false
+      t.belongs_to :variant, null: false
+      t.integer :quantity, default: 1
+    end
+  end
+end

data/lib/generators/spree_product_assembly/install/install_generator.rb

@@ -0,0 +1,24 @@
+module SpreeProductAssembly
+  module Generators
+    class InstallGenerator < Rails::Generators::Base
+      class_option :auto_run_migrations, :type => :boolean, :default => false
+
+      def add_migrations
+        run 'rake railties:install:migrations FROM=spree_product_assembly'
+      end
+
+      def add_javascripts
+        append_file "vendor/assets/javascripts/spree/backend/all.js", "//= require spree/backend/spree_product_assembly\n"
+      end
+
+      def run_migrations
+        run_migrations = options[:auto_run_migrations] || ['', 'y', 'Y'].include?(ask 'Would you like to run the migrations now? [Y/n]')
+        if run_migrations
+           run 'rake db:migrate'
+         else
+           puts "Skiping rake db:migrate, don't forget to run it!"
+         end
+      end
+    end
+  end
+end

data/lib/spree_product_assembly/engine.rb

@@ -0,0 +1,21 @@
+module SpreeProductAssembly
+  class Engine < Rails::Engine
+    engine_name 'spree_product_assembly'
+
+    config.autoload_paths += %W(#{config.root}/lib)
+
+    def self.activate
+      Dir.glob(File.join(File.dirname(__FILE__), "../../app/**/*_decorator.rb")) do |c|
+        Rails.configuration.cache_classes ? require(c) : load(c)
+      end
+
+      if ::Rails::Engine.subclasses.map(&:name).include? "Spree::Wombat::Engine"
+        Dir.glob(File.join(File.dirname(__FILE__), "../../lib/**/*_serializer.rb")) do |serializer|
+          Rails.env.production? ? require(serializer) : load(serializer)
+        end
+      end
+    end
+
+    config.to_prepare &method(:activate).to_proc
+  end
+end

data/lib/spree_product_assembly.rb

@@ -0,0 +1,4 @@
+require 'spree_core'
+require 'spree_product_assembly/engine'
+require 'coffee_script'
+require 'sass/rails'

data/lib/tasks/spree2_upgrade.rake

@@ -0,0 +1,29 @@
+namespace :spree_product_assembly do
+  desc 'Link legacy inventory units to an order line item'
+  task :upgrade => :environment do
+    shipments = Spree::Shipment.includes(:inventory_units).where("spree_inventory_units.line_item_id IS NULL")
+
+    shipments.each do |shipment|
+      shipment.inventory_units.includes(:variant).group_by(&:variant).each do |variant, units|
+
+        line_item = shipment.order.line_items.detect { |line_item| line_item.variant_id == variant.id }
+
+        unless line_item
+
+          begin
+            master = shipment.order.products.detect { |p| variant.assemblies.include? p }.master
+            supposed_line_item = shipment.order.line_items.detect { |line_item| line_item.variant_id == master.id }
+
+            if supposed_line_item
+              Spree::InventoryUnit.where(id: units.map(&:id)).update_all "line_item_id = #{supposed_line_item.id}"
+            else
+              puts "Couldn't find a matching line item for #{variant.name}"
+            end
+          rescue
+            puts "Couldn't find a matching line item for #{variant.name}"
+          end
+        end
+      end
+    end
+  end
+end

data/solidus_product_bundle.gemspec

@@ -0,0 +1,33 @@
+Gem::Specification.new do |s|
+  s.platform    = Gem::Platform::RUBY
+  s.name        = 'solidus_product_bundle'
+  s.version     = '1.0.0'
+  s.summary     = 'Adds oportunity to make bundle of products to your Spree store'
+  s.description = s.summary
+  s.required_ruby_version = '>= 1.9.3'
+
+  s.author            = 'Sapna Tomar'
+  s.email             = 'ystomar12488@gmail.com'
+  s.homepage          = 'https://github.com/YSTomar/solidus_product_bundle'
+
+  s.files         = `git ls-files`.split("\n")
+  s.test_files    = `git ls-files -- spec/*`.split("\n")
+  s.require_path = 'lib'
+  s.requirements << 'none'
+
+  s.add_dependency "solidus_api", [">= 1.0.0.pre", "< 2"]
+  s.add_dependency "solidus_backend", [">= 1.0.0.pre", "< 2"]
+  s.add_dependency "solidus_core", [">= 1.0.0.pre", "< 2"]
+
+  s.add_development_dependency 'rspec-rails', '~> 3.1.0'
+  s.add_development_dependency 'sqlite3'
+  s.add_development_dependency 'ffaker'
+  s.add_development_dependency 'factory_girl', '~> 4.4'
+  s.add_development_dependency 'coffee-rails', '~> 4.0.0'
+  s.add_development_dependency 'sass-rails', '~> 4.0.0'
+  s.add_development_dependency 'capybara', '~> 2.4'
+  s.add_development_dependency 'poltergeist', '~> 1.6'
+  s.add_development_dependency 'database_cleaner', '~> 1.3'
+  s.add_development_dependency 'simplecov'
+  s.add_development_dependency 'pg'
+end

data/spec/features/adding_items_to_cart_spec.rb

@@ -0,0 +1,203 @@
+require "spec_helper"
+
+describe "Adding items to the cart", type: :feature do
+  context "when adding a bundle to the cart" do
+    context "when none of the bundle items are packs or have options" do
+      specify "the cart lists the contents of the bundle" do
+        bundle = create(:product_in_stock, name: "Bundle", sku: "BUNDLE")
+
+        keychain = create(:product_in_stock, name: "Keychain",
+                                             sku: "KEYCHAIN",
+                                             can_be_part: true)
+        shirt = create(:product_in_stock, name: "Shirt",
+                                          sku: "SHIRT",
+                                          can_be_part: true)
+
+        add_part_to_bundle(bundle, keychain.master)
+        add_part_to_bundle(bundle, shirt.master)
+
+        visit spree.product_path(bundle)
+
+        click_button "add-to-cart-button"
+
+        within("#cart-detail") do
+          within("tbody tr:first-child") do
+            expect(page).to have_content(bundle.name)
+            expect(page).to have_css("input[value='1']")
+            expect(page).to have_content("(1) Keychain (KEYCHAIN)")
+            expect(page).to have_content("(1) Shirt (SHIRT)")
+          end
+        end
+      end
+    end
+
+    context "when one of the variants is a pack" do
+      specify "the cart displays the same quantity for part line items" do
+        bundle = create(:product_in_stock, name: "Bundle", sku: "BUNDLE")
+
+        keychain = create(:product_in_stock, name: "Keychain",
+                                             sku: "KEYCHAIN",
+                                             can_be_part: true)
+        _shirt, shirts_by_size = create_bundle_product_with_options(
+          name: "Shirt",
+          sku: "SHIRT",
+          option_type: "Size",
+          option_values: ["Small"]
+        )
+
+        add_part_to_bundle(bundle, keychain.master, count: 2)
+        add_part_to_bundle(bundle, shirts_by_size["small"])
+
+        visit spree.product_path(bundle)
+
+        click_button "add-to-cart-button"
+
+        within("#cart-detail tbody tr:first-child") do
+          expect(page).to have_content(bundle.name)
+          expect(page).to have_css("input[value='1']")
+          expect(page).to have_content("(2) Keychain (KEYCHAIN)")
+          expect(page).to have_content("(1) Shirt (Size: Small) (SHIRT-SMALL)")
+        end
+      end
+
+      context "when ordering more than one of the bundle" do
+        specify "the part quantity is multiplied by the bundle quantity" do
+          bundle = create(:product_in_stock, name: "Bundle", sku: "BUNDLE")
+
+          keychain = create(:product_in_stock, name: "Keychain",
+                                               sku: "KEYCHAIN",
+                                               can_be_part: true)
+          shirt = create(:product_in_stock, name: "Shirt",
+                                            sku: "SHIRT",
+                                            can_be_part: true)
+
+          add_part_to_bundle(bundle, keychain.master, count: 2)
+          add_part_to_bundle(bundle, shirt.master)
+
+          visit spree.product_path(bundle)
+
+          fill_in "quantity", with: 2
+
+          click_button "add-to-cart-button"
+
+          within("#cart-detail tbody tr:first-child") do
+            expect(page).to have_content(bundle.name)
+            expect(page).to have_css("input[value='2']")
+            expect(page).to have_content("(4) Keychain (KEYCHAIN)")
+            expect(page).to have_content("(2) Shirt (SHIRT)")
+          end
+        end
+      end
+    end
+
+    context "when a bundle items has a variant (that is not user-selectable)" do
+      specify "the cart includes the variant when listing items bundle items" do
+        bundle = create(:product_in_stock, name: "Bundle", sku: "BUNDLE")
+
+        keychain = create(:product_in_stock, name: "Keychain",
+                                             sku: "KEYCHAIN",
+                                             can_be_part: true)
+        _shirt, shirts_by_size = create_bundle_product_with_options(
+          name: "Shirt",
+          sku: "SHIRT",
+          option_type: "Size",
+          option_values: ["Small"]
+        )
+
+        add_part_to_bundle(bundle, keychain.master)
+        add_part_to_bundle(bundle, shirts_by_size["small"])
+
+        visit spree.product_path(bundle)
+
+        click_button "add-to-cart-button"
+
+        within("#cart-detail tbody tr:first-child") do
+          expect(page).to have_content(bundle.name)
+          expect(page).to have_css("input[value='1']")
+          expect(page).to have_content("(1) Keychain (KEYCHAIN)")
+          expect(page).to have_content("(1) Shirt (Size: Small) (SHIRT-SMALL)")
+        end
+      end
+    end
+
+    context "when one of the bundle items has a user-selectable variant" do
+      specify "the cart includes the variant when listing bundle items" do
+        bundle = create(:product_in_stock, name: "Bundle", sku: "BUNDLE")
+
+        keychain = create(:product_in_stock, name: "Keychain",
+                                             sku: "KEYCHAIN",
+                                             can_be_part: true)
+        shirt, _shirts_by_size = create_bundle_product_with_options(
+          name: "Shirt",
+          sku: "SHIRT",
+          option_type: "Size",
+          option_values: ["Small", "Medium"]
+        )
+
+        add_part_to_bundle(bundle, keychain.master, count: 1)
+        add_part_to_bundle(
+          bundle,
+          shirt.master,
+          variant_selection_deferred: true
+        )
+
+        visit spree.product_path(bundle)
+
+        select "Size: Medium", from: "Variant"
+
+        click_button "add-to-cart-button"
+
+        within("#cart-detail tbody tr:first-child") do
+          expect(page).to have_content(bundle.name)
+          expect(page).to have_css("input[value='1']")
+          expect(page).to have_content("(1) Keychain (KEYCHAIN)")
+          expect(page).to(
+            have_content("(1) Shirt (Size: Medium) (SHIRT-MEDIUM)")
+          )
+        end
+      end
+    end
+  end
+
+  def create_bundle_product_with_options(args)
+    option_type_presentation = args.fetch(:option_type)
+    option_value_presentations = args.fetch(:option_values)
+    option_values = option_value_presentations.map do |presentation|
+      create(:option_value, presentation: presentation)
+    end
+    option_type = create(:option_type,
+                         presentation: option_type_presentation,
+                         name: option_type_presentation.downcase,
+                         option_values: option_values)
+    product_attributes = args.slice(:name, :sku).merge(
+      option_types: [option_type],
+      can_be_part: true
+    )
+    product = create(:product, product_attributes)
+
+    variants = variants_by_option(product, option_values)
+
+    [product, variants]
+  end
+
+  def variants_by_option(product, option_values)
+    option_values.each_with_object({}) do |value, hash|
+      hash[value.presentation.downcase] = create(
+        :variant_in_stock,
+        product: product,
+        sku: "#{product.sku}-#{value.presentation.upcase}",
+        option_values: [value]
+      )
+    end
+  end
+
+  def add_part_to_bundle(bundle, variant, options = {})
+    attributes = options.reverse_merge(
+      assembly_id: bundle.id,
+      part_id: variant.id,
+    )
+    create(:assemblies_part, attributes).tap do |_part|
+      bundle.reload
+    end
+  end
+end

data/spec/features/admin/orders_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe "Orders", type: :feature, js: true do
+  stub_authorization!
+
+  let(:order) { create(:order_with_line_items) }
+  let(:line_item) { order.line_items.first }
+  let(:bundle) { line_item.product }
+  let(:parts) { (1..3).map { create(:variant) } }
+
+  before do
+    bundle.parts << [parts]
+    line_item.update_attributes!(quantity: 3)
+    order.reload.create_proposed_shipments
+    order.finalize! 
+  end
+
+  it "allows admin to edit product bundle" do
+    visit spree.edit_admin_order_path(order)
+
+    within("table.product-bundles") do
+      find(".edit-line-item").click
+      fill_in "quantity", :with => "2"
+      find(".save-line-item").click
+
+      sleep(1) # avoid odd "cannot rollback - no transaction is active: rollback transaction"
+    end
+  end
+end

data/spec/features/admin/parts_spec.rb

@@ -0,0 +1,183 @@
+require 'spec_helper'
+
+describe "Managing parts for a product bundle", type: :feature, js: true do
+  stub_authorization!
+
+  let!(:tshirt) { create(:product, :name => "T-Shirt") }
+  let!(:mug) { create(:product, :name => "Mug", can_be_part: true) }
+
+  context "when searching for parts" do
+    before do
+      visit spree.admin_product_path(tshirt)
+      click_on "Parts"
+    end
+
+    it "returns empty results when there is no query" do
+      fill_in "searchtext", with: ""
+      click_on "Search"
+
+      page.should have_content("No Match Found.")
+    end
+
+    it "displays no-match feedback when it does not find any products" do
+      fill_in "searchtext", with: "Foo"
+      click_on "Search"
+
+      page.should have_content("No Match Found.")
+    end
+
+    it "shows any products that were found" do
+      fill_in "searchtext", with: mug.name
+      click_on "Search"
+
+      page.should have_content(mug.name)
+    end
+  end
+
+  context "when adding parts to a bundle" do
+    it "allows adding a product with no variants" do
+      visit spree.admin_product_path(tshirt)
+      click_on "Parts"
+      fill_in "searchtext", with: mug.name
+      click_on "Search"
+
+      within("#search_hits") { click_on "Select" }
+      page.should have_content(mug.sku)
+    end
+
+    context "when a part has multiple variants" do
+      def build_option(options)
+        option_type_name = options.fetch(:type)
+        option_type = create(:option_type,
+          presentation: option_type_name,
+          name: option_type_name
+        )
+        option_value = options.fetch(:value)
+        option_type.option_values.create(
+          name: option_value.downcase,
+          presentation: option_value
+        )
+
+        option_type
+      end
+
+      def build_part_with_options(product_name, option_type)
+        product = create(:product,
+          can_be_part: true,
+          name: product_name,
+          option_types: [option_type]
+        )
+        create(:variant,
+          product: product,
+          option_values: option_type.option_values
+        )
+      end
+
+     it "allows a specific variant to be selected as part of the bundle" do
+        bundle = create(:product)
+        option = build_option(type: "Color", value: "Red")
+        part = build_part_with_options("Shirt", option)
+
+        visit spree.admin_product_path(bundle)
+        click_on "Parts"
+        fill_in "searchtext", with: "Shirt"
+        click_on "Search"
+
+        within("#search_hits") do
+          select "Color: Red", from: "part_id"
+          click_on "Select"
+        end
+
+        page.should have_content(part.sku)
+      end
+
+      it "allows admin to specify that user can select any variant" do
+        bundle = create(:product)
+        option = build_option(type: "Color", value: "Red")
+        part = build_part_with_options("Shirt", option)
+
+        visit spree.admin_product_path(bundle)
+        click_on "Parts"
+        fill_in "searchtext", with: "Shirt"
+        click_on "Search"
+
+        within("#search_hits") do
+          select Spree.t(:user_selectable), from: "part_id"
+          fill_in "part_count", with: 666
+          click_on "Select"
+        end
+
+        within("#product_parts") do
+          page.should have_content("Shirt")
+          page.should have_content(part.product.sku)
+          page.should have_content(Spree.t(:user_selectable))
+
+          input = find_field("count")
+          input[:value].should eq("666")
+        end
+      end
+    end
+  end
+
+  it "allows parts to be removed from the bundle" do
+    visit spree.admin_product_path(tshirt)
+    click_on "Parts"
+    fill_in "searchtext", with: mug.name
+    click_on "Search"
+
+    within("#search_hits") { click_on "Select" }
+    page.should have_content(mug.sku)
+
+    within("#product_parts") do
+      find(".remove_admin_product_part_link").click
+
+      page.should_not have_content(mug.sku)
+    end
+  end
+
+  context "when updating part quantities" do
+    before do
+      visit spree.admin_product_path(tshirt)
+      click_on "Parts"
+      fill_in "searchtext", with: mug.name
+      click_on "Search"
+      within("#search_hits") { click_on "Select" }
+    end
+
+    it "updates the quantity to match the newly-supplied value" do
+      within("#product_parts") do
+        fill_in "count", with: "5"
+        find(".set_count_admin_product_part_link").click
+
+        expect(find_field('count').value).to eq "5"
+      end
+    end
+
+    it "rejects a negative quantity" do
+      within("#product_parts") do
+        fill_in "count", with: "-1"
+        find(".set_count_admin_product_part_link").click
+      end
+
+      expect(page).to have_content("Quantity must be greater than 0")
+    end
+
+    it "rejects a part quantity of `0`" do
+      within("#product_parts") do
+        fill_in "count", with: "0"
+        find(".set_count_admin_product_part_link").click
+      end
+
+      expect(page).to have_content("Quantity must be greater than 0")
+    end
+
+    it "rejects a non-numeric part quantity" do
+      within("#product_parts") do
+        fill_in "count", with: "non-numeric"
+        find(".set_count_admin_product_part_link").click
+      end
+
+      expect(page).to have_content("Quantity must be greater than 0")
+    end
+  end
+end