harpiya_core 4.3.0.alpha

data/db/migrate/20150626181949_add_taxable_adjustment_total_to_line_item.rb

@@ -0,0 +1,19 @@
+class AddTaxableAdjustmentTotalToLineItem < ActiveRecord::Migration[4.2]
+  def change
+    add_column :harpiya_line_items, :taxable_adjustment_total, :decimal,
+               precision: 10, scale: 2, default: 0.0, null: false
+    add_column :harpiya_line_items, :non_taxable_adjustment_total, :decimal,
+               precision: 10, scale: 2, default: 0.0, null: false
+
+    add_column :harpiya_shipments, :taxable_adjustment_total, :decimal,
+               precision: 10, scale: 2, default: 0.0, null: false
+    add_column :harpiya_shipments, :non_taxable_adjustment_total, :decimal,
+               precision: 10, scale: 2, default: 0.0, null: false
+
+    add_column :harpiya_orders, :taxable_adjustment_total, :decimal,
+               precision: 10, scale: 2, default: 0.0, null: false
+    add_column :harpiya_orders, :non_taxable_adjustment_total, :decimal,
+               precision: 10, scale: 2, default: 0.0, null: false
+    # TODO migration that updates old orders
+  end
+end

data/db/migrate/20150627090949_migrate_payment_methods_display.rb

@@ -0,0 +1,12 @@
+class MigratePaymentMethodsDisplay < ActiveRecord::Migration[4.2]
+  def change
+    Harpiya::PaymentMethod.all.each do |method|
+      if method.display_on.blank?
+        method.display_on = "both"
+        method.save
+      end
+    end
+
+    change_column :harpiya_payment_methods, :display_on, :string, default: "both"
+  end
+end

data/db/migrate/20150707204155_enable_acts_as_paranoid_on_calculators.rb

@@ -0,0 +1,6 @@
+class EnableActsAsParanoidOnCalculators < ActiveRecord::Migration[4.2]
+  def change
+    add_column :harpiya_calculators, :deleted_at, :datetime
+    add_index :harpiya_calculators, :deleted_at
+  end
+end

data/db/migrate/20150714154102_harpiya_payment_method_store_credits.rb

@@ -0,0 +1,12 @@
+class HarpiyaPaymentMethodStoreCredits < ActiveRecord::Migration[4.2]
+  def up
+    # Reload to pick up new position column for acts_as_list
+    Harpiya::PaymentMethod.reset_column_information
+    Harpiya::PaymentMethod::StoreCredit.find_or_create_by(name: 'Store Credit', description: 'Store Credit',
+                                                        active: true, display_on: 'back_end')
+  end
+
+  def down
+    Harpiya::PaymentMethod.find_by(type: 'Harpiya::PaymentMethod::StoreCredit', name: 'Store Credit').try(&:destroy)
+  end
+end

data/db/migrate/20150726141425_rename_has_and_belongs_to_associations_to_model_names.rb

@@ -0,0 +1,18 @@
+class RenameHasAndBelongsToAssociationsToModelNames < ActiveRecord::Migration[4.2]
+  def change
+    {
+      'harpiya_option_types_prototypes' => 'harpiya_option_type_prototypes',
+      'harpiya_option_values_variants' => 'harpiya_option_value_variants',
+      'harpiya_orders_promotions' => 'harpiya_order_promotions',
+      'harpiya_products_promotion_rules' => 'harpiya_product_promotion_rules',
+      'harpiya_taxons_promotion_rules' => 'harpiya_promotion_rule_taxons',
+      'harpiya_promotion_rules_users' => 'harpiya_promotion_rule_users',
+      'harpiya_properties_prototypes' => 'harpiya_property_prototypes',
+      'harpiya_taxons_prototypes' => 'harpiya_prototype_taxons',
+      'harpiya_roles_users' => 'harpiya_role_users',
+      'harpiya_shipping_methods_zones' => 'harpiya_shipping_method_zones'
+    }.each do |old_name, new_name|
+      rename_table old_name, new_name
+    end
+  end
+end

data/db/migrate/20150727191614_harpiya_store_credit_types.rb

@@ -0,0 +1,11 @@
+class HarpiyaStoreCreditTypes < ActiveRecord::Migration[4.2]
+  def up
+    Harpiya::StoreCreditType.find_or_create_by(name: 'Expiring', priority: 1)
+    Harpiya::StoreCreditType.find_or_create_by(name: 'Non-expiring', priority: 2)
+  end
+
+  def down
+    Harpiya::StoreCreditType.find_by(name: 'Expiring').try(&:destroy)
+    Harpiya::StoreCreditType.find_by(name: 'Non-expiring').try(&:destroy)
+  end
+end

data/db/migrate/20150819154308_add_discontinued_to_products_and_variants.rb

@@ -0,0 +1,68 @@
+class AddDiscontinuedToProductsAndVariants < ActiveRecord::Migration[4.2]
+  def up
+    add_column :harpiya_products, :discontinue_on, :datetime, after: :available_on
+    add_column :harpiya_variants, :discontinue_on, :datetime, after: :deleted_at
+
+    add_index :harpiya_products, :discontinue_on
+    add_index :harpiya_variants, :discontinue_on
+
+    puts "Warning: This migration changes the meaning of 'deleted'. Before this change, 'deleted' meant products that were no longer being sold in your store. After this change, you can only delete a product or variant if it has not already been sold to a customer (a model-level check enforces this). Instead, you should use the new field 'discontinue_on' for products or variants which were sold in the past but no longer for sale. This fixes bugs when other objects are attached to deleted products and variants. (Even though acts_as_paranoid gem keeps the records in the database, most associations are automatically scoped to exclude the deleted records.) In thew meaning of 'deleted,' you can still use the delete function on products & variants which are *truly user-error mistakes*, specifically before an order has been placed or the items have gone on sale. You also must use the soft-delete function (which still works after this change) to clean up slug (product) and SKU (variant) duplicates. Otherwise, you should generally over ever need to discontinue products.
+
+Data Fix: We will attempt to reverse engineer the old meaning of 'deleted' (no longer for sale) to the new database field 'discontinue_on'. However, since Slugs and SKUs cannot be duplicated on Products and Variants, we cannot gaurantee this to be foolproof if you have deteled Products and Variants that have duplicate Slugs or SKUs in non-deleted records. In these cases, we recommend you use the additional rake task to clean up your old records (see rake db:fix_orphan_line_items). If you have such records, this migration will leave them in place, preferring the non-deleted records over the deleted ones. However, since old line items will still be associated with deleted objects, you will still the bugs in your app until you run:
+
+rake db:fix_orphan_line_items
+
+We will print out a report of the data we are fixing now: "
+
+    Harpiya::Product.only_deleted.each do |product|
+      # determine if there is a slug duplicate
+      the_dup = Harpiya::Product.find_by(slug: product.slug)
+      if the_dup.nil?
+        # check to see if there are line items attached to any variants
+        if Harpiya::Variant.with_deleted.where(product_id: product.id).map(&:line_items).any?
+          puts "recovering deleted product id #{product.id} ... this will un-delete the record and set it to be discontinued"
+
+          old_deleted = product.deleted_at
+          product.update_column(:deleted_at, nil) # for some reason .recover doesn't appear to be a method
+          product.update_column(:discontinue_on, old_deleted)
+        else
+          puts "leaving product id #{product.id} deleted because there are no line items attached to it..."
+        end
+      else
+        puts "leaving product id #{product.id} deleted because there is a duplicate slug for '#{product.slug}' (product id #{the_dup.id}) "
+        if product.variants.map(&:line_items).any?
+          puts "WARNING: You may still have bugs with product id #{product.id} (#{product.name}) until you run rake db:fix_orphan_line_items"
+        end
+      end
+    end
+
+    Harpiya::Variant.only_deleted.each do |variant|
+      # determine if there is a slug duplicate
+      the_dup = Harpiya::Variant.find_by(sku: variant.sku)
+      if the_dup.nil?
+        # check to see if there are line items attached to any variants
+        if variant.line_items.any?
+          puts "recovering deleted variant id #{variant.id} ... this will un-delete the record and set it to be discontinued"
+          old_deleted = variant.deleted_at
+          variant.update_column(:deleted_at, nil) # for some reason .recover doesn't appear to be a method
+          variant.update_column(:discontinue_on, old_deleted)
+        else
+          puts "leaving variant id #{variant.id} deleted because there are no line items attached to it..."
+        end
+      else
+        puts "leaving variant id #{variant.id} deleted because there is a duplicate SKU for '#{variant.sku}' (variant id #{the_dup.id}) "
+        if variant.line_items.any?
+          puts "WARNING: You may still have bugs with variant id #{variant.id} (#{variant.name}) until you run rake db:fix_orphan_line_items"
+        end
+      end
+    end
+  end
+
+  def down
+    execute "UPDATE harpiya_products SET deleted_at = discontinue_on WHERE deleted_at IS NULL"
+    execute "UPDATE harpiya_variants SET deleted_at = discontinue_on WHERE deleted_at IS NULL"
+
+    remove_column :harpiya_products, :discontinue_on
+    remove_column :harpiya_variants, :discontinue_on
+  end
+end

data/db/migrate/20151220072838_remove_shipping_method_id_from_harpiya_orders.rb

@@ -0,0 +1,13 @@
+class RemoveShippingMethodIdFromHarpiyaOrders < ActiveRecord::Migration[4.2]
+  def up
+    if column_exists?(:harpiya_orders, :shipping_method_id, :integer)
+      remove_column :harpiya_orders, :shipping_method_id, :integer
+    end
+  end
+
+  def down
+    unless column_exists?(:harpiya_orders, :shipping_method_id, :integer)
+      add_column :harpiya_orders, :shipping_method_id, :integer
+    end
+  end
+end

data/db/migrate/20160207191757_add_id_column_to_earlier_habtm_tables.rb

@@ -0,0 +1,16 @@
+class AddIdColumnToEarlierHabtmTables < ActiveRecord::Migration[4.2]
+  def up
+    add_column :harpiya_option_type_prototypes, :id, :primary_key
+    add_column :harpiya_option_value_variants, :id, :primary_key
+    add_column :harpiya_order_promotions, :id, :primary_key
+    add_column :harpiya_product_promotion_rules, :id, :primary_key
+    add_column :harpiya_promotion_rule_users, :id, :primary_key
+    add_column :harpiya_property_prototypes, :id, :primary_key
+    add_column :harpiya_role_users, :id, :primary_key
+    add_column :harpiya_shipping_method_zones, :id, :primary_key
+  end
+
+  def down
+    raise ActiveRecord::IrreversibleMigration
+  end
+end

data/db/migrate/20160219165458_add_indexes.rb

@@ -0,0 +1,14 @@
+class AddIndexes < ActiveRecord::Migration[4.2]
+  def change
+    add_index :harpiya_assets, :position
+    add_index :harpiya_option_types, :name
+    add_index :harpiya_option_values, :name
+    add_index :harpiya_prices, :variant_id
+    add_index :harpiya_properties, :name
+    add_index :harpiya_roles, :name
+    add_index :harpiya_shipping_categories, :name
+    add_index :harpiya_taxons, :lft
+    add_index :harpiya_taxons, :rgt
+    add_index :harpiya_taxons, :name
+  end
+end

data/db/migrate/20160509064646_remove_counter_cache_from_harpiya_variants_to_harpiya_stock_items.rb

@@ -0,0 +1,10 @@
+class RemoveCounterCacheFromHarpiyaVariantsToHarpiyaStockItems < ActiveRecord::Migration[4.2]
+  def up
+    if column_exists?(:harpiya_variants, :stock_items_count)
+      remove_column :harpiya_variants, :stock_items_count
+    end
+  end
+
+  def down
+  end
+end

data/db/migrate/20160608090604_add_zipcode_required_to_harpiya_countries.rb

@@ -0,0 +1,7 @@
+class AddZipcodeRequiredToHarpiyaCountries < ActiveRecord::Migration[4.2]
+  def change
+    add_column :harpiya_countries, :zipcode_required, :boolean, default: true
+    Harpiya::Country.reset_column_information
+    Harpiya::Country.where(iso: Harpiya::Address::NO_ZIPCODE_ISO_CODES).update_all(zipcode_required: false)
+  end
+end

data/db/migrate/20161014145148_add_created_at_to_variant.rb

@@ -0,0 +1,8 @@
+class AddCreatedAtToVariant < ActiveRecord::Migration[5.0]
+  def change
+    add_column :harpiya_variants, :created_at, :datetime
+    Harpiya::Variant.reset_column_information
+    Harpiya::Variant.unscoped.where.not(updated_at: nil).update_all('created_at = updated_at')
+    Harpiya::Variant.unscoped.where(updated_at: nil).update_all(created_at: Time.current, updated_at: Time.current)
+  end
+end

data/db/migrate/20161014152814_add_null_false_to_harpiya_variants_timestamps.rb

@@ -0,0 +1,6 @@
+class AddNullFalseToHarpiyaVariantsTimestamps < ActiveRecord::Migration[5.0]
+  def change
+    change_column_null :harpiya_variants, :created_at, false
+    change_column_null :harpiya_variants, :updated_at, false
+  end
+end

data/db/migrate/20161125065505_add_quantity_to_inventory_units.rb

@@ -0,0 +1,5 @@
+class AddQuantityToInventoryUnits < ActiveRecord::Migration[5.0]
+  def change
+    add_column :harpiya_inventory_units, :quantity, :integer, default: 1
+  end
+end

data/db/migrate/20170119122701_add_original_return_item_id_to_harpiya_inventory_units.rb

@@ -0,0 +1,29 @@
+class AddOriginalReturnItemIdToHarpiyaInventoryUnits < ActiveRecord::Migration[5.0]
+  def up
+    add_reference :harpiya_inventory_units, :original_return_item, references: :harpiya_return_items, index: true
+
+    Harpiya::InventoryUnit.reset_column_information
+
+    Harpiya::ReturnItem.where.not(exchange_inventory_unit_id: nil).find_each do |return_item|
+      if (inventory_unit = Harpiya::InventoryUnit.find_by(id: return_item.exchange_inventory_unit_id)).present?
+        inventory_unit.update_column(:original_return_item_id, return_item.id)
+      end
+    end
+
+    remove_column :harpiya_return_items, :exchange_inventory_unit_id
+  end
+
+  def down
+    add_reference :harpiya_return_items, :exchange_inventory_unit, references: :harpiya_inventory_units, index: true
+
+    Harpiya::InventoryUnit.reset_column_information
+
+    Harpiya::InventoryUnit.where.not(original_return_item_id: nil).find_each do |inventory_unit|
+      if (return_item = Harpiya::ReturnItem.find_by(id: inventory_unit.original_return_item_id)).present?
+        return_item.update_column(:exchange_inventory_unit_id, inventory_unit.id)
+      end
+    end
+
+    remove_reference :harpiya_inventory_units, :original_return_item
+  end
+end

data/db/migrate/20170315152755_add_unique_index_on_number_to_harpiya_orders.rb

@@ -0,0 +1,16 @@
+class AddUniqueIndexOnNumberToHarpiyaOrders < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:harpiya_orders, :number, unique: true)
+      numbers = Harpiya::Order.group(:number).having('sum(1) > 1').pluck(:number)
+      orders = Harpiya::Order.where(number: numbers)
+
+      orders.find_each do |order|
+        order.number = order.class.number_generator.method(:generate_permalink).call(order.class)
+        order.save
+      end
+
+      remove_index :harpiya_orders, :number if index_exists?(:harpiya_orders, :number)
+      add_index :harpiya_orders, :number, unique: true
+    end
+  end
+end

data/db/migrate/20170316154338_add_unique_index_on_number_to_harpiya_stock_transfer.rb

@@ -0,0 +1,16 @@
+class AddUniqueIndexOnNumberToHarpiyaStockTransfer < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:harpiya_stock_transfers, :number, unique: true)
+      numbers = Harpiya::StockTransfer.group(:number).having('sum(1) > 1').pluck(:number)
+      transfers = Harpiya::StockTransfer.where(number: numbers)
+
+      transfers.find_each do |transfer|
+        transfer.number = transfer.class.number_generator.method(:generate_permalink).call(transfer.class)
+        transfer.save
+      end
+
+      remove_index :harpiya_stock_transfers, :number if index_exists?(:harpiya_stock_transfers, :number)
+      add_index :harpiya_stock_transfers, :number, unique: true
+    end
+  end
+end

data/db/migrate/20170316205511_add_unique_index_on_number_to_harpiya_shipment.rb

@@ -0,0 +1,16 @@
+class AddUniqueIndexOnNumberToHarpiyaShipment < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:harpiya_shipments, :number, unique: true)
+      numbers = Harpiya::Shipment.group(:number).having('sum(1) > 1').pluck(:number)
+      shipments = Harpiya::Shipment.where(number: numbers)
+
+      shipments.find_each do |shipment|
+        shipment.number = shipment.class.number_generator.method(:generate_permalink).call(shipment.class)
+        shipment.save
+      end
+
+      remove_index :harpiya_shipments, :number if index_exists?(:harpiya_shipments, :number)
+      add_index :harpiya_shipments, :number, unique: true
+    end
+  end
+end

data/db/migrate/20170320134043_add_unique_index_on_number_to_harpiya_payments.rb

@@ -0,0 +1,17 @@
+class AddUniqueIndexOnNumberToHarpiyaPayments < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:harpiya_payments, :number, unique: true)
+      # default scope in Harpiya::Payment disturbs Postgres, hence `unscoped` is needed.
+      numbers = Harpiya::Payment.unscoped.group(:number).having('sum(1) > 1').pluck(:number)
+      payments = Harpiya::Payment.where(number: numbers)
+
+      payments.find_each do |payment|
+        payment.number = payment.class.number_generator.method(:generate_permalink).call(payment.class)
+        payment.save
+      end
+
+      remove_index :harpiya_payments, :number if index_exists?(:harpiya_payments, :number)
+      add_index :harpiya_payments, :number, unique: true
+    end
+  end
+end

data/db/migrate/20170320142750_add_unique_index_on_number_to_harpiya_return_authorizations.rb

@@ -0,0 +1,16 @@
+class AddUniqueIndexOnNumberToHarpiyaReturnAuthorizations < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:harpiya_return_authorizations, :number, unique: true)
+      numbers = Harpiya::ReturnAuthorization.group(:number).having('sum(1) > 1').pluck(:number)
+      authorizations = Harpiya::ReturnAuthorization.where(number: numbers)
+
+      authorizations.find_each do |authorization|
+        authorization.number = authorization.class.number_generator.method(:generate_permalink).call(authorization.class)
+        authorization.save
+      end
+
+      remove_index :harpiya_return_authorizations, :number if index_exists?(:harpiya_return_authorizations, :number)
+      add_index :harpiya_return_authorizations, :number, unique: true
+    end
+  end
+end

data/db/migrate/20170320145040_add_unique_index_on_number_to_harpiya_customer_returns.rb

@@ -0,0 +1,16 @@
+class AddUniqueIndexOnNumberToHarpiyaCustomerReturns < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:harpiya_customer_returns, :number, unique: true)
+      numbers = Harpiya::CustomerReturn.group(:number).having('sum(1) > 1').pluck(:number)
+      returns = Harpiya::CustomerReturn.where(number: numbers)
+
+      returns.find_each do |r|
+        r.number = r.class.number_generator.method(:generate_permalink).call(r.class)
+        r.save
+      end
+
+      remove_index :harpiya_customer_returns, :number if index_exists?(:harpiya_customer_returns, :number)
+      add_index :harpiya_customer_returns, :number, unique: true
+    end
+  end
+end

data/db/migrate/20170320145518_add_unique_index_on_number_to_harpiya_reimbursements.rb

@@ -0,0 +1,16 @@
+class AddUniqueIndexOnNumberToHarpiyaReimbursements < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:harpiya_reimbursements, :number, unique: true)
+      numbers = Harpiya::Reimbursement.group(:number).having('sum(1) > 1').pluck(:number)
+      reimbursements = Harpiya::Reimbursement.where(number: numbers)
+
+      reimbursements.find_each do |reimbursement|
+        reimbursement.number = reimbursement.class.number_generator.method(:generate_permalink).call(reimbursement.class)
+        reimbursement.save
+      end
+
+      remove_index :harpiya_reimbursements, :number if index_exists?(:harpiya_reimbursements, :number)
+      add_index :harpiya_reimbursements, :number, unique: true
+    end
+  end
+end

data/db/migrate/20170323151450_add_missing_unique_indexes_for_unique_attributes.rb

@@ -0,0 +1,37 @@
+class AddMissingUniqueIndexesForUniqueAttributes < ActiveRecord::Migration[5.0]
+  def change
+    tables = {
+      country: [:name, :iso_name],
+      refund_reason: [:name],
+      reimbursement_type: [:name],
+      return_authorization_reason: [:name],
+      role: [:name],
+      store: [:code]
+    }
+
+    tables.each do |table, columns|
+      table_class = "Harpiya::#{table.to_s.classify}".constantize
+      table_name = table_class.table_name
+
+      columns.each do |column|
+        unless index_exists?(table_name, column, unique: true)
+          attributes = table_class.unscoped.group(column).having('sum(1) > 1').pluck(column)
+          instances = table_class.where(column => [nil, attributes])
+
+          instances.find_each do |instance|
+            column_value = 'Unique String ' + SecureRandom.urlsafe_base64(8).upcase.delete('/+=_-')[0, 8]
+            instance.send("#{column}=", column_value)
+            instance.save
+          end
+
+          remove_index table_name, column if index_exists?(table_name, column)
+          if supports_expression_index?
+            add_index table_name, "(lower(#{column}))", unique: true
+          else
+            add_index table_name, column, unique: true
+          end
+        end
+      end
+    end
+  end
+end