Port from spree to solidus.

Author: njakic

.gitignore

@@ -0,0 +1,14 @@
+\#*
+*~
+.#*
+.DS_Store
+.idea
+.project
+tmp
+nbproject
+*.swp
+spec/dummy
+.rvmrc
+.ruby-version
+coverage
+Gemfile.lock

.rspec

@@ -0,0 +1 @@
+--colour

Gemfile

@@ -0,0 +1,5 @@
+source 'http://rubygems.org'
+
+gem "solidus", github: 'solidusio/solidus', branch: 'master'
+
+gemspec

LICENSE

@@ -0,0 +1,26 @@
+Copyright (c) 2012 Jonathan Dean, 2015 Renuo GmbH
+All rights reserved.
+
+Redistribution and use in source and binary forms, with or without modification,
+are permitted provided that the following conditions are met:
+
+    * Redistributions of source code must retain the above copyright notice,
+      this list of conditions and the following disclaimer.
+    * Redistributions in binary form must reproduce the above copyright notice,
+      this list of conditions and the following disclaimer in the documentation
+      and/or other materials provided with the distribution.
+    * Neither the name Spree nor the names of its contributors may be used to
+      endorse or promote products derived from this software without specific
+      prior written permission.
+
+THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+"AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
+CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
+LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

README.md

@@ -1,2 +1,158 @@
-# solidus_sale_prices
-Completely based on spree_sale_prices
+Spree Sale Prices
+=================
+
+A Spree Commerce extension (Rails Engine) that lets you set sale prices on products, either by a fixed sale price or a
+percentage off of the original price. Sale prices have a start date, end date and enabled flag to allow you to schedule
+sales, have a historical record of sale prices and put sales on hold.
+
+Requirements
+------------
+
+This Gem has been tested with Spree 2.4-stable and Ruby 2.2.1. 
+It does not support Ruby versions earlier than 1.9 for sure.
+
+Installing
+----------
+
+In your Gemfile add the following for the latest released version:
+
+    gem 'spree_sale_prices', github: 'renuo/spree-sale-prices', branch: '2.4-stable'
+
+Install the Gem:
+
+    bundle install
+
+Copy the migrations in your app:
+
+    bundle exec rake railties:install:migrations
+
+Run database migrations in your app:
+
+    bundle exec rake db:migrate
+
+Usage
+-----
+
+The following example is assuming that you have a product in your database with the price of $20 and you want to put it on sale immediately for $10:
+
+    product = Spree::Product.first
+
+    puts product.price.to_f              # => 20.0
+    puts product.on_sale?                # => false
+
+    product.put_on_sale 10
+
+    puts product.price.to_f              # => 10.0
+    puts product.original_price.to_f     # => 20.0
+    puts product.on_sale?                # => true
+
+By default it uses the supplied Spree::Calculator::FixedAmountSalePriceCalculator which essentially just returns the
+value you give it as the sale price.
+
+You can also give a certain percentage off by specifying that you want to use Spree::Calculator::PercentOffSalePriceCalculator.
+Note that the percentage is given as a float between 0 and 1, not the integer amount from 0 to 100.
+
+    product.put_on_sale 0.2, { calculator_type: Spree::Calculator::PercentOffSalePriceCalculator.new }
+    puts product.price.to_f              # => 16.0
+
+This extension gives you all of the below methods on both your Products and Variants. If accessed on the Product when reading values,
+it will return values from your Master variant. If accessed on the Product when writing values, it will by default update
+all variants including the master variants. If you change the all_variants parameter to false, it will only then write to
+the master variant and leave the other variants untouched.
+
+**price**                                             Returns the sale price if currently on sale, the original price if not
+
+**sale_price**                                        Returns the sale price if currently on sale, nil if not
+
+**original_price**                                    Always returns the original price
+
+**discount_percent**                                  Return the percentage of discount
+
+**on_sale?**                                          Return a boolean indication if it is currently on sale (enabled is set to true and we are currently within the active date range)
+
+**put\_on\_sale(value[, ...])**                       Put this item on sale (see below sections for options and more information)
+
+**create_sale**                                       Alias of ```put_on_sale```
+
+**active_sale**                                       Returns the currently active sale (Spree::SalePrice object) that price and sale_price will use. If there is more than one potentially active sale, the one with the latest created_at timestamp is used. See the section on "Multiple active sales" for the reasoning behind that.
+
+**current_sale**                                      Alias of ```active_sale```
+
+**next_active_sale**                                  Currently returns the latest created Spree::SalePrice object (active or not.) The name is kind of misleading so it should probably be changed. We may also want to make this only return the latest created inactive sale, since that's kind of the original intention of it to be used inside of ```enable_sale``` and ```start_sale```. Needs more thought.
+
+**next_current_sale**                                 Alias of ```next_active_sale```
+
+**enable_sale(all_variants = true)**                  Enable the sale returned by ```next_active_sale``` by setting that Spree::SalePrice object's enabled flag to true. Does not change the start and end dates so it does not necessary mean that the sale will then become active. Therefore, you can enable a sale in this manner and still have it not take effect on the site. (Use ```start_sale``` for that) _Note:_ The all_variants flag is only available on Spree::Product (not on Spree::Variant)
+
+**disable_sale(all_variants = true)**                 Disable the sale returned by ```active_sale``` by setting that Spree::SalePrice object's enabled flag to false. This always makes the sale inactive, regardless of the date range. _Note:_ The all_variants flag is only available on Spree::Product (not on Spree::Variant)
+
+**start_sale(end_time = nil, all_variants = true)**   Start the sale returned by ```next_active_sale``` (and make it active) by setting that Spree::SalePrice object's enabled flag to true and ensuring that the current time is in between the start and end dates. _Note:_ The all_variants flag is only available on Spree::Product (not on Spree::Variant)
+
+**stop_sale(all_variants = true)**                    Stop the sale returned by ```active_sale``` by setting that Spree::SalePrice object's enabled flag to false and the end date to the current time. _Note:_ The all_variants flag is only available on Spree::Product (not on Spree::Variant)
+
+Since you have these methods available to both your products and variants, it is possible to put the product and all
+variants on sale or just particular variants. See the explanation of put\_on\_sale below for more information.
+
+
+Options for put\_on\_sale (create_sale)
+---------------------------------------
+
+    put_on_sale(value, { calculator_type: Spree::Calculator::PercentOffSalePriceCalculator.new, all_variants: true, start_at: Time.now, end_at: nil, enabled: true })
+
+**value**           (_float_)
+
+This is either the sale price that you want to sell the product for (if using the default FixedAmountSalePriceCalculator)
+or the float representation of the percentage off of the original price (between 0 and 1)
+
+**calculator_type** (_string_)    - Default: **"Spree::Calculator::FixedAmountSalePriceCalculator"**
+
+Specify which calculator to use for determining the sale price. The default calculator will take the value as is and use it
+as the sale price. You can also pass in another calculator value to determine the sale price differently, such as the
+provided "Spree::Calculator::PercentOffSalePriceCalculator", which will take a given percentage off of the original
+price.
+
+**all_variants**    (_boolean_)   - Default: **true**
+
+_Only for Spree::Product_. By default it set all of variants (including the master variant) for the product on sale. If you change this value to false
+it will only put the master variant on sale. Only change this if you know the implications.
+
+**start_at**        (_DateTime or nil_)  - Default: **Time.now**
+
+Specify the date and time that the sale takes effect. By default it uses the current time. It can also be nil but it's not
+recommended because for future reporting reasons you will probably want to know exactly when the sale started.
+
+**end_at**          (_DateTime or nil_)  - Default: **nil**
+
+Specify the end date of the sale or nil to keep the sale running indefinitely. For future reporting reasons it's recommended
+to set this at the time you decide to deactivate the sale rather than just setting enabled to false.
+
+**enabled**         (_boolean_)   - Default: **true**
+
+Disable this sale temporarily by setting this to false (overrides the start_at and end_at range). It's not recommended to
+use this to stop the sale when you decide to end it because it could impact future reporting needs. It's mainly intended
+to keep the sale disabled while you are still working on it and it isn't quite ready, or if you need to disable temporarily
+for some reason in the middle of a sale.
+
+Multiple active sales
+---------------------
+
+Technically you can have more than one active sale at a time. However, because Spree is going to use product.price or
+variant.price throughout (with no additional parameters or means to identify a particular sale), we have to consistently
+work with a single sale price so that the customer is always charged the same price as they see on the site. What we do then
+is always take the last created sale price. We do this so that if you want to temporarily make a new sale to override a
+currently running one, you just add a new active sale. Then when that new sale ends, the old sale will be in effect again
+(provided it's still active, of course.) So you can add more than one active sale but only one will actually be used at
+a given time.
+
+Testing
+-------
+
+Tests are in progress, so there aren't any yet. I know, TDD, blah blah blah.
+
+Be sure to bundle your dependencies and then create a dummy test app for the specs to run against.
+
+    $ bundle
+    $ bundle exec rake test_app
+    $ bundle exec rspec
+
+Copyright (c) 2012 Jonathan Dean, 2015 Renuo GmbH released under the New BSD License

Rakefile

@@ -0,0 +1,21 @@
+require 'bundler'
+Bundler::GemHelper.install_tasks
+
+require 'rspec/core/rake_task'
+require 'spree/testing_support/extension_rake'
+
+RSpec::Core::RakeTask.new
+
+task :default do
+  if Dir["spec/dummy"].empty?
+    Rake::Task[:test_app].invoke
+    Dir.chdir("../../")
+  end
+  Rake::Task[:spec].invoke
+end
+
+desc 'Generates a dummy app for testing'
+task :test_app do
+  ENV['LIB_NAME'] = 'spree_sale_prices'
+  Rake::Task['extension:test_app'].invoke
+end

Versionfile

@@ -0,0 +1,10 @@
+# This file is used to designate compatibilty with different versions of Spree
+# Please see http://spreecommerce.com/documentation/extensions.html#versionfile for details
+
+# Examples
+#
+# "0.70.x" => { :branch => "master"}
+# "0.60.x" => { :branch => "0-60-stable" }
+# "0.40.x" => { :tag => "v1.0.0", :version => "1.0.0" }
+
+"2.4.x" => { branch: "2-4-stable"}

app/assets/javascripts/spree/backend/spree_sale_prices.js

@@ -0,0 +1,43 @@
+module Spree
+  module Admin
+    class SalePricesController < BaseController
+
+      before_filter :load_product
+
+      respond_to :js, :html
+
+      def index
+        @sale_prices = @product.sale_prices
+      end
+
+      def create
+        @sale_price = @product.put_on_sale params[:sale_price][:value], sale_price_params
+        respond_with(@sale_price)
+      end
+
+      def destroy
+        @sale_price = Spree::SalePrice.find(params[:id])
+        @sale_price.destroy
+        respond_with(@sale_price)
+      end
+
+      private
+
+      def load_product
+        @product = Spree::Product.find_by(slug: params[:product_id])
+        redirect_to request.referer unless @product.present?
+      end
+
+      def sale_price_params
+        params.require(:sale_price).permit(
+            :id,
+            :value,
+            :currency,
+            :start_at,
+            :end_at,
+            :enabled
+        )
+      end
+    end
+  end
+end

app/assets/javascripts/spree/frontend/spree_sale_prices.js

@@ -0,0 +1,30 @@
+Spree::BaseHelper.class_eval do
+  def display_original_price(product_or_variant)
+    product_or_variant.original_price_in(_current_currency).display_price.to_html
+  end
+  
+  def display_discount_percent(product_or_variant, append_text = 'Off')
+    discount = product_or_variant.discount_percent_in _current_currency
+    
+    if discount > 0
+      return "#{number_to_percentage(discount, precision: 0).to_html} #{append_text}"
+    else
+      return ''
+    end 
+  end
+
+  # Check if a sale is the current sale for a product, returns true or false
+  def active_for_sale_price product, sale_price
+    product.current_sale_in(_current_currency) == sale_price
+  end
+
+  def supported_currencies_for_sale_price
+    try(:supported_currencies) || [ _current_currency ]
+  end
+
+  private
+    def _current_currency
+      try(:current_currency) || Spree::Config[:currency] || 'USD'
+    end
+
+end

app/assets/stylesheets/spree/backend/spree_sale_prices.css

@@ -0,0 +1,14 @@
+module Spree
+  # <b>DEPRECATED:</b> Please use <tt>Calculator::FixedAmountSalePriceCalculator</tt> instead.
+  class Calculator::DollarAmountSalePriceCalculator < Spree::Calculator
+    warn '[DEPRECATION] `DollarAmountSalePriceCalculator` is deprecated.  Please use `FixedAmountSalePriceCalculator` instead. (%s)' % Kernel.caller.first
+
+    def self.description
+      "Calculates the sale price for a Variant by returning the provided fixed sale price"
+    end
+
+    def compute(sale_price)
+      sale_price.value
+    end
+  end
+end

app/assets/stylesheets/spree/frontend/spree_sale_prices.css

@@ -0,0 +1,12 @@
+module Spree
+  class Calculator::FixedAmountSalePriceCalculator < Spree::Calculator
+    # TODO validate that the sale price is less than the original price
+    def self.description
+      "Calculates the sale price for a Variant by returning the provided fixed sale price"
+    end
+
+    def compute(sale_price)
+      sale_price.value
+    end
+  end
+end

app/controllers/spree/admin/sale_prices_controller.rb

@@ -0,0 +1,12 @@
+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