spree_order_reporting 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7263a53957c780a378dc610972b849020d84cf66
-  data.tar.gz: 1594359c8fbf45812e3a0cc82d15f74bda606930
+  metadata.gz: edcf697cf996bd51352b2212dad592e83830f68d
+  data.tar.gz: b740df8bf3a8bb68446807b7a2c0e54e331cc4e5
 SHA512:
-  metadata.gz: 20dac91708a611885968f814ee5912da68841e9397153a9b563f89d6900dba2d213d087baa299c803b083c7eb5559493c0171ccbf2059dd8c8b42ab240c87935
-  data.tar.gz: 7e5be726dbd1768961ee3dba21b4b8c5cbbf370f50ff66223c01d58eec39195862cb64c27591aff7afd9a34b592f12e745b41fe4a554ffe93239b26d136d2dfd
+  metadata.gz: 6583553d587d0a9b8f494dbe31aa959a6c7eefa6a35bcd905a952f9b700c97a60ccaead2e80d8e41f570c666299dbbdf70cf1f73623ea022c0626fcb5455a4ac
+  data.tar.gz: 291c035253b896467c88a82243732b575b852125d109554f9e0af0d9dd83ce69fcd129d3efa974c146adb2bc96d4a172d841c79e3865de79f91b3dae3189b8dd

data/app/jobs/order_reporting/recurring_report_delayed_job.rb

@@ -4,7 +4,10 @@ module OrderReporting
   class RecurringReportDelayedJob < RecurringReportJob
     def perform
       report.send_report
-    ensure
+    end
+
+    def success
+      options[:even_if_job_exists] = true
       scheduler.schedule(report, options)
     end
   end

data/lib/order_reporting/report.rb

@@ -1,5 +1,7 @@
 module OrderReporting
   class Report
+    attr_reader :name
+
     def initialize(name)
       @name = name
     end

data/lib/order_reporting/schedulers/delayed_job_scheduler.rb

@@ -1,6 +1,10 @@
 module OrderReporting
   class DelayedJobScheduler
     def schedule(report, options)
+      unless options[:even_if_job_exists]
+        return if Delayed::Job.where('handler LIKE ?', "%#{report.name}%").any?
+      end
+
       job = RecurringReportDelayedJob.new(self, report, options)
       Delayed::Job.enqueue(job, run_at: options[:send_every].from_now)
     end

data/lib/order_reporting/version.rb

@@ -1,3 +1,3 @@
 module OrderReporting
-  VERSION = '0.0.2'
+  VERSION = '0.0.3'
 end

data/spec/dummy/app/assets/stylesheets/application.css

@@ -3,7 +3,7 @@
  * listed below.
  *
  * Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
- * or any plugin's vendor/assets/stylesheets directory can be referenced here using a relative path.
+ * or vendor/assets/stylesheets of plugins, if any, can be referenced here using a relative path.
  *
  * You're free to add application-wide styles to this file and they'll appear at the bottom of the
  * compiled file so the styles you add here take precedence over styles defined in any styles

data/spec/dummy/bin/rails

@@ -1,4 +1,4 @@
 #!/usr/bin/env ruby
-APP_PATH = File.expand_path('../../config/application', __FILE__)
+APP_PATH = File.expand_path('../../config/application',  __FILE__)
 require_relative '../config/boot'
 require 'rails/commands'

data/spec/dummy/config.ru

@@ -1,4 +1,4 @@
 # This file is used by Rack-based servers to start the application.
 
-require ::File.expand_path('../config/environment', __FILE__)
+require ::File.expand_path('../config/environment',  __FILE__)
 run Rails.application

data/spec/dummy/config/application.rb

@@ -22,7 +22,7 @@ begin
 rescue LoadError
   # spree_api is not available.
 end
-      require 'solidus_order_reporting'
+      require 'spree_order_reporting'
 
 module Dummy
   class Application < Rails::Application
@@ -50,9 +50,6 @@ module Dummy
     # The default locale is :en and all translations from config/locales/*.rb,yml are auto loaded.
     # config.i18n.load_path += Dir[Rails.root.join('my', 'locales', '*.{rb,yml}').to_s]
     # config.i18n.default_locale = :de
-
-    # Do not swallow errors in after_commit/after_rollback callbacks.
-    config.active_record.raise_in_transactional_callbacks = true
   end
 end
 

data/spec/dummy/config/environments/development.rb

@@ -27,10 +27,6 @@ Rails.application.configure do
   # number of complex assets.
   config.assets.debug = true
 
-  # Asset digests allow you to set far-future HTTP expiration dates on all assets,
-  # yet still be able to expire them through the digest params.
-  config.assets.digest = true
-
   # Adds additional error checking when serving assets at runtime.
   # Checks for improperly declared sprockets dependencies.
   # Raises helpful error messages.

data/spec/dummy/config/environments/production.rb

@@ -16,13 +16,11 @@ Rails.application.configure do
 
   # Enable Rack::Cache to put a simple HTTP cache in front of your application
   # Add `rack-cache` to your Gemfile before enabling this.
-  # For large-scale production use, consider using a caching reverse proxy like
-  # NGINX, varnish or squid.
+  # For large-scale production use, consider using a caching reverse proxy like nginx, varnish or squid.
   # config.action_dispatch.rack_cache = true
 
-  # Disable serving static files from the `/public` folder by default since
-  # Apache or NGINX already handles this.
-  config.serve_static_files = ENV['RAILS_SERVE_STATIC_FILES'].present?
+  # Disable Rails's static asset server (Apache or nginx will already do this).
+  config.serve_static_assets = false
 
   # Compress JavaScripts and CSS.
   config.assets.js_compressor = :uglifier
@@ -31,22 +29,20 @@ Rails.application.configure do
   # Do not fallback to assets pipeline if a precompiled asset is missed.
   config.assets.compile = false
 
-  # Asset digests allow you to set far-future HTTP expiration dates on all assets,
-  # yet still be able to expire them through the digest params.
+  # Generate digests for assets URLs.
   config.assets.digest = true
 
   # `config.assets.precompile` and `config.assets.version` have moved to config/initializers/assets.rb
 
   # Specifies the header that your server uses for sending files.
-  # config.action_dispatch.x_sendfile_header = 'X-Sendfile' # for Apache
-  # config.action_dispatch.x_sendfile_header = 'X-Accel-Redirect' # for NGINX
+  # config.action_dispatch.x_sendfile_header = "X-Sendfile" # for apache
+  # config.action_dispatch.x_sendfile_header = 'X-Accel-Redirect' # for nginx
 
   # Force all access to the app over SSL, use Strict-Transport-Security, and use secure cookies.
   # config.force_ssl = true
 
-  # Use the lowest log level to ensure availability of diagnostic information
-  # when problems arise.
-  config.log_level = :debug
+  # Set to :debug to see everything in the log.
+  config.log_level = :info
 
   # Prepend all log lines with the following tags.
   # config.log_tags = [ :subdomain, :uuid ]
@@ -58,7 +54,7 @@ Rails.application.configure do
   # config.cache_store = :mem_cache_store
 
   # Enable serving of images, stylesheets, and JavaScripts from an asset server.
-  # config.action_controller.asset_host = 'http://assets.example.com'
+  # config.action_controller.asset_host = "http://assets.example.com"
 
   # Ignore bad email addresses and do not raise email delivery errors.
   # Set this to true and configure the email server for immediate delivery to raise delivery errors.
@@ -71,6 +67,9 @@ Rails.application.configure do
   # Send deprecation notices to registered listeners.
   config.active_support.deprecation = :notify
 
+  # Disable automatic flushing of the log to improve performance.
+  # config.autoflush_log = false
+
   # Use default logging formatter so that PID and timestamp are not suppressed.
   config.log_formatter = ::Logger::Formatter.new
 

data/spec/dummy/config/environments/test.rb

@@ -8,7 +8,7 @@ Dummy::Application.configure do
   config.cache_classes = true
 
   # Configure static asset server for tests with Cache-Control for performance
-  config.serve_static_files = true
+  config.serve_static_assets = true
   config.static_cache_control = "public, max-age=3600"
 
   # Show full error reports and disable caching

data/spec/dummy/config/initializers/assets.rb

@@ -3,9 +3,6 @@
 # Version of your assets, change this if you want to expire all your assets.
 Rails.application.config.assets.version = '1.0'
 
-# Add additional assets to the asset load path
-# Rails.application.config.assets.paths << Emoji.images_path
-
 # Precompile additional assets.
 # application.js, application.css, and all non-JS/CSS in app/assets folder are already added.
 # Rails.application.config.assets.precompile += %w( search.js )

data/spec/dummy/config/initializers/cookies_serializer.rb

@@ -1,3 +1,3 @@
 # Be sure to restart your server when you modify this file.
 
-Rails.application.config.action_dispatch.cookies_serializer = :json
+Rails.application.config.action_dispatch.cookies_serializer = :json

data/spec/dummy/config/initializers/devise.rb

@@ -1,3 +1,3 @@
 if Object.const_defined?("Devise")
-  Devise.secret_key = "7c86621b0551ccc23c777d687764da0126d7d0d2d5a1d101616c39fda2eb730b9aa5d949c5dd4c2696c50ad775829e1d1b7a"
+  Devise.secret_key = "c24d7942ef4a7fcc007f2529c37e47b3deb17da0aa0aa08f0dc7856c87aaf4993be22f2087811be32d8eddde1c8120786bf9"
 end

data/spec/dummy/config/initializers/spree.rb

@@ -1,57 +1,14 @@
-# Configure Solidus Preferences
-# See http://docs.solidus.io/Spree/AppConfiguration.html for details
-
+# Configure Spree Preferences
+#
+# Note: Initializing preferences available within the Admin will overwrite any changes that were made through the user interface when you restart.
+#       If you would like users to be able to update a setting with the Admin it should NOT be set here.
+#
+# In order to initialize a setting do:
+# config.setting_name = 'new value'
 Spree.config do |config|
-  # Without this preferences are loaded and persisted to the database. This
-  # changes them to be stored in memory.
-  # This will be the default in a future version.
-  config.use_static_preferences!
-
-  # Core:
-
-  # Default currency for new sites
-  config.currency = "USD"
-
-  # from address for transactional emails
-  config.mails_from = "store@example.com"
-
+  # Example:
   # Uncomment to stop tracking inventory levels in the application
   # config.track_inventory_levels = false
-
-  # When set, product caches are only invalidated when they fall below or rise
-  # above the inventory_cache_threshold that is set. Default is to invalidate cache on
-  # any inventory changes.
-  # config.inventory_cache_threshold = 3
-
-
-  # Frontend:
-
-  # Custom logo for the frontend
-  # config.logo = "logo/solidus_logo.png"
-
-  # Template to use when rendering layout
-  # config.layout = "spree/layouts/spree_application"
-
-
-  # Admin:
-
-  # Custom logo for the admin
-  # config.admin_interface_logo = "logo/solidus_logo.png"
-
-  # Gateway credentials can be configured statically here and referenced from
-  # the admin. They can also be fully configured from the admin.
-  #
-  # config.static_model_preferences.add(
-  #   Spree::Gateway::StripeGateway,
-  #   'stripe_env_credentials',
-  #   secret_key: ENV['STRIPE_SECRET_KEY'],
-  #   publishable_key: ENV['STRIPE_PUBLISHABLE_KEY'],
-  #   server: Rails.env.production? ? 'production' : 'test',
-  #   test_mode: !Rails.env.production?
-  # )
 end
 
-
-
-
 Spree.user_class = "Spree::LegacyUser"

data/spec/dummy/config/secrets.yml

@@ -11,10 +11,10 @@
 # if you're sharing your code publicly.
 
 development:
-  secret_key_base: c600a252726c72eb90fbde355fe6cab623cb636f27e55c461b503084cd41ac135a29b0cb1f554d33df350a080ad76d8629b3ca43404bd992eafbb934645ef072
+  secret_key_base: c8566737f29570d1940e65caac2b8694554afce800e1bb7dc0ec8afa505f0dfb49710f5bf7a344a425af9db90c9a7dc3a24d1e74ab4cfde4e37980dbf4bd15cf
 
 test:
-  secret_key_base: ab3fda1f77541e6a3c730cc501cd8195cbba37ee8efa6fc8ed6a8e083988c00e36991fdf1187e36146d6f657265776ca2cd5ae59a8624eb5efe8619f279e98ee
+  secret_key_base: 593f2b5ea2ac1d56cc587cc9c52ae5b7f730542c5321c2ca8a0c0048fbf4d650e9c7103a1331e33ec85eb80372b70366d4c48b173b2ffab43d7fde10e54c1540
 
 # Do not keep production secrets in the repository,
 # instead read values from the environment.

data/spec/dummy/config/spree.yml

@@ -0,0 +1,2 @@
+---
+version: 2.4.11.beta

data/spec/dummy/db/migrate/{20160707103353_spree_one_two.spree.rb → 20160708163029_spree_one_two.spree.rb}

@@ -19,7 +19,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.string     :code
       t.boolean    :advertise,    :default => false
       t.string     :path
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_addresses do |t|
@@ -35,7 +35,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.string     :company
       t.references :state
       t.references :country
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_addresses, [:firstname], :name => 'index_addresses_on_firstname'
@@ -50,7 +50,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.boolean    :mandatory
       t.boolean    :locked
       t.boolean    :eligible,   :default => true
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_adjustments, [:adjustable_id], :name => 'index_adjustments_on_order_id'
@@ -74,13 +74,13 @@ class SpreeOneTwo < ActiveRecord::Migration
     create_table :spree_calculators do |t|
       t.string     :type
       t.references :calculable, :polymorphic => true
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_configurations do |t|
       t.string     :name
       t.string     :type, :limit => 50
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_configurations, [:name, :type], :name => 'index_spree_configurations_on_name_and_type'
@@ -106,7 +106,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.references :address
       t.string     :gateway_customer_profile_id
       t.string     :gateway_payment_profile_id
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_gateways do |t|
@@ -117,7 +117,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.string     :environment, :default => 'development'
       t.string     :server,      :default => 'test'
       t.boolean    :test_mode,   :default => true
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_inventory_units do |t|
@@ -127,7 +127,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.references :order
       t.references :shipment
       t.references :return_authorization
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_inventory_units, [:order_id],    :name => 'index_inventory_units_on_order_id'
@@ -139,7 +139,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.references :order
       t.integer    :quantity,                               :null => false
       t.decimal    :price,    :precision => 8, :scale => 2, :null => false
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_line_items, [:order_id],   :name => 'index_spree_line_items_on_order_id'
@@ -148,20 +148,20 @@ class SpreeOneTwo < ActiveRecord::Migration
     create_table :spree_log_entries do |t|
       t.references :source, :polymorphic => true
       t.text     :details
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_mail_methods do |t|
       t.string     :environment
       t.boolean    :active,     :default => true
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_option_types do |t|
       t.string    :name,         :limit => 100
       t.string    :presentation, :limit => 100
       t.integer   :position,                   :default => 0, :null => false
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_option_types_prototypes, :id => false do |t|
@@ -174,7 +174,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.string     :name
       t.string     :presentation
       t.references :option_type
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_option_values_variants, :id => false do |t|
@@ -201,7 +201,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.string     :payment_state
       t.string     :email
       t.text       :special_instructions
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_orders, [:number], :name => 'index_spree_orders_on_number'
@@ -214,7 +214,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.string     :environment, :default => 'development'
       t.datetime   :deleted_at
       t.string     :display_on
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_payments do |t|
@@ -225,7 +225,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.string     :state
       t.string     :response_code
       t.string     :avs_response
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_preferences do |t|
@@ -234,7 +234,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.text       :value
       t.string     :key
       t.string     :value_type
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_preferences, [:key], :name => 'index_spree_preferences_on_key', :unique => true
@@ -243,14 +243,14 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.integer    :position
       t.references :product
       t.references :option_type
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_product_properties do |t|
       t.string     :value
       t.references :product
       t.references :property
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_product_properties, [:product_id], :name => 'index_product_properties_on_product_id'
@@ -266,7 +266,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.references :tax_category
       t.references :shipping_category
       t.integer    :count_on_hand,        :default => 0,  :null => false
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_products, [:available_on], :name => 'index_spree_products_on_available_on'
@@ -285,7 +285,7 @@ class SpreeOneTwo < ActiveRecord::Migration
     create_table :spree_properties do |t|
       t.string     :name
       t.string     :presentation, :null => false
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_properties_prototypes, :id => false do |t|
@@ -295,7 +295,7 @@ class SpreeOneTwo < ActiveRecord::Migration
 
     create_table :spree_prototypes do |t|
       t.string     :name
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_return_authorizations do |t|
@@ -304,7 +304,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.decimal    :amount, :precision => 8, :scale => 2, :default => 0.0, :null => false
       t.references :order
       t.text       :reason
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_roles do |t|
@@ -328,14 +328,14 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.references :shipping_method
       t.references :address
       t.string     :state
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_shipments, [:number], :name => 'index_shipments_on_number'
 
     create_table :spree_shipping_categories do |t|
       t.string   :name
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_shipping_methods do |t|
@@ -347,7 +347,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.boolean    :match_all
       t.boolean    :match_one
       t.datetime   :deleted_at
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_state_changes do |t|
@@ -357,7 +357,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.references :user
       t.string     :stateful_type
       t.string     :next_state
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_states do |t|
@@ -371,7 +371,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.string     :description
       t.boolean    :is_default, :default => false
       t.datetime   :deleted_at
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_tax_rates do |t|
@@ -379,12 +379,12 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.references :zone
       t.references :tax_category
       t.boolean    :included_in_price, :default => false
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_taxonomies do |t|
       t.string     :name, :null => false
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_taxons do |t|
@@ -400,7 +400,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.integer    :icon_file_size
       t.datetime   :icon_updated_at
       t.text       :description
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_taxons, [:parent_id],   :name => 'index_taxons_on_parent_id'
@@ -410,7 +410,7 @@ class SpreeOneTwo < ActiveRecord::Migration
     create_table :spree_tokenized_permissions, :force => true do |t|
       t.references :permissable, :polymorphic => true
       t.string     :token
-      t.timestamps null: true
+      t.timestamps
     end
 
     add_index :spree_tokenized_permissions, [:permissable_id, :permissable_type], :name => 'index_tokenized_name_and_type'
@@ -419,7 +419,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.string     :environment
       t.string     :analytics_id
       t.boolean    :active,       :default => true
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_users do |t|
@@ -445,7 +445,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.datetime   :locked_at
       t.datetime   :remember_created_at
       t.datetime   :reset_password_sent_at
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_variants do |t|
@@ -468,7 +468,7 @@ class SpreeOneTwo < ActiveRecord::Migration
     create_table :spree_zone_members do |t|
       t.references :zoneable, :polymorphic => true
       t.references :zone
-      t.timestamps null: true
+      t.timestamps
     end
 
     create_table :spree_zones do |t|
@@ -476,7 +476,7 @@ class SpreeOneTwo < ActiveRecord::Migration
       t.string     :description
       t.boolean    :default_tax,        :default => false
       t.integer    :zone_members_count, :default => 0
-      t.timestamps null: true
+      t.timestamps
     end
   end
 end