stall 0.2.0 → 0.3.1

data/db/migrate/20161129101956_add_type_to_stall_address_ownerships.rb

@@ -0,0 +1,52 @@
+# Before this migration, the AddressOwnership class is removed so we declare it
+# here in its minimal version as a temporary fallback allowing us to easily
+# migrate our data to the new address direct-relation model
+#
+class AddressOwnership < ActiveRecord::Base
+  self.table_name = 'stall_address_ownerships'
+  belongs_to :address
+  belongs_to :addressable, polymorphic: true
+end
+
+class AddTypeToStallAddressOwnerships < ActiveRecord::Migration
+  def up
+    change_table :stall_addresses do |t|
+      t.string :type
+      t.references :addressable, polymorphic: true
+    end
+
+    AddressOwnership.find_each do |ownership|
+      address = ownership.address
+      address.addressable = ownership.addressable
+
+      if ownership.shipping
+        address.type = 'ShippingAddress'
+        address.save!
+
+        if ownership.billing
+          billing_address = BillingAddress.new(addressable: ownership.addressable)
+          Stall::Addresses::Copy.new(address, billing_address)
+          billing_address.save!
+        end
+      elsif ownership.billing
+        address.type = 'BillingAddress'
+        ownership.save!
+      else
+        ownership.destroy
+      end
+    end
+
+    remove_column :stall_address_ownerships, :billing
+    remove_column :stall_address_ownerships, :shipping
+  end
+
+  def down
+    add_column :stall_address_ownerships, :billing, :boolean
+    add_column :stall_address_ownerships, :shipping, :boolean
+
+    change_table :stall_addresses do |t|
+      t.remove :type
+      t.remove_references :addressable, polymorphic: true
+    end
+  end
+end

data/db/migrate/20161202080218_add_reference_to_product_lists.rb

@@ -0,0 +1,17 @@
+class AddReferenceToProductLists < ActiveRecord::Migration
+  def up
+    add_column :stall_product_lists, :reference, :string
+    add_index :stall_product_lists, :reference, unique: true
+
+    # Migrate all references stored in the JSON data columns to the new
+    # reference column
+    Cart.update_all("reference = data->>'reference'")
+    # Remove all reference keys in the JSON data columns
+    Cart.update_all("data = (data - 'reference')")
+  end
+
+  def down
+    remove_index :stall_product_lists, :reference
+    remove_column :stall_product_lists, :reference
+  end
+end

data/db/migrate/20170118103916_create_credit_notes.rb

@@ -0,0 +1,17 @@
+class CreateCreditNotes < ActiveRecord::Migration
+  def change
+    create_table :stall_credit_notes do |t|
+      t.string     :reference
+      t.references :customer,    index: true
+      t.string     :currency
+      t.monetize   :eot_amount,  null: false,        currency: { present: false }
+      t.monetize   :amount,      null: false,        currency: { present: false }
+      t.decimal    :vat_rate,    precision: 11,      scale: 2
+      t.references :source,      polymorphic: true,  index: true
+
+      t.timestamps null: false
+    end
+
+    add_foreign_key :stall_credit_notes, :stall_customers, column: :customer_id
+  end
+end

data/db/migrate/20170118144047_create_credit_note_adjustments.rb

@@ -0,0 +1,13 @@
+class CreateCreditNoteAdjustments < ActiveRecord::Migration
+  def change
+    create_table :stall_credit_note_usages do |t|
+      t.references :credit_note, index: true
+      t.references :adjustment, index: true
+
+      t.timestamps null: false
+    end
+
+    add_foreign_key :stall_credit_note_usages, :stall_credit_notes, column: :credit_note_id
+    add_foreign_key :stall_credit_note_usages, :stall_adjustments, column: :adjustment_id
+  end
+end

data/db/migrate/20170123123115_create_stall_product_categories.rb

@@ -0,0 +1,12 @@
+class CreateStallProductCategories < ActiveRecord::Migration
+  def change
+    create_table :stall_product_categories do |t|
+      t.string :name
+      t.text :slug
+      t.integer :position, default: 0
+      t.integer :parent_id
+
+      t.timestamps null: false
+    end
+  end
+end

data/db/migrate/20170123123326_create_stall_products.rb

@@ -0,0 +1,17 @@
+class CreateStallProducts < ActiveRecord::Migration
+  def change
+    create_table :stall_products do |t|
+      t.string :name
+      t.attachment :image
+      t.text :description
+      t.text :slug
+      t.integer :position
+      t.boolean :visible, default: true
+      t.references :product_category, index: true
+
+      t.timestamps null: false
+    end
+
+    add_foreign_key :stall_products, :stall_product_categories, column: 'product_category_id'
+  end
+end

data/db/migrate/20170123125030_create_stall_variants.rb

@@ -0,0 +1,13 @@
+class CreateStallVariants < ActiveRecord::Migration
+  def change
+    create_table :stall_variants do |t|
+      t.references :product, index: true
+      t.monetize :price
+      t.boolean :published, default: true
+
+      t.timestamps null: false
+    end
+
+    add_foreign_key :stall_variants, :stall_products, column: 'product_id'
+  end
+end

data/db/migrate/20170123131748_create_stall_product_category_hierarchies.rb

@@ -0,0 +1,16 @@
+class CreateStallProductCategoryHierarchies < ActiveRecord::Migration
+  def change
+    create_table :stall_product_category_hierarchies, id: false do |t|
+      t.integer :ancestor_id, null: false
+      t.integer :descendant_id, null: false
+      t.integer :generations, null: false
+    end
+
+    add_index :stall_product_category_hierarchies, [:ancestor_id, :descendant_id, :generations],
+      unique: true,
+      name: "stall_product_category_anc_desc_idx"
+
+    add_index :stall_product_category_hierarchies, [:descendant_id],
+      name: "stall_product_category_desc_idx"
+  end
+end

data/db/migrate/20170123143704_create_stall_product_details.rb

@@ -0,0 +1,14 @@
+class CreateStallProductDetails < ActiveRecord::Migration
+  def change
+    create_table :stall_product_details do |t|
+      t.references :product, index: true
+      t.string :name
+      t.text :content
+      t.integer :position, default: 0
+
+      t.timestamps null: false
+    end 
+
+    add_foreign_key :stall_product_details, :stall_products, column: 'product_id'
+  end
+end

data/db/migrate/20170125152622_convert_all_money_fields_to_decimal_to_use_infinite_precision.rb

@@ -0,0 +1,27 @@
+class ConvertAllMoneyFieldsToDecimalToUseInfinitePrecision < ActiveRecord::Migration
+  def up
+    change_column :stall_adjustments, :price_cents, :decimal, precision: 13, scale: 3
+    change_column :stall_adjustments, :eot_price_cents, :decimal, precision: 13, scale: 3
+    change_column :stall_credit_notes, :eot_amount_cents, :decimal, precision: 13, scale: 3
+    change_column :stall_credit_notes, :amount_cents, :decimal, precision: 13, scale: 3
+    change_column :stall_line_items, :unit_eot_price_cents, :decimal, precision: 13, scale: 3
+    change_column :stall_line_items, :unit_price_cents, :decimal, precision: 13, scale: 3
+    change_column :stall_line_items, :eot_price_cents, :decimal, precision: 13, scale: 3
+    change_column :stall_line_items, :price_cents, :decimal, precision: 13, scale: 3
+    change_column :stall_shipments, :price_cents, :decimal, precision: 13, scale: 3
+    change_column :stall_shipments, :eot_price_cents, :decimal, precision: 13, scale: 3
+  end
+
+  def down
+    change_column :stall_adjustments, :price_cents, :integer
+    change_column :stall_adjustments, :eot_price_cents, :integer
+    change_column :stall_credit_notes, :eot_amount_cents, :integer
+    change_column :stall_credit_notes, :amount_cents, :integer
+    change_column :stall_line_items, :unit_eot_price_cents, :integer
+    change_column :stall_line_items, :unit_price_cents, :integer
+    change_column :stall_line_items, :eot_price_cents, :integer
+    change_column :stall_line_items, :price_cents, :integer
+    change_column :stall_shipments, :price_cents, :integer
+    change_column :stall_shipments, :eot_price_cents, :integer
+  end
+end

data/db/migrate/20170131162537_add_data_to_stall_adjustments.rb

@@ -0,0 +1,5 @@
+class AddDataToStallAdjustments < ActiveRecord::Migration
+  def change
+    add_column :stall_adjustments, :data, :jsonb
+  end
+end

data/db/migrate/20170202165514_create_stall_properties.rb

@@ -0,0 +1,9 @@
+class CreateStallProperties < ActiveRecord::Migration
+  def change
+    create_table :stall_properties do |t|
+      t.string :name
+
+      t.timestamps null: false
+    end
+  end
+end

data/db/migrate/20170202165516_create_stall_property_values.rb

@@ -0,0 +1,13 @@
+class CreateStallPropertyValues < ActiveRecord::Migration
+  def change
+    create_table :stall_property_values do |t|
+      t.references :property, index: true
+      t.string :value
+      t.integer :position, default: 0
+
+      t.timestamps null: false
+    end
+
+    add_foreign_key :stall_property_values, :stall_properties, column: :property_id
+  end
+end

data/db/migrate/20170202165518_create_stall_variant_property_values.rb

@@ -0,0 +1,13 @@
+class CreateStallVariantPropertyValues < ActiveRecord::Migration
+  def change
+    create_table :stall_variant_property_values do |t|
+      t.references :property_value, index: true
+      t.references :variant, index: true
+
+      t.timestamps null: false
+    end
+
+    add_foreign_key :stall_variant_property_values, :stall_property_values, column: :property_value_id
+    add_foreign_key :stall_variant_property_values, :stall_variants, column: :variant_id
+  end
+end

data/lib/generators/stall/install/templates/initializer.rb

@@ -25,6 +25,20 @@ Stall.configure do |config|
   #
   # config.sender_email = ENV['STALL_SENDER_EMAIL']
 
+  # Set the customer associated user model used by the shop to bind user
+  # accounts to.
+  #
+  # This model should have an e-mail, password and password_confirmation
+  # fields.
+  #
+  # config.default_user_model_name = 'User'
+
+  # A method available in all controllers to fetch the current signed in user
+  # in your app. It should return a user model if the visitor is signed in, and
+  # nil if it's unsigned.
+  #
+  # config.default_user_helper_method = :current_user
+
   # Global default VAT rate, can be overrided by products
   #
   # config.vat_rate = BigDecimal.new('20.0')
@@ -94,6 +108,13 @@ Stall.configure do |config|
   #
   # config.default_product_weight = 0
 
+  # When a cart total balance is negative, automatically convert the remaining
+  # amount into a credit note for the cart customer
+  #
+  # Defaults to false, losing the remainder for the customer when paid
+  #
+  # config.convert_cart_remainder_to_credit_note = false
+
   # Allows configuring which countries are available for free shipping offers
   #
   # Accepts an array of country codes or a proc with an Address argument

data/lib/generators/stall/view/view_generator.rb

@@ -1,37 +1,59 @@
 module Stall
   class ViewGenerator < ::Rails::Generators::Base
-    class ViewNotFound < StandardError; end
-
     VIEWS_DIR = File.expand_path('../../../../../app/views', __FILE__)
     source_root VIEWS_DIR
 
     argument :file_paths, type: :array, required: true
 
     def copy_view_template
-      paths = file_paths.map do |file_path|
-        source_path = source_path_for(file_path)
-        raise ViewNotFound, "Could not find any stall view for #{ file_path }" unless File.exist?(source_path)
-        target_path = "app/views/#{ file_path_with_ext_for(file_path) }"
+      paths = file_paths.map { |file_path| ViewTemplate.new(file_path) }
 
-        [source_path, target_path]
+      paths.each do |view_template|
+        template(view_template.source_path, view_template.target_path)
       end
+    end
+
+
+    class ViewTemplate
+      class ViewNotFound < StandardError; end
 
-      paths.each do |source_path, target_path|
-        template(source_path, target_path)
+      attr_reader :file_path
+
+      def initialize(file_path)
+        @file_path = file_path
       end
-    end
 
-    private
+      def source_path
+        source_file_for(file_path_with_ext)
+      end
 
-    def source_path_for(file_path)
-      File.join(VIEWS_DIR, file_path_with_ext_for(file_path))
-    end
+      def target_path
+        "app/views/#{ file_path_with_ext }"
+      end
+
+      private
+
+      def file_path_with_ext
+        return @file_path_with_ext if @file_path_with_ext
+        @file_path += '.html.haml' unless file_path.match(/\.html\.haml\z/)
+
+        if File.exist?(source_file_for(file_path))
+          @file_path_with_ext = file_path
+          return @file_path_with_ext
+        end
+
+        partial_path = [File.dirname(file_path), File.basename(file_path)].join('/_')
+
+        if File.exist?(source_file_for(partial_path))
+          @file_path_with_ext = partial_path
+          return @file_path_with_ext
+        end
+
+        raise ViewNotFound, "No Stall view was found for #{ file_path } !"
+      end
 
-    def file_path_with_ext_for(file_path)
-      if file_path.match(/\.html\.haml\z/)
-        file_path
-      else
-        "#{ file_path }.html.haml"
+      def source_file_for(path)
+        File.join(Stall::ViewGenerator::VIEWS_DIR, path)
       end
     end
   end

data/lib/para/stall.rb

@@ -0,0 +1,32 @@
+require 'para'
+require 'stall'
+require 'simple_form'
+
+require 'para/stall/inputs'
+
+module Para
+  module Stall
+    extend ActiveSupport::Autoload
+
+    autoload :Routes
+    autoload :VariantsPropertyConfig
+
+    class << self
+      def config(&block)
+        if block
+          block.call(self)
+        else
+          self
+        end
+      end
+
+      def javascript_includes
+        %w(para/stall)
+      end
+
+      def stylesheet_includes
+        %w(para/stall)
+      end
+    end
+  end
+end

data/lib/para/stall/inputs.rb

@@ -0,0 +1,13 @@
+module Para
+  module Stall
+    module Inputs
+      extend ActiveSupport::Autoload
+
+      autoload :VariantInputHelper
+      autoload :VariantsMatrixInput
+      autoload :VariantSelectInput
+    end
+  end
+end
+
+SimpleForm.custom_inputs_namespaces << 'Para::Stall::Inputs'

data/lib/para/stall/inputs/variant_input_helper.rb

@@ -0,0 +1,34 @@
+module Para
+  module Stall
+    module Inputs
+      module VariantInputHelper
+        private
+
+        def resource
+          @resource ||= @builder.object
+        end
+
+        def model
+          @model ||= resource.class
+        end
+
+        def variant_sort_method(variant)
+          properties.map do |property_config|
+            if (variant_value = property_config.variant_property_value_for(variant))
+              variant_value.property_value.position
+            end
+          end.join(':')
+        end
+
+        # Raises a comprehensive error for easy wrong form attribute catching
+        #
+        def ensure_target_relation_present!
+          unless model.reflect_on_association(attribute_name)
+            raise NoMethodError,
+              "Relation ##{ attribute_name } does not exist for model #{ model.name }."
+          end
+        end
+      end
+    end
+  end
+end