stall 0.2.0 → 0.3.1

data/app/models/credit_note.rb

@@ -0,0 +1,3 @@
+class CreditNote < ActiveRecord::Base
+  include Stall::Models::CreditNote
+end

data/app/models/credit_note_adjustment.rb

@@ -0,0 +1,3 @@
+class CreditNoteAdjustment < Adjustment
+  include Stall::Models::CreditNoteAdjustment
+end

data/app/models/credit_note_usage.rb

@@ -0,0 +1,3 @@
+class CreditNoteUsage < ActiveRecord::Base
+  include Stall::Models::CreditNoteUsage
+end

data/app/models/product.rb

@@ -0,0 +1,3 @@
+class Product < ActiveRecord::Base
+  include Stall::Models::Product  
+end

data/app/models/product_category.rb

@@ -0,0 +1,3 @@
+class ProductCategory < ActiveRecord::Base
+  include Stall::Models::ProductCategory  
+end

data/app/models/product_detail.rb

@@ -0,0 +1,3 @@
+class ProductDetail < ActiveRecord::Base
+  include Stall::Models::ProductDetail
+end

data/app/models/property.rb

@@ -0,0 +1,3 @@
+class Property < ActiveRecord::Base
+  include Stall::Models::Property
+end

data/app/models/property_value.rb

@@ -0,0 +1,3 @@
+class PropertyValue < ActiveRecord::Base
+  include Stall::Models::PropertyValue
+end

data/app/models/shipping_address.rb

@@ -0,0 +1,2 @@
+class ShippingAddress < Address
+end

data/app/models/stall/models/address.rb

@@ -6,7 +6,7 @@ module Stall
       included do
         self.table_name = 'stall_addresses'
 
-        has_one :address_ownership, dependent: :destroy
+        belongs_to :addressable, polymorphic: true
 
         enum civility: { :m => 1, :mme => 2 }
 
@@ -17,7 +17,7 @@ module Stall
         end
 
         def civility_name
-          I18n.t("stall.addresses.civilities.#{ civility }")
+          I18n.t("stall.addresses.civilities.#{ civility }") if civility.present?
         end
 
         def country_name

data/app/models/stall/models/cart.rb

@@ -6,12 +6,8 @@ module Stall
       included do
         include Stall::Addressable
         include Stall::Payable
-
-        has_one :shipment, dependent: :destroy, inverse_of: :cart
-        accepts_nested_attributes_for :shipment
-
-        has_many :adjustments, dependent: :destroy, inverse_of: :cart
-        accepts_nested_attributes_for :adjustments
+        include Stall::Shippable
+        include Stall::Adjustable
 
         attr_accessor :terms
       end
@@ -25,15 +21,6 @@ module Stall
       def active?
         !paid?
       end
-
-      private
-
-      def items
-        items = line_items.to_a
-        items << shipment if shipment
-        items += adjustments.to_a
-        items
-      end
     end
   end
 end

data/app/models/stall/models/cart_credit_note_adjustment.rb

@@ -0,0 +1,11 @@
+module Stall
+  module Models
+    module CartCreditNoteAdjustment
+      extend ActiveSupport::Concern
+
+      included do
+        has_one :credit_note, as: :source, dependent: :nullify
+      end
+    end
+  end
+end

data/app/models/stall/models/credit_note.rb

@@ -0,0 +1,50 @@
+module Stall
+  module Models
+    module CreditNote
+      extend ActiveSupport::Concern
+
+      included do
+        self.table_name = 'stall_credit_notes'
+
+        include Stall::Priceable
+        include Stall::DefaultCurrencyManager
+        include Stall::ReferenceManager
+
+        monetize :eot_amount_cents, :amount_cents,
+                 with_model_currency: :currency, allow_nil: true
+
+        belongs_to :customer
+        belongs_to :source, polymorphic: true
+
+        has_many :credit_note_usages, dependent: :destroy
+        has_many :adjustments, through: :credit_note_usages
+
+        validates :amount, :customer, presence: true
+
+        def amount_with_eot_management=(value)
+          (self.amount_without_eot_management = value).tap do
+            self.eot_amount = amount / vat_coefficient
+          end
+        end
+
+        # TODO : Check if we can use Module#prepend here without getting too
+        #        complex
+        #
+        alias_method :amount_without_eot_management=, :amount=
+        alias_method :amount=, :amount_with_eot_management=
+      end
+
+      def remaining_amount
+        amount - adjustments.map(&:price).sum.abs
+      end
+
+      def vat_rate
+        read_attribute(:vat_rate) || write_attribute(:vat_rate, Stall.config.vat_rate)
+      end
+
+      def with_remaining_money?
+        remaining_amount.to_d > 0
+      end
+    end
+  end
+end

data/app/models/stall/models/credit_note_adjustment.rb

@@ -0,0 +1,13 @@
+module Stall
+  module Models
+    module CreditNoteAdjustment
+      extend ActiveSupport::Concern
+
+      included do
+        has_one :credit_note_usage, foreign_key: :adjustment_id,
+                                    dependent: :destroy
+        has_one :credit_note, through: :credit_note_usage
+      end
+    end
+  end
+end

data/app/models/stall/models/credit_note_usage.rb

@@ -0,0 +1,16 @@
+# Joint model between credit notes and adjustments
+#
+module Stall
+  module Models
+    module CreditNoteUsage
+      extend ActiveSupport::Concern
+
+      included do
+        self.table_name = 'stall_credit_note_usages'
+
+        belongs_to :credit_note
+        belongs_to :adjustment, class_name: 'CreditNoteAdjustment'
+      end
+    end
+  end
+end

data/app/models/stall/models/customer.rb

@@ -8,23 +8,35 @@ module Stall
 
         include Stall::Addressable
 
-        belongs_to :user, polymorphic: true, inverse_of: :customer
-        accepts_nested_attributes_for :user
+        if Stall.config.default_user_model
+          belongs_to :user, polymorphic: true, inverse_of: :customer
+          accepts_nested_attributes_for :user
 
-        has_many :product_lists, dependent: :destroy
+          before_validation :ensure_user_email
+        end
 
-        validates :email, format: { with: /\A[^@\s]+@([^@\s]+\.)+[^@\W]+\z/ },
-                          allow_blank: true
+        has_many :product_lists, dependent: :destroy
+        has_many :credit_notes, dependent: :destroy
 
-        before_validation :ensure_user_email
+        validates :email, presence: true,
+                          format: { with: /\A[^@\s]+@([^@\s]+\.)+[^@\W]+\z/ }
 
         def user_or_default
           user || build_user
         end
 
         def build_user(attributes = {})
-          attributes.reverse_merge!(customer: self)
-          self.user = Stall.config.default_user_model.new(attributes)
+          (self.user = Stall.config.default_user_model.new(attributes)).tap do
+            user.customer = self if user.respond_to?(:customer)
+          end if Stall.config.default_user_model
+        end
+
+        def credit(currency = Stall.config.default_currency)
+          credit_notes.for_currency(currency).map(&:remaining_amount).sum
+        end
+
+        def credit?(currency = Stall.config.default_currency)
+          credit(currency).to_d > 0
         end
 
         private

data/app/models/stall/models/line_item.rb

@@ -27,6 +27,7 @@ module Stall
 
         validate  :stock_availability
 
+        before_validation :restore_valid_quantity
         before_validation :refresh_total_prices
 
         scope :ordered, -> { order(created_at: :asc) }
@@ -52,6 +53,14 @@ module Stall
         self.eot_price = unit_eot_price * quantity if unit_eot_price && quantity
         self.price     = unit_price     * quantity if unit_price     && quantity
       end
+
+      # Ensures that a quantity set to 0 to an existing line item doesn't return
+      # an error.
+      def restore_valid_quantity
+        if persisted? && quantity && (quantity < 1) && quantity_changed?
+          restore_quantity!
+        end
+      end
     end
   end
 end

data/app/models/stall/models/product.rb

@@ -0,0 +1,45 @@
+module Stall
+  module Models
+    module Product
+      extend ActiveSupport::Concern
+
+      included do
+        self.table_name = 'stall_products'
+
+        acts_as_orderable
+        extend FriendlyId
+        friendly_id :name, use: [:slugged, :finders]
+
+        belongs_to :product_category
+
+        has_many :variants, dependent: :destroy, inverse_of: :product
+        accepts_nested_attributes_for :variants, allow_destroy: true
+
+        has_many :product_details, dependent: :destroy, inverse_of: :product
+        accepts_nested_attributes_for :product_details, allow_destroy: true
+
+        has_attached_file :image, styles: {
+          thumb: '100x100#',
+          show: '555x'
+        }
+
+        validates :name, :image, presence: true
+        validates_attachment :image, content_type: { content_type: /\Aimage\/.*\z/ }
+
+        scope :visible, -> { where(visible: true) }
+
+        def should_generate_new_friendly_id?
+          slug.blank?
+        end
+
+        def vat_rate
+          Stall.config.vat_rate
+        end
+
+        def price
+          variants.map(&:price).min
+        end
+      end
+    end
+  end
+end

data/app/models/stall/models/product_category.rb

@@ -0,0 +1,31 @@
+module Stall
+  module Models
+    module ProductCategory
+      extend ActiveSupport::Concern
+
+      included do
+        self.table_name = 'stall_product_categories'
+
+        acts_as_tree order: 'position'
+        class_attribute :max_depth
+
+        extend FriendlyId
+        friendly_id :name, use: [:slugged, :finders]
+        
+        has_many :products, dependent: :nullify
+
+        validates :name, presence: true
+
+        scope :ordered, -> { order(position: 'asc') }
+
+        def self.max_depth
+          2
+        end
+
+        def should_generate_new_friendly_id?
+          slug.blank?
+        end
+      end
+    end
+  end
+end

data/app/models/stall/models/product_detail.rb

@@ -0,0 +1,17 @@
+module Stall
+  module Models
+    module ProductDetail
+      extend ActiveSupport::Concern
+
+      included do
+        self.table_name = 'stall_product_details'
+
+        acts_as_orderable
+
+        belongs_to :product
+
+        validates :name, presence: true
+      end
+    end
+  end
+end

data/app/models/stall/models/product_list.rb

@@ -6,7 +6,9 @@ module Stall
       included do
         self.table_name = 'stall_product_lists'
 
-        store_accessor :data, :reference
+        include Stall::DefaultCurrencyManager
+        include Stall::ReferenceManager
+        include Stall::TotalPricesManager
 
         has_secure_token
 
@@ -16,13 +18,15 @@ module Stall
         belongs_to :customer
         accepts_nested_attributes_for :customer
 
+        has_many :generated_credit_notes, as: :source,
+                                          class_name: 'CreditNote',
+                                          dependent: :nullify
+
         validates :type, presence: true
 
-        after_initialize :ensure_currency
         after_initialize :ensure_state
 
         before_save :save_customer_if_changed
-        after_save :ensure_reference, on: :create
 
         scope :empty, -> {
           joins(
@@ -52,27 +56,11 @@ module Stall
       end
 
       def subtotal
-        price = line_items.map(&:price).sum
-        price = Money.new(price, currency) unless Money === price
-        price
+        ensure_money(line_items.map(&:price).sum)
       end
 
       def eot_subtotal
-        line_items.map(&:eot_price).sum
-      end
-
-      def total_price
-        price = items.map(&:price).sum
-        price = Money.new(price, currency) unless Money === price
-        price
-      end
-
-      def total_eot_price
-        items.map(&:eot_price).sum
-      end
-
-      def total_vat
-        items.map(&:vat).sum
+        ensure_money(line_items.map(&:eot_price).sum)
       end
 
       def total_quantity
@@ -91,20 +79,8 @@ module Stall
         true
       end
 
-      def currency
-        @currency ||= if (currency = read_attribute(:currency).presence)
-          Money::Currency.new(currency)
-        else
-          self.currency = Money.default_currency
-        end
-      end
-
       private
 
-      def ensure_currency
-        self.currency ||= Money.default_currency
-      end
-
       def ensure_state
         self.state ||= (wizard.try(:steps).try(:first) || 'pending')
       end
@@ -113,23 +89,11 @@ module Stall
         line_items.to_a
       end
 
-      def ensure_reference
-        unless reference.present?
-          reference = [Time.now.strftime('%Y%m%d'), ('%05d' % id)].join('-')
-          self.reference = reference
-          save(validate: false)
-        end
-      end
-
       def save_customer_if_changed
         customer.save if customer && customer.changed?
       end
 
       module ClassMethods
-        def find_by_reference(reference)
-          where("data->>'reference' = ?", reference).first
-        end
-
         # The .aborted and .finalized scopes cannot be declared as actual rails
         # scopes since subclasses that override the .wizard method wouldn't
         # be taken into account, scopes being executed in the context of the