5166: Remove item category

Author: dorner

app/models/item.rb

@@ -6,7 +6,6 @@
 #  active                       :boolean          default(TRUE)
 #  additional_info              :text
 #  barcode_count                :integer
-#  category                     :string
 #  distribution_quantity        :integer
 #  name                         :string
 #  on_hand_minimum_quantity     :integer          default(0), not null

db/migrate/20250504183911_remove_short_name_from_organizations.rb

@@ -1,8 +1,12 @@
 class RemoveShortNameFromOrganizations < ActiveRecord::Migration[8.0]
   def up
     safety_assured do
-      remove_index :organizations, :short_name
-      remove_column :organizations, :short_name
+      if index_exists?(:organizations, :short_name)
+        remove_index :organizations, :short_name
+      end
+      if column_exists?(:organizations, :short_name)
+        remove_column :organizations, :short_name
+      end
     end
   end
 

db/migrate/20250523203808_remove_item_category.rb

@@ -0,0 +1,11 @@
+class RemoveItemCategory < ActiveRecord::Migration[8.0]
+  def up
+    safety_assured do
+      remove_column :items, :category
+    end
+  end
+
+  def down
+    add_column :items, :category, :string
+  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[8.0].define(version: 2025_05_04_183911) do
+ActiveRecord::Schema[8.0].define(version: 2025_05_23_203808) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "pg_catalog.plpgsql"
 
@@ -392,7 +392,6 @@
 
   create_table "items", id: :serial, force: :cascade do |t|
     t.string "name"
-    t.string "category"
     t.datetime "created_at", precision: nil, null: false
     t.datetime "updated_at", precision: nil, null: false
     t.integer "barcode_count"

spec/factories/items.rb

@@ -6,7 +6,6 @@
 #  active                       :boolean          default(TRUE)
 #  additional_info              :text
 #  barcode_count                :integer
-#  category                     :string
 #  distribution_quantity        :integer
 #  name                         :string
 #  on_hand_minimum_quantity     :integer          default(0), not null

spec/models/item_spec.rb

@@ -6,7 +6,6 @@
 #  active                       :boolean          default(TRUE)
 #  additional_info              :text
 #  barcode_count                :integer
-#  category                     :string
 #  distribution_quantity        :integer
 #  name                         :string
 #  on_hand_minimum_quantity     :integer          default(0), not null