spree_cm_commissioner 2.3.1 → 2.3.2.pre.pre4

data/app/models/spree_cm_commissioner/tenant.rb

@@ -12,7 +12,10 @@ module SpreeCmCommissioner
                                       through: :vendors,
                                       source: :payment_methods
 
-    validates :host, uniqueness: true, presence: true
+    validates :host, presence: true, uniqueness: true,
+                     format: { without: %r{\Ahttps?://},
+                               message: :format
+                             }
 
     enum state: { enabled: 0, disabled: 1 }
 
@@ -22,6 +25,16 @@ module SpreeCmCommissioner
       vendors.where(state: :active).first
     end
 
+    def url
+      host
+    end
+
+    def formatted_url
+      return if host.blank?
+
+      Rails.env.development? || Rails.env.test? ? "http://#{host}" : "https://#{host}"
+    end
+
     private
 
     def generate_slug

data/app/models/spree_cm_commissioner/variant_decorator.rb

@@ -30,6 +30,9 @@ module SpreeCmCommissioner
 
       base.has_many :inventory_items, class_name: 'SpreeCmCommissioner::InventoryItem'
 
+      base.has_many :pricing_model_variants, class_name: 'SpreeCmCommissioner::PricingModelVariant', dependent: :destroy
+      base.has_many :pricing_models, through: :pricing_model_variants, class_name: 'SpreeCmCommissioner::PricingModel'
+
       base.scope :subscribable, -> { active.joins(:product).where(product: { subscribable: true, status: :active }) }
       base.scope :with_permanent_stock, -> { where(product_type: base::PERMANENT_STOCK_PRODUCT_TYPES) }
       base.scope :with_non_permanent_stock, -> { where.not(product_type: base::PERMANENT_STOCK_PRODUCT_TYPES) }

data/app/models/spree_cm_commissioner/vendor_decorator.rb

@@ -91,6 +91,7 @@ module SpreeCmCommissioner
       base.has_many :trip_stops, through: :trips, class_name: 'SpreeCmCommissioner::TripStop'
       base.has_many :boarding_trip_stops, through: :trips, class_name: 'SpreeCmCommissioner::TripStop', source: :boarding_trip_stops
       base.has_many :drop_off_trip_stops, through: :trips, class_name: 'SpreeCmCommissioner::TripStop', source: :drop_off_trip_stops
+      base.has_many :pricing_models, class_name: 'SpreeCmCommissioner::PricingModel', foreign_key: :vendor_id, dependent: :destroy
 
       base.validates :account_name, :account_number, presence: true, if: lambda {
                                                                            payment_qrcode.present? && Spree::Store.default.code.include?('billing')

data/app/models/spree_cm_commissioner/vendor_place.rb

@@ -1,8 +1,9 @@
 module SpreeCmCommissioner
   class VendorPlace < Base
-    enum place_type: { location: 0, branch: 1, stop: 2 }
+    enum place_type: { venue: 0, branch: 1, stop: 2, location: 3 }
     acts_as_list scope: :vendor
 
+    belongs_to :tenant, class_name: 'SpreeCmCommissioner::Tenant', optional: true
     belongs_to :vendor, class_name: 'Spree::Vendor', optional: false
     belongs_to :place, class_name: 'SpreeCmCommissioner::Place', optional: false
     belongs_to :location, class_name: 'SpreeCmCommissioner::VendorPlace', optional: true
@@ -16,6 +17,8 @@ module SpreeCmCommissioner
 
     accepts_nested_attributes_for :place, allow_destroy: true
 
+    before_save :set_tenant_id
+
     # include place when using these delegate to avoid N+1
     def display_name
       place&.name
@@ -28,5 +31,11 @@ module SpreeCmCommissioner
     def selected
       vendor.selected_place_references.include?(place&.reference)
     end
+
+    private
+
+    def set_tenant_id
+      self.tenant_id = vendor&.tenant_id
+    end
   end
 end

data/app/overrides/spree/admin/stores/_form/store_preferences.html.erb.deface

@@ -37,6 +37,15 @@
             <% end %>
           </div>
 
+          <!-- Telegram New Vendor Alert Chat ID -->
+          <div class="col-6">
+            <%= f.field_container :preferred_telegram_new_vendor_alert_chat_id do %>
+              <%= f.label :preferred_telegram_new_vendor_alert_chat_id, Spree.t(:telegram_new_vendor_alert_chat_id) %>
+              <%= f.text_field :preferred_telegram_new_vendor_alert_chat_id, class: 'form-control' %>
+              <%= f.error_message_on :preferred_telegram_new_vendor_alert_chat_id %>
+            <% end %>
+          </div>
+
           <!-- Assetlinks -->
           <div class="col-12">
             <%= f.field_container :preferred_assetlinks do %>

data/app/queries/spree_cm_commissioner/trip_query.rb

@@ -1,12 +1,13 @@
 module SpreeCmCommissioner
   class TripQuery
-    attr_reader :origin_id, :destination_id, :date, :vendor_id, :number_of_guests, :route_type, :params
+    attr_reader :origin_id, :destination_id, :date, :vendor_id, :tenant_id, :number_of_guests, :route_type, :params
 
-    def initialize(origin_id:, destination_id:, date:, route_type: nil, vendor_id: nil, number_of_guests: nil, params: {}) # rubocop:disable Metrics/ParameterLists
+    def initialize(origin_id:, destination_id:, date:, route_type: nil, vendor_id: nil, tenant_id: nil, number_of_guests: nil, params: {}) # rubocop:disable Metrics/ParameterLists
       @origin_id = origin_id
       @destination_id = destination_id
       @date = date.to_date == Time.zone.now.to_date ? Time.zone.now : Time.zone.parse(date.to_s)
       @vendor_id = vendor_id
+      @tenant_id = tenant_id
       @number_of_guests = number_of_guests || 1
       @route_type = route_type
       @params = params
@@ -36,8 +37,9 @@ module SpreeCmCommissioner
 
     def direct_trips
       result = trip_scope
-      result = result.where({ vendor_id: vendor_id }.compact)
+      result = result.where({ vendor_id: vendor_id }.compact) if vendor_id.present?
       result = result.where(route_type: route_type) if route_type.present?
+      result = result.where(spree_vendors: { tenant_id: tenant_id }) if tenant_id.present?
       paginate(result)
     end
 
@@ -56,20 +58,24 @@ module SpreeCmCommissioner
     private
 
     def trip_scope
-      SpreeCmCommissioner::Trip
-        .select(<<~SQL.squish)
-          cm_trips.*,
-          boarding.departure_time AS boarding_departure_time,
-          boarding.stop_place_id AS boarding_stop_id, boarding.stop_name AS boarding_stop_name,
-          drop_off.stop_name AS drop_off_stop_name, drop_off.stop_place_id AS drop_off_stop_id,
-          drop_off.arrival_time AS drop_off_arrival_time,
-          COALESCE(iv.quantity_available, 0) AS quantity_available,
-          COALESCE(iv.max_capacity, 0) AS max_capacity,
-          origin_places.name AS origin_place_name, dest_places.name AS destination_place_name,
-          prices.amount AS amount, prices.compare_at_amount AS compare_at_amount,
-          prices.currency AS currency
-        SQL
-        .includes(vendor: :logo, vehicle: :option_values)
+      scope = SpreeCmCommissioner::Trip
+              .select(<<~SQL.squish)
+                cm_trips.*,
+                boarding.departure_time AS boarding_departure_time,
+                boarding.stop_place_id AS boarding_stop_id, boarding.stop_name AS boarding_stop_name,
+                drop_off.stop_name AS drop_off_stop_name, drop_off.stop_place_id AS drop_off_stop_id,
+                drop_off.arrival_time AS drop_off_arrival_time,
+                COALESCE(iv.quantity_available, 0) AS quantity_available,
+                COALESCE(iv.max_capacity, 0) AS max_capacity,
+                origin_places.name AS origin_place_name, dest_places.name AS destination_place_name,
+                prices.amount AS amount, prices.compare_at_amount AS compare_at_amount,
+                prices.currency AS currency
+              SQL
+              .includes(vendor: :logo, vehicle: :option_values)
+
+      scope = scope.joins(:vendor) if tenant_id.present?
+
+      scope
         .joins(<<~SQL.squish)
           INNER JOIN cm_trip_stops AS boarding ON boarding.trip_id = cm_trips.id AND boarding.stop_type = 0
           INNER JOIN cm_trip_stops AS drop_off ON drop_off.trip_id = cm_trips.id AND drop_off.stop_type = 1

data/app/request_schemas/spree_cm_commissioner/intercity_taxi_draft_order_schema.rb

@@ -0,0 +1,10 @@
+module SpreeCmCommissioner
+  class IntercityTaxiDraftOrderSchema < ApplicationRequestSchema
+    params do
+      required(:trip_id).value(:integer)
+      required(:from_date).value(:date)
+      required(:to_date).value(:date)
+      required(:quantity).value(:integer)
+    end
+  end
+end

data/app/request_schemas/spree_cm_commissioner/intercity_taxi_draft_order_update_schema.rb

@@ -0,0 +1,29 @@
+module SpreeCmCommissioner
+  class IntercityTaxiDraftOrderUpdateSchema < ApplicationRequestSchema
+    params do
+      optional(:remark).filled(:string)
+      optional(:pickup_map_place_attributes).hash do
+        optional(:place_name).filled(:string)
+        optional(:lat).filled(:string)
+        optional(:lng).filled(:string)
+        optional(:oob_confirmed).filled(:bool)
+      end
+      optional(:dropoff_map_place_attributes).hash do
+        optional(:place_name).filled(:string)
+        optional(:lat).filled(:string)
+        optional(:lng).filled(:string)
+        optional(:oob_confirmed).filled(:bool)
+      end
+      optional(:distance_attributes).hash do
+        optional(:distance_km).filled(:float)
+        optional(:ordered_points).filled(:array)
+        optional(:estimated_time_minutes).filled(:integer)
+        optional(:base_km).filled(:float)
+        optional(:detour_pickup_km).filled(:float)
+        optional(:detour_dropoff_km).filled(:float)
+        optional(:extra_pickup_km).filled(:float)
+        optional(:extra_dropoff_km).filled(:float)
+      end
+    end
+  end
+end

data/app/request_schemas/spree_cm_commissioner/route_places_request_schema.rb

@@ -0,0 +1,12 @@
+module SpreeCmCommissioner
+  class RoutePlacesRequestSchema < ApplicationRequestSchema
+    params do
+      required(:place_type).filled(:string)
+      required(:query).filled(:string)
+    end
+
+    rule(:place_type) do
+      key.failure('Invalid place_type. Use origin or destination') unless %w[origin destination].include?(value)
+    end
+  end
+end

data/app/serializers/spree/v2/storefront/place_serializer.rb

@@ -4,8 +4,9 @@ module Spree
       class PlaceSerializer < BaseSerializer
         set_type :place
 
-        has_many :vendors
-        has_many :nearby_places, serializer: :nearby_place
+        # Included by default, pass false to exclude
+        has_many :vendors, if: proc { |_, params| params.nil? || params[:include_vendors] != false }
+        has_many :nearby_places, serializer: :nearby_place, if: proc { |_, params| params.nil? || params[:include_nearby_places] != false }
 
         attributes :reference, :name, :vicinity, :lat, :lon, :icon, :url, :rating,
                    :formatted_phone_number, :formatted_address, :address_components, :types

data/app/serializers/spree/v2/tenant/cart_serializer.rb

@@ -20,6 +20,7 @@ module Spree
         has_many :promotions, object_method_name: :valid_promotions, id_method_name: :valid_promotion_ids,
                               serializer: Spree::V2::Storefront::PromotionSerializer
 
+        has_many :saved_guests, serializer: SpreeCmCommissioner::V2::Storefront::SavedGuestSerializer
         has_many :variants, serializer: Spree::V2::Tenant::VariantSerializer
         has_many :products, serializer: Spree::V2::Tenant::ProductSerializer
         has_many :line_items, serializer: Spree::V2::Tenant::LineItemSerializer

data/app/serializers/spree/v2/tenant/intercity_taxi_cart_serializer.rb

@@ -0,0 +1,12 @@
+module Spree
+  module V2
+    module Tenant
+      class IntercityTaxiCartSerializer < CartSerializer
+        has_many :line_items,
+                 serializer: SpreeCmCommissioner::V2::Storefront::IntercityTaxiLineItemSerializer
+
+        cache_options store: nil
+      end
+    end
+  end
+end

data/app/serializers/spree/v2/tenant/line_item_serializer.rb

@@ -20,6 +20,7 @@ module Spree
 
         has_one :vendor, serializer: Spree::V2::Tenant::VendorSerializer
         has_one :product, serializer: Spree::V2::Storefront::ProductSerializer
+        has_many :saved_guests, serializer: SpreeCmCommissioner::V2::Storefront::SavedGuestSerializer
 
         belongs_to :variant, serializer: Spree::V2::Tenant::VariantSerializer
         belongs_to :order, serializer: Spree::V2::Tenant::OrderSerializer

data/app/serializers/spree/v2/tenant/trip_place_serializer.rb

@@ -0,0 +1,8 @@
+module Spree
+  module V2
+    module Tenant
+      class TripPlaceSerializer < SpreeCmCommissioner::V2::Storefront::TripPlaceSerializer
+      end
+    end
+  end
+end

data/app/serializers/spree/v2/tenant/trip_query_result_serializer.rb

@@ -0,0 +1,8 @@
+module Spree
+  module V2
+    module Tenant
+      class TripQueryResultSerializer < SpreeCmCommissioner::V2::Storefront::TripQueryResultSerializer
+      end
+    end
+  end
+end

data/app/serializers/spree/v2/tenant/trip_serializer.rb

@@ -0,0 +1,8 @@
+module Spree
+  module V2
+    module Tenant
+      class TripSerializer < SpreeCmCommissioner::V2::Storefront::TripSerializer
+      end
+    end
+  end
+end

data/app/serializers/spree_cm_commissioner/v2/storefront/intercity_taxi_line_item_serializer.rb

@@ -2,29 +2,28 @@ module SpreeCmCommissioner
   module V2
     module Storefront
       class IntercityTaxiLineItemSerializer < BaseSerializer
-        attributes :pickup_place_name,
-                   :drop_off_place_name,
-                   :pickup_lat,
-                   :pickup_lng,
-                   :drop_off_lat,
-                   :drop_off_lng,
-                   :passenger_count,
-                   :distance_km,
-                   :ordered_points,
-                   :base_km,
-                   :detour_pickup_km,
-                   :detour_dropoff_km,
-                   :extra_pickup_km,
-                   :extra_dropoff_km,
-                   :extra_pickup_charge_usd,
-                   :extra_dropoff_charge_usd,
-                   :estimated_time_minutes,
+        # Nested-only API shape
+        attributes :pickup_map_place,
+                   :dropoff_map_place,
+                   :distance,
                    :from_date,
                    :to_date,
-                   :direction,
+                   :remark,
                    :trip_id
 
         cache_options store: nil
+
+        def pickup_map_place
+          object.pickup_map_place&.to_h
+        end
+
+        def dropoff_map_place
+          object.dropoff_map_place&.to_h
+        end
+
+        def distance
+          object.distance&.to_h
+        end
       end
     end
   end

data/app/serializers/spree_cm_commissioner/v2/storefront/route_serializer.rb

@@ -0,0 +1,12 @@
+module SpreeCmCommissioner
+  module V2
+    module Storefront
+      class RouteSerializer < BaseSerializer
+        attributes :route_name
+
+        has_one :origin_place, serializer: ::Spree::V2::Storefront::PlaceSerializer
+        has_one :destination_place, serializer: ::Spree::V2::Storefront::PlaceSerializer
+      end
+    end
+  end
+end

data/app/serializers/spree_cm_commissioner/v2/storefront/trip_vehicle_serializer.rb

@@ -4,7 +4,7 @@ module SpreeCmCommissioner
       class TripVehicleSerializer < BaseSerializer
         set_type :vehicle
 
-        attributes :id, :code, :vehicle_type
+        attributes :id, :code, :vehicle_type, :number_of_seats
 
         has_many :vehicle_photos, serializer: ::SpreeCmCommissioner::V2::Storefront::AssetSerializer
         has_many :amenities, serializer: ::SpreeCmCommissioner::V2::Storefront::AmenitySerializer

data/app/services/spree_cm_commissioner/intercity_taxi_order/create.rb

@@ -1,35 +1,19 @@
 module SpreeCmCommissioner
   module IntercityTaxiOrder
     class Create
-      attr_reader :trip_id, :from_date, :to_date, :user_id, :quantity
+      prepend ::Spree::ServiceModule::Base
 
-      def self.call(trip_id:, from_date:, to_date:, user_id: nil, quantity: 1)
-        new(
-          trip_id: trip_id,
-          from_date: from_date,
-          to_date: to_date,
-          user_id: user_id,
-          quantity: quantity
-        ).call
-      end
-
-      def initialize(trip_id:, from_date:, to_date:, user_id:, quantity:)
-        @trip_id = trip_id
-        @from_date = from_date
-        @to_date = to_date
-        @user_id = user_id
-        @quantity = quantity.to_i <= 0 ? 1 : quantity.to_i
-      end
+      def call(trip_id:, from_date:, to_date:, user: nil, quantity: 1)
+        validate!(trip_id: trip_id, from_date: from_date, to_date: to_date, quantity: quantity)
 
-      def call
-        validate!
+        quantity = quantity.to_i
 
-        ActiveRecord::Base.transaction do
+        order = ActiveRecord::Base.transaction do
           trip = SpreeCmCommissioner::Trip.find(trip_id)
           variant = trip.product.variants.first
           raise ArgumentError, 'No variant found for trip' if variant.blank?
 
-          order = Spree::Order.new(state: 'cart', use_billing: true, user_id: user_id)
+          order = Spree::Order.new(state: 'cart', use_billing: true, user: user)
 
           line_item = order.line_items.new(
             variant_id: variant.id,
@@ -42,26 +26,39 @@ module SpreeCmCommissioner
           )
 
           build_guests_for!(line_item, quantity)
+          assign_saved_guests_to_line_item(line_item)
 
           order.save!
           order.update_with_updater!
 
           order
         end
+
+        success(order)
+      rescue StandardError => e
+        failure(nil, e.message)
       end
 
       private
 
-      def validate!
+      def validate!(trip_id:, from_date:, to_date:, quantity:)
         raise ArgumentError, 'trip_id is required' if trip_id.blank?
         raise ArgumentError, 'from_date is required' if from_date.blank?
         raise ArgumentError, 'to_date is required' if to_date.blank?
         raise ArgumentError, 'to_date must be after from_date' if from_date.present? && to_date.present? && to_date < from_date
+        raise ArgumentError, 'quantity must be 1' if quantity.to_i != 1
       end
 
       def build_guests_for!(line_item, qty)
         Array.new(qty) { line_item.guests.new }
       end
+
+      def assign_saved_guests_to_line_item(line_item)
+        line_item.guests.each do |guest|
+          saved_guest = SpreeCmCommissioner::SavedGuest.new
+          guest.saved_guest = saved_guest
+        end
+      end
     end
   end
 end

data/app/services/spree_cm_commissioner/intercity_taxi_order/update.rb

@@ -1,78 +1,42 @@
 module SpreeCmCommissioner
   module IntercityTaxiOrder
     class Update
-      attr_reader :order_number, :params
-
-      def self.call(order_number:, params:)
-        new(order_number: order_number, params: params).call
-      end
-
-      def initialize(order_number:, params:)
-        @order_number = order_number
-        @params = ensure_parameters(params)
-      end
-
-      def call
-        order = Spree::Order.find_by!(number: order_number)
-        order.update!(order_params)
-
-        create_billing_address_if_missing(order)
-
-        order
-      end
-
-      private
-
-      def ensure_parameters(params)
-        return params if params.is_a?(::ActionController::Parameters)
-
-        ::ActionController::Parameters.new(params)
-      end
-
-      def create_billing_address_if_missing(order)
-        return if order.bill_address.present?
-
-        SpreeCmCommissioner::BillingAddressCreator.call(order: order)
-      end
-
-      def order_params
-        params.require(:order).permit(
-          :intel_phone_number,
-          :email,
-          line_items_attributes: [
-            :id,
-            :pickup_place_name,
-            :drop_off_place_name,
-            :from_date,
-            :pickup_lat,
-            :pickup_lng,
-            :pickup_oob_confirmed,
-            :drop_off_lat,
-            :drop_off_lng,
-            :drop_off_oob_confirmed,
-            :distance_km,
-            :ordered_points,
-            :base_km,
-            :detour_pickup_km,
-            :detour_dropoff_km,
-            :extra_pickup_km,
-            :extra_dropoff_km,
-            :extra_pickup_charge_usd,
-            :extra_dropoff_charge_usd,
-            :estimated_time_minutes,
-            :remark,
-            :passenger_count,
-            { guests_attributes: %i[
-              id
-              first_name
-              last_name
-              gender
-              nationality_id
-              age
-            ]
-            }
-          ]
-        )
+      prepend ::Spree::ServiceModule::Base
+
+      def call(
+        order:,
+        remark: nil,
+        pickup_map_place_attributes: nil,
+        dropoff_map_place_attributes: nil,
+        distance_attributes: nil
+      )
+        line_item = order.line_items.first
+
+        raise ArgumentError, 'Order has no line items' if line_item.blank?
+
+        if pickup_map_place_attributes.present?
+          line_item.pickup_map_place = SpreeCmCommissioner::IntercityTaxi::MapPlace.new(
+            pickup_map_place_attributes
+          )
+        end
+
+        if dropoff_map_place_attributes.present?
+          line_item.dropoff_map_place = SpreeCmCommissioner::IntercityTaxi::MapPlace.new(
+            dropoff_map_place_attributes
+          )
+        end
+
+        if distance_attributes.present?
+          line_item.distance = SpreeCmCommissioner::IntercityTaxi::Distance.new(
+            distance_attributes
+          )
+        end
+
+        line_item.update!(remark: remark)
+
+        success(order.reload)
+      rescue StandardError => e
+        failure(nil, e.message)
       end
     end
   end

data/app/services/spree_cm_commissioner/pricing_models/apply.rb

@@ -0,0 +1,51 @@
+module SpreeCmCommissioner
+  module PricingModels
+    class Apply
+      def initialize(line_item_id:, pricing_model_id:)
+        @line_item_id = line_item_id
+        @pricing_model_id = pricing_model_id
+      end
+
+      def call
+        pricing_model.pricing_rule_groups.each do |group|
+          apply_group(group, line_item)
+        end
+      end
+
+      private
+
+      attr_reader :line_item_id, :pricing_model_id
+
+      def line_item
+        @line_item ||= Spree::LineItem.find(line_item_id)
+      end
+
+      def pricing_model
+        @pricing_model ||= SpreeCmCommissioner::PricingModel.find(pricing_model_id)
+      end
+
+      def apply_group(group, line_item)
+        eligible_rules = group.pricing_rules.select { |r| r.eligible?(line_item) }
+
+        return if eligible_rules.empty?
+
+        if group_eligible?(group, eligible_rules)
+          group.pricing_action&.perform(line_item)
+        else
+          Rails.logger.info("Group #{group.id} not eligible for line_item #{line_item.id}")
+        end
+      end
+
+      def group_eligible?(group, eligible_rules)
+        case group.match_type
+        when 'all'
+          eligible_rules.size == group.pricing_rules.size
+        when 'any'
+          eligible_rules.any?
+        else
+          false
+        end
+      end
+    end
+  end
+end

data/app/services/spree_cm_commissioner/pricing_rules/build_params.rb

@@ -0,0 +1,53 @@
+module SpreeCmCommissioner
+  module PricingRules
+    module BuildParams
+      module_function
+
+      def permitted_group_params(params)
+        group_params = params[:spree_cm_commissioner_pricing_rule_group] || params
+
+        {
+          name: group_params[:name],
+          status: group_params[:status],
+          template_key: group_params[:template_key],
+          match_type: group_params[:match_type],
+          pricing_rules_attributes: build_rules_attributes(params[:pricing_rules]),
+          pricing_action_attributes: build_action_attributes(params[:pricing_action])
+        }
+      end
+
+      def build_rules_attributes(rules_array)
+        return [] unless rules_array
+
+        rules_array.map do |r|
+          {
+            id: r[:id],
+            type: r[:type],
+            private_metadata: r[:private_metadata],
+            _destroy: r[:_destroy]
+          }
+        end
+      end
+
+      def build_action_attributes(action_data)
+        return nil unless action_data&.dig(:calculator)
+
+        calc_data = action_data[:calculator]
+        calculator = calc_data[:type].safe_constantize&.new
+        set_preferences(calculator, calc_data[:preferences]) if calculator
+
+        { id: action_data[:id], calculator: calculator }
+      end
+
+      def set_preferences(calculator, preferences)
+        return unless preferences && calculator
+
+        prefs = preferences.respond_to?(:to_unsafe_h) ? preferences.to_unsafe_h : preferences
+        prefs.each do |key, value|
+          setter = "preferred_#{key}="
+          calculator.public_send(setter, value) if calculator.respond_to?(setter)
+        end
+      end
+    end
+  end
+end