From 271475893d1f1298034e402646c726d89b1001e4 Mon Sep 17 00:00:00 2001 From: Maikel Linke Date: Thu, 19 Sep 2024 15:55:51 +1000 Subject: [PATCH] Remove unused stock aggregation --- app/models/spree/product.rb | 4 ---- spec/models/spree/product_spec.rb | 8 -------- 2 files changed, 12 deletions(-) diff --git a/app/models/spree/product.rb b/app/models/spree/product.rb index bb1f2d2f59..8930d6868a 100755 --- a/app/models/spree/product.rb +++ b/app/models/spree/product.rb @@ -211,10 +211,6 @@ module Spree end end - def total_on_hand - stock_items.sum(&:count_on_hand) - end - def properties_including_inherited # Product properties override producer properties ps = product_properties.all diff --git a/spec/models/spree/product_spec.rb b/spec/models/spree/product_spec.rb index b51a4736f1..26fef88569 100644 --- a/spec/models/spree/product_spec.rb +++ b/spec/models/spree/product_spec.rb @@ -84,14 +84,6 @@ module Spree end end - describe '#total_on_hand' do - it 'returns sum of stock items count_on_hand' do - product = build(:product) - allow(product).to receive_messages stock_items: [double(Spree::StockItem, count_on_hand: 5)] - expect(product.total_on_hand).to eql(5) - end - end - context "has stock movements" do let(:product) { create(:product) } let(:variant) { product.variants.first }