I created a custom `ShippingCalculator` following the instru solidus #support

I created a custom `ShippingCalculator` following ...

Sabo

09/23/2022, 1:41 PM

I created a custom

ShippingCalculator

following the instructions in https://guides.solidus.io/developers/shipments/custom-shipping-calculators The custom ShippingCalculator (seems) work correctly in the rails console

Shop::Calculator::Shipping::CustomCalculator

but when I try to register it in

intializers/spree.rb

Copy code

Rails.application.config.spree.calculators.shipping_methods << Shop::Calculator::Shipping::CustomCalculator

I get :

Copy code

uninitialized constant Shop::Calculator (NameError)
Rails.application.config.spree.calculators.shipping_methods << Shop::Calculator::Shipping::CustomCalculator

kennyadsl

09/23/2022, 1:46 PM

Can you please show how the calculator is defined and its location in the filesystem?

Sabo

09/23/2022, 2:07 PM

shop/app/models/shop/calculator/shipping/custom_calculator.rb

Copy code

module Shop
	class Calculator::Shipping::CustomCalculator < Spree::ShippingCalculator
		preference :currency, :string, default: -> { Spree::Config[:currency] }
	
		def self.description
			"Calculateur de livraison pour France (FR)"
		end
		
		def compute_package(package)
			package.weight / 50
		end	
		
	end
end

kennyadsl

09/23/2022, 2:52 PM

can you try writing the modules one per line?

Copy code

module Shop
  module Calculator
    module Shipping
      class CustomCalculator < ...

👍 1

Sabo

09/23/2022, 4:14 PM

I get the same result

kennyadsl

09/23/2022, 6:14 PM

Ok, can you please try:

Copy code

Rails.application.config.to_prepare do
  Rails.application.config.spree.calculators.shipping_methods << Shop::Calculator::Shipping::CustomCalculator
end

👍 1

kennyadsl

09/23/2022, 6:14 PM

From Rails 7, this is required

kennyadsl

09/23/2022, 6:15 PM

and yes, guides should reflect that, can you please let me know if this solves?

👍 1

Sabo

09/23/2022, 6:59 PM

Ok I have Rails 7.0.4 and Solidus 3.2.2 I will try

Sabo

09/23/2022, 7:02 PM

It's working thank you @kennyadsl 😉

kennyadsl

09/23/2022, 8:03 PM

Credits to @waiting_for_dev for suggesting me that solution

❤️ 1

Sabo

09/24/2022, 7:29 AM

Thank you @waiting_for_dev x)

❤️ 1

waiting_for_dev

09/27/2022, 4:34 AM

Hey @Sabo, by the way, be aware of the new guides: https://edgeguides.solidus.io/advanced-solidus/tax-calculation#customizing-tax-calculation

waiting_for_dev

09/27/2022, 4:39 AM

They propose an alternative configuration, pushing a

String

instead of the class itself. So, as an alternative, you can also have:

Copy code

Rails.application.config.spree.calculators.shipping_methods << 'Shop::Calculator::Shipping::CustomCalculator'

If you go with the

<http://config.to|config.to>_prepare

variant, be aware that the block is run at every code reload. That means the class will be pushed more than once in development. I think that won’t change your app behavior, but a to be safer:

Copy code

Rails.application.config.to_prepare do
  Rails.application.config.spree.calculators.shipping_methods |= [Shop::Calculator::Shipping::CustomCalculator]
end

👍 1

waiting_for_dev

09/27/2022, 4:39 AM

The latter will only push the `CustomCalculator` class unless it’s already there.

Open in Slack

Previous Next