Remove old/unused #for_csv_export methods (#4995)

Author: coalest

app/models/adjustment.rb

@@ -22,10 +22,6 @@ class Adjustment < ApplicationRecord
   include Filterable
   scope :at_location, ->(location_id) { where(storage_location_id: location_id) }
   scope :by_user, ->(user_id) { where(user_id: user_id) }
-  scope :for_csv_export, ->(organization, *) {
-    where(organization: organization)
-      .includes(:storage_location, :line_items)
-  }
   scope :during, ->(range) { where(adjustments: { created_at: range }) }
 
   validate :storage_locations_belong_to_organization

app/models/barcode_item.rb

@@ -42,11 +42,6 @@ class BarcodeItem < ApplicationRecord
 
   scope :by_value, ->(value) { where(value: value) }
 
-  scope :for_csv_export, ->(organization, *) {
-    where(organization: organization)
-      .includes(:barcodeable)
-  }
-
   scope :global, -> { where(barcodeable_type: "BaseItem") }
 
   # aliases of barcodeable

app/models/concerns/provideable.rb

@@ -8,10 +8,6 @@ module Provideable
   included do
     belongs_to :organization # Automatically validates presence as of Rails 5
 
-    scope :for_csv_export, ->(organization, *) {
-      where(organization: organization).order(:business_name)
-    }
-
     def self.import_csv(csv, organization)
       csv.each do |row|
         loc = new(row.to_hash)

app/models/distribution.rb

@@ -61,11 +61,6 @@ class Distribution < ApplicationRecord
   scope :recent, ->(count = 3) { order(issued_at: :desc).limit(count) }
   scope :future, -> { where(issued_at: Time.zone.tomorrow..) }
   scope :during, ->(range) { where(distributions: { issued_at: range }) }
-  scope :for_csv_export, ->(organization, filters = {}, date_range = nil) {
-    where(organization: organization)
-      .includes(:partner, :storage_location, :line_items)
-      .apply_filters(filters, date_range)
-  }
   scope :apply_filters, ->(filters, date_range) {
     class_filter(filters.merge(during: date_range))
   }

app/models/donation.rb

@@ -51,11 +51,6 @@ class Donation < ApplicationRecord
   scope :from_manufacturer, ->(manufacturer_id) {
     where(manufacturer_id: manufacturer_id)
   }
-  scope :for_csv_export, ->(organization, *) {
-    where(organization: organization)
-      .includes(:line_items, :storage_location, :donation_site)
-      .order(created_at: :desc)
-  }
 
   before_create :combine_duplicates
 

app/models/donation_site.rb

@@ -31,11 +31,6 @@ class DonationSite < ApplicationRecord
   include Geocodable
   include Exportable
 
-  scope :for_csv_export, ->(organization, *) {
-    where(organization: organization)
-      .order(:name)
-  }
-
   scope :active, -> { where(active: true) }
 
   scope :alphabetized, -> { order(:name) }

app/models/item.rb

@@ -60,11 +60,6 @@ class Item < ApplicationRecord
   scope :by_partner_key, ->(partner_key) { where(partner_key: partner_key) }
 
   scope :by_size, ->(size) { joins(:base_item).where(base_items: { size: size }) }
-  scope :for_csv_export, ->(organization, *) {
-    where(organization: organization)
-      .includes(:base_item)
-      .alphabetized
-  }
 
   # Scopes - explanation of business rules for filtering scopes as of 20240527.  This was a mess, but is much better now.
   # 1/  Disposable.   Disposables are only the disposable diapers for children.  So we deliberately exclude adult and cloth

app/models/partner.rb

@@ -57,11 +57,6 @@ class Partner < ApplicationRecord
   before_save { email&.downcase! }
   before_update :invite_new_partner, if: :should_invite_because_email_changed?
 
-  scope :for_csv_export, ->(organization, *) {
-    where(organization: organization)
-      .order(:name)
-  }
-
   scope :alphabetized, -> { order(:name) }
   scope :active, -> { where.not(status: :deactivated) }
 

app/models/purchase.rb

@@ -48,11 +48,6 @@ class Purchase < ApplicationRecord
   scope :purchased_from, ->(purchased_from) { where(purchased_from: purchased_from) }
   scope :during, ->(range) { where(purchases: { issued_at: range }) }
   scope :recent, ->(count = 3) { order(issued_at: :desc).limit(count) }
-  scope :for_csv_export, ->(organization, *) {
-    where(organization: organization)
-      .includes(:line_items, :storage_location)
-      .order(created_at: :desc)
-  }
 
   scope :active, -> { joins(:line_items).joins(:items).where(items: { active: true }) }
 

app/models/request.rb

@@ -49,11 +49,6 @@ class Request < ApplicationRecord
   scope :by_status, ->(status) { where(status: status) }
   scope :by_request_type, ->(request_type) { where(request_type: request_type) }
   scope :during, ->(range) { where(created_at: range) }
-  scope :for_csv_export, ->(organization, *) {
-    where(organization: organization)
-      .includes(:partner)
-      .order(created_at: :desc)
-  }
 
   def total_items
     request_items.sum { |item| item["quantity"] }