Module: Spree::Stock::PackerDecorator

Defined in:
app/models/spree/stock/packer_decorator.rb

Instance Method Summary collapse

Instance Method Details

#default_package ⇒ Object

同一 variant の複数 line_item が order に存在する場合 (例: spree_embed_gravity_forms の wp_gf_entry_id で line_item を分離するケース) に、Spree 本家の実装では inventory_units.index_by(&:variant_id) で 1 つに集約してしまい、最初に作られた line_item の inventory_unit が package に渡らず orphan になる問題への対処。 Issue #1187 を参照。

修正方針: variant ごとに qty 合計で fill_status を呼び (本家の在庫判定ロジックを維持)、 結果を line_item 単位で先着順に on_hand / backordered へ按分する。 単純な each 置換では fill_status が読み取り専用 (count_on_hand を減らさない) なため 過剰販売が起きてしまうので、按分配置が必須。

@gem-override spree_core-5.3.6/app/models/spree/stock/packer.rb#default_package

See Also:



17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
 
# File 'app/models/spree/stock/packer_decorator.rb', line 17

def default_package
  package = Package.new(stock_location)

  inventory_units.group_by(&:variant_id).each do |variant_id, units_for_variant|
    variant = Spree::Variant.find(variant_id)

    if variant.should_track_inventory?
      next unless stock_location.stocks?(variant)

      total_qty = units_for_variant.sum(&:quantity)
      on_hand, _backordered = stock_location.fill_status(variant, total_qty)

      allocate_units_to_package(package, units_for_variant, on_hand)
    else
      units_for_variant.each { |inventory_unit| package.add(inventory_unit.dup) }
    end
  end

  package
end