[Kit Redesign] Phase 1 - Change itemizable source

app/controllers/kits_controller.rb

@@ -1,8 +1,8 @@
 class KitsController < ApplicationController
   def index
-    @kits = current_organization.kits.includes(line_items: :item, inventory_items: :storage_location).class_filter(filter_params)
+    @items = Item.where(partner_key: "kit").includes(line_items: :item, inventory_items: :storage_location).class_filter(filter_params)
     unless params[:include_inactive_items]
-      @kits = @kits.active
+      @items = @items.active
     end
     @selected_filter_name = filter_params[:by_name]
   end

app/models/base_item.rb

@@ -14,6 +14,7 @@
 #
 
 class BaseItem < ApplicationRecord
+  include Itemizable
   has_many :items, dependent: :destroy, inverse_of: :base_item, foreign_key: :partner_key, primary_key: :partner_key
   has_many :barcode_items, as: :barcodeable, dependent: :destroy
 

app/models/item.rb

@@ -170,6 +170,16 @@ def inventory_item_at(storage_location_id)
     inventory_items.find_by(storage_location_id: storage_location_id)
   end
 
+  def line_items_quantities_by_name
+    results = {}
+    self.line_items.each do |li|
+      next if li.quantity.zero?
+
+      results[li.id] = { item_id: li.item.id, name: li.item.name, quantity: li.quantity }
+    end
+    results
+  end
+
   private
 
   def update_associated_kit_name

app/views/kits/_table.html.erb

@@ -8,12 +8,12 @@
 </tr>
 </thead>
 <tbody>
-  <% @kits.each do |kit| %>
+  <% @items.each do |item| %>
     <tr>
-      <td><%= kit.name %></td>
+      <td><%= item.name %></td>
       <td>
         <ul>
-          <% kit.line_items.quantities_by_name.map do |id, item_hash| %>
+          <% item.line_items_quantities_by_name.map do |id, item_hash| %>
             <li><%= "#{item_hash[:quantity]} #{item_hash[:name]}" %></li>
           <% end %>
         </ul>
@@ -26,7 +26,7 @@
               <td> Quantity </td>
             </tr>
           </thead>
-          <% kit.inventory_items.map do |inventory_item| %>
+          <% item.inventory_items.map do |inventory_item| %>
             <% next if inventory_item.storage_location.discarded_at %>
             <tr>
               <td><%= inventory_item.storage_location.name %></td>

db/migrate/20230705105404_change_itemizable_id.rb

@@ -0,0 +1,5 @@
+class ChangeItemizableId < ActiveRecord::Migration[7.0]
+  def change
+    Rake::Task['change_itemizable_id'].invoke
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[7.0].define(version: 2023_06_16_055847) do
+ActiveRecord::Schema[7.0].define(version: 2023_07_05_105404) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
 

lib/tasks/change_kit_line_items_itemizable_id.rake

@@ -0,0 +1,18 @@
+desc "change the itemizable_id for all kits' line items to kit.item.id"
+task change_itemizable_id: :environment do
+  line_items = LineItem.where(itemizable_type: "Kit")
+  puts "Total line_items: #{line_items.count}"
+
+  line_items.each do |line_item|
+    puts "Processing line_item with id: #{line_item.id}"
+    item = line_item.itemizable.item
+
+    item_id = line_item.item_id
+    line_item.itemizable = item
+    line_item.item_id = item_id
+
+    line_item.save!
+  end
+
+  puts "Done..."
+end