made some possibly-unecessary changes to the way the calculators are …

…set up
jsqu99 · Nov 21, 2013 · b4d1f18 · b4d1f18 · qinghe · Sep 10, 2014
1 parent 221e3fd
commit b4d1f18
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 9 deletions.
diff --git a/app/models/spree/calculator/customization_image.rb b/app/models/spree/calculator/customization_image.rb
@@ -1,3 +1,5 @@
+require_dependency 'spree/calculator'
+
 module Spree
   class Calculator::CustomizationImage < Calculator
     preference :price, :decimal

diff --git a/app/models/spree/calculator/engraving.rb b/app/models/spree/calculator/engraving.rb
@@ -1,3 +1,5 @@
+require_dependency 'spree/calculator'
+
 module Spree
   class Calculator::Engraving < Calculator
     preference :price_per_letter, :decimal

diff --git a/app/models/spree/calculator/no_charge.rb b/app/models/spree/calculator/no_charge.rb
@@ -1,3 +1,5 @@
+require_dependency 'spree/calculator'
+
 module Spree
   class Calculator::NoCharge < Calculator
     def self.description

diff --git a/lib/spree_flexi_variants/engine.rb b/lib/spree_flexi_variants/engine.rb
@@ -28,21 +28,16 @@ def self.activate
     initializer "spree.flexi_variants.assets.precompile" do |app|
         app.config.assets.precompile += ['store/spree_flexi_variants_exclusions.js','admin/orders/flexi_configuration.js','store/spree-flexi-variants.*']
     end
-    # Had a good reason for this rescue below, and wish I'd commented it better when I wrote it
-    # TODO - figure this out and de-ugly
-    begin
-      initializer "spree.register.calculators" do |app|
-        app.config.spree.calculators.add_class('product_customization_types')
-        app.config.spree.calculators.product_customization_types = [
+
+    initializer "spree.flexi_variants.register.calculators" do |app|
+      app.config.spree.calculators.add_class('product_customization_types')
+      app.config.spree.calculators.product_customization_types = [
                                                                     Spree::Calculator::Engraving,
                                                                     Spree::Calculator::AmountTimesConstant,
                                                                     Spree::Calculator::ProductArea,
                                                                     Spree::Calculator::CustomizationImage,
                                                                     Spree::Calculator::NoCharge
                                                                    ]
-      end
-    rescue => problem
-      puts "intentionally ignoring problem in calculator registration #{problem}"
     end
   end
 end