spree_cm_commissioner 1.18.2.pre.pre1 → 2.0.0.pre.pre

data/app/models/spree_cm_commissioner/order_decorator.rb

@@ -2,6 +2,7 @@ module SpreeCmCommissioner
   module OrderDecorator
     def self.prepended(base) # rubocop:disable Metrics/MethodLength,Metrics/AbcSize
       base.include SpreeCmCommissioner::PhoneNumberSanitizer
+      base.include SpreeCmCommissioner::OrderSeatable
       base.include SpreeCmCommissioner::OrderStateMachine
 
       base.scope :subscription, -> { where.not(subscription_id: nil) }
@@ -42,6 +43,8 @@ module SpreeCmCommissioner
       base.has_many :vendors, through: :products, class_name: 'Spree::Vendor'
       base.has_many :taxons, through: :products, class_name: 'Spree::Taxon'
       base.has_many :guests, through: :line_items, class_name: 'SpreeCmCommissioner::Guest'
+      base.has_many :blocks, through: :guests, class_name: 'SpreeCmCommissioner::Block', source: :block
+      base.has_many :reserved_blocks, through: :guests, class_name: 'SpreeCmCommissioner::ReservedBlock'
       base.has_many :guest_card_classes, class_name: 'SpreeCmCommissioner::GuestCardClass', through: :variants
 
       base.delegate :customer, to: :user, allow_nil: true
@@ -58,6 +61,13 @@ module SpreeCmCommissioner
       end
     end
 
+    # override
+    # spree use this method to check stock availability & consider whether :order can continue to next state.
+    def insufficient_stock_lines
+      checker = SpreeCmCommissioner::Stock::OrderAvailabilityChecker.new(self)
+      checker.insufficient_stock_lines
+    end
+
     def ticket_seller_user?
       return false if user.nil?
 
@@ -106,7 +116,14 @@ module SpreeCmCommissioner
     end
 
     def mark_as_archive
-      update(archived_at: Time.current)
+      blocks_canceled = begin
+        cancel_blocks!
+        true
+      rescue StandardError
+        false
+      end
+
+      update(archived_at: Time.current) if blocks_canceled
     end
 
     # overrided
@@ -199,6 +216,18 @@ module SpreeCmCommissioner
 
     private
 
+    def unstock_inventory_in_redis!
+      CmAppLogger.log(label: "#{self.class.name}#unstock_inventory_in_redis!", data: { order_id: id }) do
+        SpreeCmCommissioner::RedisStock::InventoryUpdater.new(line_item_ids).unstock!
+      end
+    end
+
+    def restock_inventory_in_redis!
+      CmAppLogger.log(label: "#{self.class.name}#restock_inventory_in_redis!", data: { order_id: id }) do
+        SpreeCmCommissioner::RedisStock::InventoryUpdater.new(line_item_ids).restock!
+      end
+    end
+
     # override :spree_api
     def webhook_payload_body
       resource_serializer.new(

data/app/models/spree_cm_commissioner/pin_code.rb

@@ -2,7 +2,7 @@ module SpreeCmCommissioner
   class PinCode < SpreeCmCommissioner::Base
     has_secure_token
 
-    enum contact_type: { 'phone_number' => 0, 'email' => 1, 'telegram' => 2 }
+    enum contact_type: { 'phone_number' => 0, 'email' => 1 }
 
     validates :code, length: { maximum: 6 }
     validates :contact, presence: true
@@ -130,8 +130,7 @@ module SpreeCmCommissioner
         'SpreeCmCommissioner::PinCodeMobileConfirm' => I18n.t('pincode.readable_type.confirmation_code'),
         'SpreeCmCommissioner::PinCodeUpdateUserLogin' => I18n.t('pincode.readable_type.update_user_login'),
         'SpreeCmCommissioner::PinCodeEmailConfirm' => I18n.t('pincode.readable_type.confirmation_code'),
-        'SpreeCmCommissioner::OtpCode' => I18n.t('pincode.readable_type.otp_code'),
-        'SpreeCmCommissioner::PinCodeTelegram' => I18n.t('pincode.readable_type.telegram_verification')
+        'SpreeCmCommissioner::OtpCode' => I18n.t('pincode.readable_type.otp_code')
       }
     end
 

data/app/models/spree_cm_commissioner/place.rb

@@ -1,7 +1,5 @@
-require_dependency 'spree_cm_commissioner'
-
 module SpreeCmCommissioner
-  class Place < ApplicationRecord
+  class Place < Base
     acts_as_nested_set
 
     validates :reference, presence: true, if: :validate_reference?

data/app/models/spree_cm_commissioner/price_decorator.rb

@@ -0,0 +1,9 @@
+module SpreeCmCommissioner
+  module PriceDecorator
+    def self.prepended(base)
+      base.belongs_to :inventory_item, class_name: 'SpreeCmCommissioner::InventoryItem', inverse_of: :prices, optional: true
+    end
+  end
+end
+
+Spree::Price.prepend(SpreeCmCommissioner::PriceDecorator) unless Spree::Price.included_modules.include?(SpreeCmCommissioner::PriceDecorator)

data/app/models/spree_cm_commissioner/product_decorator.rb

@@ -6,7 +6,6 @@ module SpreeCmCommissioner
       base.include SpreeCmCommissioner::KycBitwise
       base.include SpreeCmCommissioner::Metafield
       base.include SpreeCmCommissioner::TenantUpdatable
-      base.include SpreeCmCommissioner::RouteType
 
       base.has_many :variant_kind_option_types, -> { where(kind: :variant).order(:position) },
                     through: :product_option_types, source: :option_type
@@ -33,6 +32,7 @@ module SpreeCmCommissioner
       base.has_one :google_wallet, class_name: 'SpreeCmCommissioner::GoogleWallet', dependent: :destroy
 
       base.has_many :complete_line_items, through: :classifications, source: :line_items
+      base.has_many :inventory_items, through: :variants
       base.has_many :guests, through: :line_items
 
       base.has_many :product_places, class_name: 'SpreeCmCommissioner::ProductPlace', dependent: :destroy
@@ -64,12 +64,11 @@ module SpreeCmCommissioner
       base.before_validation :set_event_id
 
       base.validate :validate_event_taxons, if: -> { taxons.event.present? }
-
       base.validate :validate_product_date, if: -> { available_on.present? && discontinue_on.present? }
-
+      base.validate :product_type_unchanged, on: :update
       base.validates :commission_rate, numericality: { greater_than_or_equal_to: 0, less_than_or_equal_to: 100 }, allow_nil: true
 
-      base.whitelisted_ransackable_attributes = %w[description name slug discontinue_on status vendor_id short_name route_type]
+      base.whitelisted_ransackable_attributes = %w[description name slug discontinue_on status vendor_id short_name]
 
       base.after_update :update_variants_vendor_id, if: :saved_change_to_vendor_id?
       base.after_update :sync_event_id_to_children, if: :saved_change_to_event_id?
@@ -114,6 +113,13 @@ module SpreeCmCommissioner
 
       errors.add(:discontinue_on, 'must be after the available on date')
     end
+
+    def product_type_unchanged
+      return if product_type_was.nil?
+      return unless product_type_changed?
+
+      errors.add(:product_type, 'cannot be changed once set')
+    end
   end
 end
 

data/app/models/spree_cm_commissioner/redis_stock/cached_inventory_items_builder.rb

@@ -0,0 +1,41 @@
+module SpreeCmCommissioner
+  module RedisStock
+    class CachedInventoryItemsBuilder
+      attr_reader :inventory_items
+
+      def initialize(inventory_items)
+        @inventory_items = inventory_items
+      end
+
+      # output: [ CachedInventoryItem(...), CachedInventoryItem(...) ]
+      def call
+        keys = inventory_items.map { |item| "inventory:#{item.id}" }
+        return [] unless keys.any?
+
+        counts = SpreeCmCommissioner.redis_pool.with { |redis| redis.mget(*keys) }
+        inventory_items.map.with_index do |inventory_item, i|
+          ::SpreeCmCommissioner::CachedInventoryItem.new(
+            inventory_key: keys[i],
+            active: inventory_item.active?,
+            quantity_available: cache_inventory(keys[i], inventory_item, counts[i]),
+            inventory_item_id: inventory_item.id,
+            variant_id: inventory_item.variant_id
+          )
+        end
+      end
+
+      private
+
+      def cache_inventory(key, inventory_item, count_in_redis)
+        return count_in_redis.to_i if count_in_redis.present?
+        return inventory_item.quantity_available unless inventory_item.active?
+
+        SpreeCmCommissioner.redis_pool.with do |redis|
+          redis.set(key, inventory_item.quantity_available, ex: inventory_item.redis_expired_in)
+        end
+
+        inventory_item.quantity_available
+      end
+    end
+  end
+end

data/app/models/spree_cm_commissioner/redis_stock/inventory_updater.rb

@@ -0,0 +1,126 @@
+module SpreeCmCommissioner
+  module RedisStock
+    class InventoryUpdater
+      class UnableToRestock < StandardError; end
+      class UnableToUnstock < StandardError; end
+
+      def initialize(line_item_ids)
+        @line_item_ids = line_item_ids
+      end
+
+      def unstock!
+        keys, quantities, inventory_ids = extract_inventory_data
+
+        raise UnableToUnstock, Spree.t(:insufficient_stock_lines_present) unless unstock(keys, quantities)
+
+        inventory_id_and_quantities = inventory_ids.map.with_index do |inventory_id, i|
+          { inventory_id: inventory_id, quantity: -quantities[i] }
+        end
+
+        schedule_sync_inventory(inventory_id_and_quantities)
+      end
+
+      def restock!
+        keys, quantities, inventory_ids = extract_inventory_data
+
+        raise UnableToRestock unless restock(keys, quantities)
+
+        inventory_id_and_quantities = inventory_ids.map.with_index do |inventory_id, i|
+          { inventory_id: inventory_id, quantity: quantities[i] }
+        end
+
+        schedule_sync_inventory(inventory_id_and_quantities)
+      end
+
+      private
+
+      # only unstock or restock when variants is available, track inventory, not backorderable & not need_confirmation.
+      # otherwise, we can ignore them.
+      def line_items
+        return @line_items if defined?(@line_items)
+
+        @line_items = Spree::LineItem.where(id: @line_item_ids).includes(variant: :stock_items)
+        @line_items = @line_items.map do |line_item|
+          next unless line_item.variant.available?
+          next unless line_item.variant.should_track_inventory?
+          next if line_item.variant.backorderable?
+          next if line_item.variant.need_confirmation?
+
+          line_item
+        end.compact
+      end
+
+      def unstock(keys, quantities)
+        SpreeCmCommissioner.redis_pool.with do |redis|
+          redis.eval(unstock_redis_script, keys: keys, argv: quantities)
+        end.positive?
+      end
+
+      def restock(keys, quantities)
+        SpreeCmCommissioner.redis_pool.with do |redis|
+          redis.eval(restock_redis_script, keys: keys, argv: quantities)
+        end.positive?
+      end
+
+      # Return: [CachedInventoryItem(...), CachedInventoryItem(...)]
+      def cached_inventory_items
+        @cached_inventory_items ||= SpreeCmCommissioner::RedisStock::LineItemsCachedInventoryItemsBuilder.new(line_item_ids: line_items.pluck(:id))
+                                                                                                         .call.values.flatten
+      end
+
+      def extract_inventory_data
+        keys = []
+        quantities = []
+        inventory_ids = []
+
+        cached_inventory_items.each do |cached_inventory_item|
+          keys << cached_inventory_item.inventory_key
+          quantities << line_items.find { |item| item.variant_id == cached_inventory_item.variant_id }.quantity
+          inventory_ids << cached_inventory_item.inventory_item_id
+        end
+
+        [keys, quantities, inventory_ids]
+      end
+
+      def unstock_redis_script
+        <<~LUA
+          local keys = KEYS
+          local quantities = ARGV
+
+          -- Check availability first
+          for i, key in ipairs(keys) do
+            local current = tonumber(redis.call('GET', key) or 0)
+            if current - tonumber(quantities[i]) < 0 then
+              return 0
+            end
+          end
+
+          -- Apply updates
+          for i, key in ipairs(keys) do
+            redis.call('DECRBY', key, tonumber(quantities[i]))
+          end
+
+          return 1
+        LUA
+      end
+
+      def restock_redis_script
+        <<~LUA
+          local keys = KEYS
+          local quantities = ARGV
+
+          -- Apply restock updates
+          for i, key in ipairs(keys) do
+            redis.call('INCRBY', key, tonumber(quantities[i]))
+          end
+
+          return 1
+        LUA
+      end
+
+      def schedule_sync_inventory(inventory_id_and_quantities)
+        SpreeCmCommissioner::InventoryItemSyncerJob.perform_later(inventory_id_and_quantities:)
+      end
+    end
+  end
+end

data/app/models/spree_cm_commissioner/redis_stock/line_items_cached_inventory_items_builder.rb

@@ -0,0 +1,36 @@
+module SpreeCmCommissioner
+  module RedisStock
+    class LineItemsCachedInventoryItemsBuilder
+      attr_reader :line_item_ids
+
+      def initialize(line_item_ids:)
+        @line_item_ids = line_item_ids
+      end
+
+      # return list of inventory items group by :line_item_id:
+      # {
+      #   1: [ CachedInventoryItem(...), CachedInventoryItem(...) ],
+      #   2: [ CachedInventoryItem(...), CachedInventoryItem(...) ],
+      # }
+      def call
+        cached_inventory_items.group_by do |cached_inventory_item|
+          line_item = line_items.find { |item| item.variant_id == cached_inventory_item.variant_id }
+          line_item.id
+        end
+      end
+
+      def cached_inventory_items
+        @cached_inventory_items ||= SpreeCmCommissioner::RedisStock::CachedInventoryItemsBuilder.new(inventory_items)
+                                                                                                .call
+      end
+
+      def inventory_items
+        @inventory_items ||= line_items.flat_map(&:inventory_items)
+      end
+
+      def line_items
+        @line_items ||= Spree::LineItem.where(id: line_item_ids).includes(:inventory_items)
+      end
+    end
+  end
+end

data/app/models/spree_cm_commissioner/redis_stock/variant_cached_inventory_items_builder.rb

@@ -0,0 +1,25 @@
+module SpreeCmCommissioner
+  module RedisStock
+    class VariantCachedInventoryItemsBuilder
+      attr_reader :variant_id, :dates
+
+      def initialize(variant_id:, dates: [])
+        @variant_id = variant_id
+        @dates = dates
+      end
+
+      # output: [ CachedInventoryItem(...), CachedInventoryItem(...) ]
+      def call
+        ::SpreeCmCommissioner::RedisStock::CachedInventoryItemsBuilder.new(inventory_items).call
+      end
+
+      def inventory_items
+        variant = Spree::Variant.find(variant_id)
+
+        inventory_items = variant.inventory_items
+        inventory_items = inventory_items.where(inventory_date: dates) if variant.permanent_stock?
+        inventory_items
+      end
+    end
+  end
+end

data/app/models/spree_cm_commissioner/reserved_block.rb

@@ -0,0 +1,30 @@
+module SpreeCmCommissioner
+  class ReservedBlock < Base
+    has_paper_trail
+
+    belongs_to :updated_by, optional: true, class_name: 'Spree::User'
+    belongs_to :guest, optional: true, class_name: 'SpreeCmCommissioner::Guest'
+    belongs_to :inventory_item, class_name: 'SpreeCmCommissioner::InventoryItem'
+    belongs_to :block, class_name: 'SpreeCmCommissioner::Block'
+
+    enum status: {
+      reserved: 0, # confirmed reservation
+      on_hold: 1, # temporary hold
+      canceled: 2
+    }
+
+    validates :status, presence: true
+    validates :expired_at, presence: true, if: -> { on_hold? }
+
+    scope :active_on_hold, -> { on_hold.where('expired_at IS NULL OR expired_at > ?', Time.current) }
+    scope :active, -> { where('expired_at IS NULL OR expired_at > ?', Time.current) }
+
+    def expired?
+      expired_at.present? && expired_at <= Time.current
+    end
+
+    def active_on_hold?
+      on_hold? && !expired?
+    end
+  end
+end

data/app/models/spree_cm_commissioner/seat_layout.rb

@@ -0,0 +1,20 @@
+module SpreeCmCommissioner
+  class SeatLayout < Base
+    has_many :seat_sections, class_name: 'SpreeCmCommissioner::SeatSection', dependent: :destroy
+
+    has_many :top_level_blocks, -> { where(seat_section_id: nil) }, class_name: 'SpreeCmCommissioner::Block', dependent: :destroy
+    has_many :section_blocks, -> { where.not(seat_section_id: nil) }, class_name: 'SpreeCmCommissioner::Block', dependent: :destroy
+
+    belongs_to :layoutable, polymorphic: true
+
+    enum status: {
+      inactive: 0,
+      active: 1,
+      archived: 2
+    }
+
+    validates :name, presence: true
+
+    accepts_nested_attributes_for :seat_sections, :top_level_blocks
+  end
+end

data/app/models/spree_cm_commissioner/seat_section.rb

@@ -0,0 +1,16 @@
+module SpreeCmCommissioner
+  class SeatSection < Base
+    belongs_to :seat_layout, class_name: 'SpreeCmCommissioner::SeatLayout', optional: false
+
+    has_many :blocks, class_name: 'SpreeCmCommissioner::Block', dependent: :destroy
+
+    validates :name, presence: true
+    validates :width, presence: true, numericality: { greater_than: 0 }
+    validates :height, presence: true, numericality: { greater_than: 0 }
+    validates :x, presence: true, numericality: true
+    validates :y, presence: true, numericality: true
+    validates :rotation, presence: true, numericality: { greater_than_or_equal_to: -360, less_than_or_equal_to: 360 }
+
+    accepts_nested_attributes_for :blocks
+  end
+end

data/app/models/spree_cm_commissioner/seats/blocks_canceler.rb

@@ -0,0 +1,30 @@
+module SpreeCmCommissioner
+  module Seats
+    class BlocksCanceler
+      def initialize(order_id:, cancel_by: nil)
+        @order_id = order_id
+        @cancel_by = cancel_by
+      end
+
+      # Cancels all reserved blocks associated with a given order.
+      #
+      # Typically used when:
+      # - The order is canceled.
+      # - The user abandons the order before completing checkout.
+      def cancel_blocks!
+        order = Spree::Order.find(@order_id)
+
+        ActiveRecord::Base.transaction do
+          order.reserved_blocks.each do |reserved_block|
+            reserved_block.assign_attributes(
+              status: :canceled,
+              expired_at: nil,
+              updated_by: @cancel_by
+            )
+            raise Errors::UnableToSaveReservedBlockRecord unless reserved_block.save
+          end
+        end
+      end
+    end
+  end
+end

data/app/models/spree_cm_commissioner/seats/blocks_holder.rb

@@ -0,0 +1,53 @@
+module SpreeCmCommissioner
+  module Seats
+    class BlocksHolder
+      HOLD_DURATION = 8.minutes
+
+      def initialize(line_item_ids:, hold_by: nil)
+        @line_item_ids = line_item_ids
+        @hold_by = hold_by
+      end
+
+      # Hold blocks for the given line items, preventing dublicate block selection.
+      #
+      # It ensures:
+      # - Raises an error if already reserved.
+      # - Raises an error if currently on_hold by someone else.
+      # - Allows re-locking if already on_hold by the same guest.
+      # - Otherwise, proceeds to lock the given blocks.
+      def hold_blocks!
+        line_items = Spree::LineItem.where(id: @line_item_ids).includes(:inventory_items, guests_with_blocks: :block)
+        guests_with_blocks = line_items.flat_map(&:guests_with_blocks).compact
+        inventory_items = line_items.flat_map(&:inventory_items).compact
+
+        ActiveRecord::Base.transaction do
+          guests_with_blocks.each do |guest|
+            inventory_items.each do |inventory_item|
+              hold_specific_block!(inventory_item, guest)
+            end
+          end
+        end
+      end
+
+      private
+
+      def hold_specific_block!(inventory_item, guest)
+        reserved_block = SpreeCmCommissioner::ReservedBlock.find_or_initialize_by(inventory_item: inventory_item, block: guest.block)
+
+        raise Errors::BlocksAreReservedByOtherGuest if reserved_block.reserved? && reserved_block.guest_id != guest.id
+        raise Errors::BlocksAreReservedBySameGuest if reserved_block.reserved? && reserved_block.guest_id == guest.id
+        raise Errors::BlocksAreOnHoldByOtherGuest if reserved_block.active_on_hold? && reserved_block.guest_id != guest.id
+
+        reserved_block.assign_attributes(
+          status: :on_hold,
+          guest: guest,
+          expired_at: HOLD_DURATION.from_now,
+          inventory_date: inventory_item.inventory_date,
+          updated_by: @hold_by
+        )
+
+        raise Errors::UnableToSaveReservedBlockRecord unless reserved_block.save
+      end
+    end
+  end
+end

data/app/models/spree_cm_commissioner/seats/blocks_reserver.rb

@@ -0,0 +1,49 @@
+module SpreeCmCommissioner
+  module Seats
+    class BlocksReserver
+      def initialize(line_item_ids:, reserve_by: nil)
+        @line_item_ids = line_item_ids
+        @reserve_by = reserve_by
+      end
+
+      # Mark blocks as reserved for the given line items.
+      #
+      # It ensures:
+      # - Raises an error if already reserved by same guest or other guest.
+      # - Raises an error if currently on_hold by someone else.
+      # - Otherwise, proceeds to reserve the given blocks.
+      def reserve_blocks!
+        line_items = Spree::LineItem.where(id: @line_item_ids).includes(:inventory_items, guests_with_blocks: :block)
+        guests_with_blocks = line_items.flat_map(&:guests_with_blocks).compact
+        inventory_items = line_items.flat_map(&:inventory_items).compact
+
+        ActiveRecord::Base.transaction do
+          guests_with_blocks.each do |guest|
+            inventory_items.each do |inventory_item|
+              reserve_specific_block!(inventory_item, guest)
+            end
+          end
+        end
+      end
+
+      def reserve_specific_block!(inventory_item, guest)
+        reserved_block = SpreeCmCommissioner::ReservedBlock.find_or_initialize_by(inventory_item_id: inventory_item.id, block_id: guest.block_id)
+
+        raise Errors::BlocksAreReservedByOtherGuest if reserved_block.reserved? && reserved_block.guest_id != guest.id
+        raise Errors::BlocksAreReservedBySameGuest if reserved_block.reserved? && reserved_block.guest_id == guest.id
+        raise Errors::BlocksAreOnHoldByOtherGuest if reserved_block.active_on_hold? && reserved_block.guest_id != guest.id
+
+        # mark the block as reserved if not on_hold or lock by anyone but already expired
+        reserved_block.assign_attributes(
+          guest: guest,
+          status: :reserved,
+          expired_at: nil,
+          inventory_date: inventory_item.inventory_date,
+          updated_by: @reserve_by
+        )
+
+        raise Errors::UnableToSaveReservedBlockRecord unless reserved_block.save
+      end
+    end
+  end
+end

data/app/models/spree_cm_commissioner/seats/errors/blocks_are_on_hold_by_other_guest.rb

@@ -0,0 +1,4 @@
+module SpreeCmCommissioner
+  class Seats::Errors::BlocksAreOnHoldByOtherGuest < StandardError
+  end
+end

data/app/models/spree_cm_commissioner/seats/errors/blocks_are_reserved_by_other_guest.rb

@@ -0,0 +1,4 @@
+module SpreeCmCommissioner
+  class Seats::Errors::BlocksAreReservedByOtherGuest < StandardError
+  end
+end

data/app/models/spree_cm_commissioner/seats/errors/blocks_are_reserved_by_same_guest.rb

@@ -0,0 +1,4 @@
+module SpreeCmCommissioner
+  class Seats::Errors::BlocksAreReservedBySameGuest < StandardError
+  end
+end

data/app/models/spree_cm_commissioner/seats/errors/unable_to_save_reserved_block_record.rb

@@ -0,0 +1,4 @@
+module SpreeCmCommissioner
+  class Seats::Errors::UnableToSaveReservedBlockRecord < StandardError
+  end
+end

data/app/models/spree_cm_commissioner/service_calendar.rb

@@ -1,5 +1,3 @@
-require_dependency 'spree_cm_commissioner'
-
 module SpreeCmCommissioner
   class ServiceCalendar < SpreeCmCommissioner::Base
     # exception_rules = [

data/app/models/spree_cm_commissioner/state_decorator.rb

@@ -3,7 +3,6 @@ module SpreeCmCommissioner
     def self.prepended(base)
       base.whitelisted_ransackable_attributes |= %w[name abbr country_id]
       base.has_many :vendors, foreign_key: 'default_state_id', class_name: 'Spree::Vendor', inverse_of: :default_state, dependent: :nullify
-      base.has_many :stops, class_name: 'SpreeCmCommissioner::Place'
 
       def update_total_inventory
         update(total_inventory: vendors.pluck(:total_inventory).compact.sum)