Hard-Delete prices

We don't need soft delete on a model that belongs to an already
soft-deleted model. It'll disappear anyway.

Author: mamhoff

backend/app/controllers/spree/admin/prices_controller.rb

@@ -8,7 +8,7 @@ class PricesController < ResourceController
       def index
         params[:q] ||= {}
 
-        @search = @product.prices.kept.accessible_by(current_ability, :index).ransack(params[:q])
+        @search = @product.prices.accessible_by(current_ability, :index).ransack(params[:q])
         @master_prices = @search.result
           .currently_valid
           .for_master

backend/app/views/spree/admin/prices/_master_variant_table.html.erb

@@ -20,17 +20,17 @@
       </tr>
     </thead>
     <% master_prices.each do |price| %>
-      <tr id="<%= spree_dom_id price %>" data-hook="prices_row" class="<%= "deleted" if price.discarded? %>">
+      <tr id="<%= spree_dom_id price %>" data-hook="prices_row">
         <td><%= price.display_country %></td>
         <td><%= price.currency %></td>
         <td><%= price.money.to_html %></td>
         <td class="actions">
           <% if can?(:edit, price) %>
-            <%= link_to_edit(price, no_text: true) unless price.discarded? %>
+            <%= link_to_edit(price, no_text: true) %>
           <% end %>
           <% if can?(:destroy, price) %>
             &nbsp;
-            <%= link_to_delete(price, no_text: true) unless price.discarded? %>
+            <%= link_to_delete(price, no_text: true) %>
           <% end %>
         </td>
       </tr>

backend/app/views/spree/admin/prices/_table.html.erb

@@ -14,18 +14,18 @@
     </thead>
     <tbody>
       <% variant_prices.each do |price| %>
-        <tr id="<%= spree_dom_id price %>" data-hook="prices_row" class="<%= "deleted" if price.discarded? %>">
+        <tr id="<%= spree_dom_id price %>" data-hook="prices_row">
           <td><%= price.variant.descriptive_name %></td>
           <td><%= price.display_country %></td>
           <td><%= price.currency %></td>
           <td><%= price.money.to_html %></td>
           <td class="actions">
             <% if can?(:edit, price) %>
-              <%= link_to_edit(price, no_text: true) unless price.discarded? %>
+              <%= link_to_edit(price, no_text: true) %>
             <% end %>
             <% if can?(:destroy, price) %>
               &nbsp;
-              <%= link_to_delete(price, no_text: true) unless price.discarded? %>
+              <%= link_to_delete(price, no_text: true) %>
             <% end %>
           </td>
         </tr>

backend/spec/controllers/spree/admin/prices_controller_spec.rb

@@ -11,7 +11,7 @@
     context "when only given a product" do
       let(:product) { create(:product) }
 
-      let!(:deleted_master_price) { create(:price, variant: product.master).tap(&:discard!) }
+      let!(:deleted_master_price) { create(:price, variant: product.master).tap(&:destroy!) }
 
       subject { get :index, params: { product_id: product.slug } }
 
@@ -31,7 +31,7 @@
       let(:variant) { create(:variant) }
       let(:product) { variant.product }
 
-      let!(:deleted_variant_price) { create(:price, variant:).tap(&:discard!) }
+      let!(:deleted_variant_price) { create(:price, variant:).tap(&:destroy!) }
 
       subject { get :index, params: { product_id: product.slug, variant_id: variant.id } }
 

core/app/models/concerns/spree/default_price.rb

@@ -41,7 +41,7 @@ def default_price
     end
 
     def has_default_price?
-      default_price.present? && !default_price.discarded?
+      default_price.present?
     end
   end
 end

core/app/models/spree/price.rb

@@ -2,7 +2,7 @@
 
 module Spree
   class Price < Spree::Base
-    include Spree::SoftDeletable
+    include Spree::HardDeletable
 
     MAXIMUM_AMOUNT = BigDecimal('99_999_999.99')
 

core/app/models/spree/variant.rb

@@ -59,7 +59,6 @@ class Variant < Spree::Base
     has_many :images, -> { order(:position) }, as: :viewable, dependent: :destroy, class_name: "Spree::Image"
 
     has_many :prices,
-      -> { with_discarded },
       class_name: 'Spree::Price',
       dependent: :destroy,
       inverse_of: :variant,

core/app/models/spree/variant/price_selector.rb

@@ -39,9 +39,7 @@ def price_for_options(price_options)
       #
       # @return [Array<Spree::Price>]
       def sorted_prices_for(variant)
-        variant.prices.select do |price|
-          variant.discarded? || price.kept?
-        end.sort_by do |price|
+        variant.prices.sort_by do |price|
           [
             price.country_iso.nil? ? 0 : 1,
             price.updated_at || Time.zone.now,

core/db/migrate/20251223220550_remove_soft_deleted_prices.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+class RemoveSoftDeletedPrices < ActiveRecord::Migration[7.0]
+  def up
+    discarded_prices = Spree::Price.where.not(deleted_at: nil)
+    affected_variant_ids = discarded_prices.map(&:variant_id).uniq
+    discarded_prices.delete_all
+    Spree::Variant.where(id: affected_variant_ids).find_each(&:touch)
+    remove_column :spree_prices, :deleted_at
+  end
+
+  def down
+    add_column :spree_prices, :deleted_at, :timestamp, null: true
+  end
+end

core/spec/models/spree/product/scopes_spec.rb

@@ -166,14 +166,6 @@
         end
       end
 
-      context "with soft-deleted master price" do
-        before { product.master.prices.discard_all }
-
-        it "doesn't include the product" do
-          expect(Spree::Product.available).to match_array([])
-        end
-      end
-
       context "with multiple prices" do
         let!(:second_price) { create(:price, variant: product.master) }