diff options
author | Alessandro Desantis <desa.alessandro@gmail.com> | 2021-01-20 10:50:50 +0100 |
---|---|---|
committer | Alessandro Desantis <desa.alessandro@gmail.com> | 2021-01-30 15:23:41 +0100 |
commit | ce4edc06e6079d8c098f1d0754e3c8e31b355e2d (patch) | |
tree | 064f3682b13e2a67768314802859f6b2b546abfd /lib/solidus_subscriptions/subscription_line_item_builder.rb | |
parent | 58972478854a4f8f137506591a186a4d528900b9 (diff) |
Move all business logic to `lib`
It wasn't clear why certain business logic should live in `app/services`
while other should live in `lib`. By unifying everything in one directory,
we make it easier for developers to inspect the code and reduce the
cognitive load when implementing new classes.
Diffstat (limited to 'lib/solidus_subscriptions/subscription_line_item_builder.rb')
-rw-r--r-- | lib/solidus_subscriptions/subscription_line_item_builder.rb | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/lib/solidus_subscriptions/subscription_line_item_builder.rb b/lib/solidus_subscriptions/subscription_line_item_builder.rb new file mode 100644 index 0000000..7354102 --- /dev/null +++ b/lib/solidus_subscriptions/subscription_line_item_builder.rb @@ -0,0 +1,23 @@ +# frozen_string_literal: true + +module SolidusSubscriptions + module SubscriptionLineItemBuilder + private + + def create_subscription_line_item(line_item) + SolidusSubscriptions::LineItem.create!( + subscription_params.merge(spree_line_item: line_item) + ) + + # Rerun the promotion handler to pickup subscription promotions + ::Spree::PromotionHandler::Cart.new(line_item.order).activate + line_item.order.recalculate + end + + def subscription_params + params.require(:subscription_line_item).permit( + SolidusSubscriptions.configuration.subscription_line_item_attributes + ) + end + end +end |