spree_sale_prices 1.0.0 → 2.4.6

data/app/models/spree/calculator/percent_off_sale_price_calculator.rb

@@ -1,12 +0,0 @@
-module Spree
-  class Calculator::PercentOffSalePriceCalculator < Spree::Calculator
-    # TODO validate that the sale price is between 0 and 1
-    def self.description
-      "Calculates the sale price for a Variant by taking off a percentage of the original price"
-    end
-
-    def compute(sale_price)
-      (1.0 - sale_price.value.to_f) * sale_price.variant.original_price.to_f
-    end
-  end
-end

data/app/models/spree/price_decorator.rb

@@ -1,97 +0,0 @@
-Spree::Price.class_eval do
-  has_many :sale_prices
-  
-  def put_on_sale(value, params = {})
-    new_sale(value, params).save
-  end
-
-  def new_sale(value, params = {})
-    sale_price_params = {
-        value: value,
-        start_at: params.fetch(:start_at, Time.now),
-        end_at: params.fetch(:end_at, nil),
-        enabled: params.fetch(:enabled, true),
-        calculator: params.fetch(:calculator_type, Spree::Calculator::FixedAmountSalePriceCalculator.new)
-    }
-    return sale_prices.new(sale_price_params)
-  end
-  
-  # TODO make update_sale method
-
-  def active_sale
-    first_sale(sale_prices.active) if on_sale?
-  end
-  alias :current_sale :active_sale
-
-  def next_active_sale
-    first_sale(sale_prices) if sale_prices.present?
-  end
-  alias :next_current_sale :next_active_sale
-
-  def sale_price
-    active_sale.calculated_price if on_sale?
-  end
-  
-  def sale_price=(value)
-    if on_sale?
-      active_sale.update_attribute(:value, value)
-    else
-      put_on_sale(value)
-    end
-  end
-
-  def discount_percent
-    return 0.0 unless original_price > 0
-    return 0.0 unless on_sale?
-    (1 - (sale_price / original_price)) * 100
-  end
-
-  def on_sale?
-    sale_prices.active.present? && first_sale(sale_prices.active).value < original_price
-  end
-
-  def original_price
-    self[:amount]
-  end
-  
-  def original_price=(value)
-    self[:amount] = Spree::LocalizedNumber.parse(value)
-  end
-  
-  def price
-    on_sale? ? sale_price : original_price
-  end
-
-  def price=(price)
-    if on_sale?
-      sale_price = price
-    else
-      self[:amount] = Spree::LocalizedNumber.parse(price)
-    end
-  end
-  
-  def amount
-    price
-  end
-
-  def enable_sale
-    next_active_sale.enable if next_active_sale.present?
-  end
-
-  def disable_sale
-    active_sale.disable if active_sale.present?
-  end
-
-  def start_sale(end_time = nil)
-    next_active_sale.start(end_time) if next_active_sale.present?
-  end
-
-  def stop_sale
-    active_sale.stop if active_sale.present?
-  end
-  
-  private
-    def first_sale(scope)
-      scope.order("created_at DESC").first
-    end
-end

data/app/models/spree/product_decorator.rb

@@ -1,50 +0,0 @@
-Spree::Product.class_eval do
-
-  has_many :sale_prices, through: :prices
-
-  # Essentially all read values here are delegated to reading the value on the Master variant
-  # All write values will write to all variants (including the Master) unless that method's all_variants parameter is
-  # set to false, in which case it will only write to the Master variant.
-
-  delegate_belongs_to :master, :active_sale_in, :current_sale_in, :next_active_sale_in, :next_current_sale_in,
-                      :sale_price_in, :on_sale_in?, :original_price_in, :discount_percent_in, :discount_percent, :sale_price,
-                      :original_price, :on_sale?
-
-  # TODO also accept a class reference for calculator type instead of only a string
-  def put_on_sale(value, params = {})
-    all_variants = params[:all_variants] || true
-    run_on_variants(all_variants) { |v| v.put_on_sale(value, params) }
-    self.touch
-  end
-
-  alias :create_sale :put_on_sale
-
-  def enable_sale(all_variants = true)
-    run_on_variants(all_variants) { |v| v.enable_sale }
-    self.touch
-  end
-
-  def disable_sale(all_variants = true)
-    run_on_variants(all_variants) { |v| v.disable_sale }
-    self.touch
-  end
-
-  def start_sale(end_time = nil, all_variants = true)
-    run_on_variants(all_variants) { |v| v.start_sale(end_time) }
-    self.touch
-  end
-
-  def stop_sale(all_variants = true)
-    run_on_variants(all_variants) { |v| v.stop_sale }
-    self.touch
-  end
-
-  private
-    def run_on_variants(all_variants, &block)
-      if all_variants && variants.present?
-        variants.each { |v| block.call v }
-      end
-      block.call master
-    end
-
-end

data/app/models/spree/sale_price.rb

@@ -1,63 +0,0 @@
-module Spree
-  class SalePrice < ActiveRecord::Base
-
-    belongs_to :price, class_name: "Spree::Price"
-    delegate_belongs_to :price, :currency
-
-    has_one :variant, through: :price
-
-    has_one :calculator, class_name: "Spree::Calculator", as: :calculable, dependent: :destroy
-    validates :calculator, presence: true
-    accepts_nested_attributes_for :calculator
-
-    scope :active, -> { where(enabled: true).where('(start_at <= ? OR start_at IS NULL) AND (end_at >= ? OR end_at IS NULL)', Time.now, Time.now) }
-
-    before_destroy :touch_product
-    # TODO make this work or remove it
-    #def self.calculators
-    #  Rails.application.config.spree.calculators.send(self.to_s.tableize.gsub('/', '_').sub('spree_', ''))
-    #end
-
-    def calculator_type
-      calculator.class.to_s if calculator
-    end
-
-    def calculated_price
-      calculator.compute self
-    end
-
-    def enable
-      update_attribute(:enabled, true)
-    end
-
-    def disable
-      update_attribute(:enabled, false)
-    end
-
-    def active?
-      Spree::SalePrice.active.include? self
-    end
-
-    def start(end_time = nil)
-      end_time = nil if end_time.present? && end_time <= Time.now # if end_time is not in the future then make it nil (no end)
-      attr = { end_at: end_time, enabled: true }
-      attr[:start_at] = Time.now if self.start_at.present? && self.start_at > Time.now # only set start_at if it's not set in the past
-      update_attributes(attr)
-    end
-
-    def stop
-      update_attributes({ end_at: Time.now, enabled: false })
-    end
-
-    # Convenience method for displaying the price of a given sale_price in the table
-    def display_price
-      Spree::Money.new(value || 0, { currency: price.currency })
-    end
-
-    protected
-      def touch_product
-        self.variant.product.touch
-      end
-
-  end
-end

data/app/models/spree/variant_decorator.rb

@@ -1,71 +0,0 @@
-Spree::Variant.class_eval do
-
-  has_many :sale_prices, through: :prices
-
-  delegate_belongs_to :default_price, :sale_price, :original_price, :on_sale?, :discount_percent
-
-  def put_on_sale(value, params = {})
-    currencies = params.fetch(:currencies, [])
-    if params[:currency].present?
-      currencies << params[:currency] unless currencies.include? params[:currency]
-    end
-    run_on_prices(currencies) { |p| p.put_on_sale value, params }
-  end
-  alias :create_sale :put_on_sale
-
-  # TODO make update_sale method
-
-  def active_sale_in(currency)
-    price_in(currency).active_sale
-  end
-  alias :current_sale_in :active_sale_in
-
-  def next_active_sale_in(currency)
-    price_in(currency).next_active_sale
-  end
-  alias :next_current_sale_in :next_active_sale_in
-
-  def sale_price_in(currency)
-    Spree::Price.new variant_id: self.id, currency: currency, amount: price_in(currency).sale_price
-  end
-  
-  def discount_percent_in(currency)
-    price_in(currency).discount_percent
-  end
-  
-  def on_sale_in?(currency)
-    price_in(currency).on_sale?
-  end
-
-  def original_price_in(currency)
-    Spree::Price.new variant_id: self.id, currency: currency, amount: price_in(currency).original_price
-  end
-
-  def enable_sale(currencies = nil)
-    run_on_prices(currencies) { |p| p.enable_sale }
-  end
-
-  def disable_sale(currencies = nil)
-    run_on_prices(currencies) { |p| p.disable_sale }
-  end
-
-  def start_sale(end_time = nil, currencies = nil)
-    run_on_prices(currencies) { |p| p.start_sale end_time }
-  end
-
-  def stop_sale(currencies = nil)
-    run_on_prices(currencies) { |p| p.stop_sale }
-  end
-  
-  private
-    # runs on all prices or on the ones with the currencies you've specified
-    def run_on_prices(currencies = nil, &block)
-      if currencies.present? && currencies.any?
-        prices_with_currencies = prices.select { |p| currencies.include?(p.currency) }
-        prices_with_currencies.each { |p| block.call p }
-      else
-        prices.each { |p| block.call p }
-      end
-    end
-
-end

data/app/overrides/admin_product_tabs.rb

@@ -1,6 +0,0 @@
-Deface::Override.new(virtual_path: "spree/admin/shared/_product_tabs",
-                     name: "add_sale_prices_tab",
-                     insert_bottom: "[data-hook='admin_product_tabs']",
-                     text: "<%= content_tag :li, :class => ('active' if current == 'Product Sale Prices') do %><%= link_to_with_icon 'money', Spree.t(:product_sale_prices), admin_product_sale_prices_path(@product) %><% end %>",
-                     disabled: false)
-

data/app/views/spree/admin/sale_prices/_sale_price.html.slim

@@ -1,8 +0,0 @@
-tr data-hook="products_row" id = spree_dom_id(sale_price) class = "spree-product-sale-price"
-  td.align-center = sale_price.active? ? 'Yes' : ''
-  td = sale_price.display_price.to_html
-  td = (l sale_price.start_at ) if sale_price.start_at
-  td = (l sale_price.end_at) if sale_price.end_at
-  td.align-center = sale_price.enabled
-  td.actions
-    = link_to_delete sale_price, {url: admin_product_sale_price_url(product, sale_price.id), no_text: true}

data/app/views/spree/admin/sale_prices/_table.html.slim

@@ -1,18 +0,0 @@
-table.index#product_sale_prices data-hook="sale_price_table"
-  colgroup
-    col
-    col
-    col style="width: 30%"
-    col style="width: 30%"
-    col
-    col
-  thead
-    tr data-hook="products_header"
-      th = Spree.t('sale_price_active')
-      th = Spree.t('sale_price')
-      th = Spree.t('sale_price_start_at')
-      th = Spree.t('sale_price_end_at')
-      th = Spree.t('sale_price_enabled')
-      th.actions
-  tbody
-    = render partial: "sale_price", collection: product.master.sale_prices, locals: {product: product}

data/app/views/spree/admin/sale_prices/create.js.erb

@@ -1 +0,0 @@
-$('table#product_sale_prices').replaceWith('<%= j render partial: "table", locals: {product: @product} %>');

data/app/views/spree/admin/sale_prices/destroy.js.erb

@@ -1 +0,0 @@
-$('td.spree-product-sale-price#<%= j spree_dom_id(@sale_price) %>').remove();

data/app/views/spree/admin/sale_prices/index.html.slim

@@ -1,36 +0,0 @@
-= render :partial => 'spree/admin/shared/product_sub_menu'
-= render :partial => 'spree/admin/shared/product_tabs', :locals => {:current => "Product Sale Prices"}
-
-= csrf_meta_tag
-
-#add-line-item
-  fieldset
-    legend align="center" = Spree.t('add_product_sale_prices')
-
-    = form_for [:admin, @product, Spree::SalePrice.new], remote: true do |f|
-      .row
-        .columns.three
-          = label_tag t('original_price')
-          = text_field_tag '', @product.original_price.to_f, disabled: true
-
-      .row
-        .columns.three
-          = f.label :value, t('sale_price_amount')
-          = f.text_field :value
-        .columns.three
-          = f.label :currency, t('sale_price_currency')
-          = f.select :currency, options_for_select(supported_currencies_for_sale_price)
-
-      .row
-        .columns.three
-          = f.label :start_at, t('sale_price_start_at')
-          = f.text_field :start_at, class: "datepicker"
-        .columns.three
-          = f.label :end_at, t('sale_price_end_at')
-          = f.text_field :end_at, class: "datepicker"
-        .columns.three
-          = f.label :submit, style: 'visibility: hidden;'
-          = f.submit "Add Sale Price"
-
-#product-sale-prices-table
-  = render partial: "table", locals: { product: @product }

data/config/locales/de.yml

@@ -1,14 +0,0 @@
-# Sample localization file for English. Add more files in this directory for other locales.
-# See https://github.com/svenfuchs/rails-i18n/tree/master/rails%2Flocale for starting points.
-
-de:
-  sale_price: "Aktionspreis"
-  original_price: "Originalpreis (Standard Währung)"
-  product_sale_prices: "Aktionspreise des Produktes"
-  add_product_sale_prices: "Aktionspreise hinzufügen"
-  sale_price_amount: 'Wert'
-  sale_price_active: 'Aktiv'
-  sale_price_currency: "Währung"
-  sale_price_start_at: "Startdatum"
-  sale_price_end_at: "Enddatum"
-  sale_price_enabled: "Aktiviert"

data/config/locales/en.yml

@@ -1,14 +0,0 @@
-# Sample localization file for English. Add more files in this directory for other locales.
-# See https://github.com/svenfuchs/rails-i18n/tree/master/rails%2Flocale for starting points.
-
-en:
-  sale_price: "Sale Price"
-  original_price: "Original Price (Standard Currency)"
-  product_sale_prices: "Product Sale Prices"
-  add_product_sale_prices: "Add Product Sale Price"
-  sale_price_amount: 'Amount'
-  sale_price_active: 'Active'
-  sale_price_currency: "Currency"
-  sale_price_start_at: "Sale Start Date"
-  sale_price_end_at: "Sale End Date"
-  sale_price_enabled: "Enabled"

data/config/routes.rb

@@ -1,7 +0,0 @@
-Spree::Core::Engine.routes.draw do
-  namespace :admin do
-    resources :products, only: [] do
-      resources :sale_prices
-    end
-  end
-end

data/db/migrate/20120403135246_create_sale_prices.rb

@@ -1,19 +0,0 @@
-class CreateSalePrices < ActiveRecord::Migration
-  def change
-    create_table :spree_sale_prices do |t|
-      t.integer :price_id
-      t.float :value
-      t.datetime :start_at
-      t.datetime :end_at
-      t.boolean :enabled
-      t.timestamps
-    end
-
-    # Getting active sale prices for a price
-    add_index :spree_sale_prices, [:price_id, :start_at, :end_at, :enabled], name: "index_active_sale_prices_for_price"
-    # Getting all active sale prices for all prices
-    add_index :spree_sale_prices, [:start_at, :end_at, :enabled], name: "index_active_sale_prices_for_all_variants"
-    # Getting all sale prices for a price
-    add_index :spree_sale_prices, :price_id, name: "index_sale_prices_for_price"
-  end
-end

data/db/migrate/20140309000000_change_data_type_for_value.rb

@@ -1,5 +0,0 @@
-class ChangeDataTypeForValue < ActiveRecord::Migration
-  def change
-    change_column :spree_sale_prices, :value, :decimal, precision: 10, scale: 2, null: false
-  end
-end