spree_purchase_order 3.2.0 → 3.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 727d8ce24fbc909265ca679814e933c92d6201ae
-  data.tar.gz: 12c9c0ff01500a18456caebd59ecea84617a9e43
+  metadata.gz: 45b1dc4a8ec8ad4a0570995dc77645b4debe6e05
+  data.tar.gz: 3ffdc47367fa2e2158a958a11caf5ffab7d08f8e
 SHA512:
-  metadata.gz: 73cc1a2fe0a348047f2939a1bc1b134b5ec1f63294f93916601c27c90ff94fef928824b1ad4598061d2f8a5c10dcb3362157950d0b2f41bc80e66ae6e25f3170
-  data.tar.gz: 2d5d717898a90809f107b9862ea24316c435589a98eba3ab30738b309e3ae5ac60d79e2f52b67d48f2c9e765a22521d4b5ee5b7d64fecac6f71f6cdd2f8dd613
+  metadata.gz: ac48dbd487e74079b9058fd8681019db9c7f228c2210a3217bab19afcc32a86659d4d19ee8a44be4af73dabdb52a9b4728bde4bbeda7e9a51631744f494ab56e
+  data.tar.gz: 61ae6bce21914b3b0e870060c88b7eee34f276f50e181f99e7a131a9ef974db871de0ab9fa7a6850c7faed131a0fa0395ce9635716ae9d11b997ea20d5dd944d

data/db/migrate/20160111123141_create_spree_purchase_orders.rb

@@ -1,4 +1,4 @@
-class CreateSpreePurchaseOrders < ActiveRecord::Migration
+class CreateSpreePurchaseOrders < ActiveRecord::Migration[4.2]
   def change
     create_table :spree_purchase_orders do |t|
       t.string :po_number

data/db/migrate/20161214190805_add_payment_method_id_to_spree_purchase_orders.rb

@@ -1,4 +1,4 @@
-class AddPaymentMethodIdToSpreePurchaseOrders < ActiveRecord::Migration
+class AddPaymentMethodIdToSpreePurchaseOrders < ActiveRecord::Migration[4.2]
   def change
     add_column :spree_purchase_orders, :payment_method_id, :integer
     add_index :spree_purchase_orders, :payment_method_id

data/db/migrate/20161214191512_add_user_id_to_spree_purchase_orders.rb

@@ -1,4 +1,4 @@
-class AddUserIdToSpreePurchaseOrders < ActiveRecord::Migration
+class AddUserIdToSpreePurchaseOrders < ActiveRecord::Migration[4.2]
   def change
     add_column :spree_purchase_orders, :user_id, :integer
     add_index :spree_purchase_orders, :user_id

data/lib/spree_purchase_order/version.rb

@@ -1,3 +1,3 @@
 module SpreePurchaseOrder
-  VERSION = '3.2.0'
+  VERSION = '3.3.0'
 end

data/spec/dummy/config/application.rb

@@ -21,8 +21,6 @@ module Dummy
         Rails.configuration.cache_classes ? require(c) : load(c)
       end
     end
-
-    config.active_record.raise_in_transactional_callbacks = true
   end
 end
 

data/spec/dummy/db/migrate/20180516191805_add_quantity_to_inventory_units.spree.rb

@@ -0,0 +1,6 @@
+# This migration comes from spree (originally 20161125065505)
+class AddQuantityToInventoryUnits < ActiveRecord::Migration[5.0]
+  def change
+    add_column :spree_inventory_units, :quantity, :integer, default: 1
+  end
+end

data/spec/dummy/db/migrate/20180516191806_add_original_return_item_id_to_spree_inventory_units.spree.rb

@@ -0,0 +1,30 @@
+# This migration comes from spree (originally 20170119122701)
+class AddOriginalReturnItemIdToSpreeInventoryUnits < ActiveRecord::Migration[5.0]
+  def up
+    add_reference :spree_inventory_units, :original_return_item, references: :spree_return_items, index: true
+
+    Spree::InventoryUnit.reset_column_information
+
+    Spree::ReturnItem.where.not(exchange_inventory_unit_id: nil).find_each do |return_item|
+      if (inventory_unit = Spree::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 :spree_return_items, :exchange_inventory_unit_id
+  end
+
+  def down
+    add_reference :spree_return_items, :exchange_inventory_unit, references: :spree_inventory_units, index: true
+
+    Spree::InventoryUnit.reset_column_information
+
+    Spree::InventoryUnit.where.not(original_return_item_id: nil).find_each do |inventory_unit|
+      if (return_item = Spree::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 :spree_inventory_units, :original_return_item
+  end
+end

data/spec/dummy/db/migrate/20180516191807_add_unique_index_on_number_to_spree_orders.spree.rb

@@ -0,0 +1,17 @@
+# This migration comes from spree (originally 20170315152755)
+class AddUniqueIndexOnNumberToSpreeOrders < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:spree_orders, :number, unique: true)
+      numbers = Spree::Order.group(:number).having('sum(1) > 1').pluck(:number)
+      orders = Spree::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 :spree_orders, :number if index_exists?(:spree_orders, :number)
+      add_index :spree_orders, :number, unique: true
+    end
+  end
+end

data/spec/dummy/db/migrate/20180516191808_add_unique_index_on_number_to_spree_stock_transfer.spree.rb

@@ -0,0 +1,17 @@
+# This migration comes from spree (originally 20170316154338)
+class AddUniqueIndexOnNumberToSpreeStockTransfer < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:spree_stock_transfers, :number, unique: true)
+      numbers = Spree::StockTransfer.group(:number).having('sum(1) > 1').pluck(:number)
+      transfers = Spree::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 :spree_stock_transfers, :number if index_exists?(:spree_stock_transfers, :number)
+      add_index :spree_stock_transfers, :number, unique: true
+    end
+  end
+end

data/spec/dummy/db/migrate/20180516191809_add_unique_index_on_number_to_spree_shipment.spree.rb

@@ -0,0 +1,17 @@
+# This migration comes from spree (originally 20170316205511)
+class AddUniqueIndexOnNumberToSpreeShipment < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:spree_shipments, :number, unique: true)
+      numbers = Spree::Shipment.group(:number).having('sum(1) > 1').pluck(:number)
+      shipments = Spree::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 :spree_shipments, :number if index_exists?(:spree_shipments, :number)
+      add_index :spree_shipments, :number, unique: true
+    end
+  end
+end

data/spec/dummy/db/migrate/20180516191810_add_unique_index_on_number_to_spree_payments.spree.rb

@@ -0,0 +1,18 @@
+# This migration comes from spree (originally 20170320134043)
+class AddUniqueIndexOnNumberToSpreePayments < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:spree_payments, :number, unique: true)
+      # default scope in Spree::Payment disturbs Postgres, hence `unscoped` is needed.
+      numbers = Spree::Payment.unscoped.group(:number).having('sum(1) > 1').pluck(:number)
+      payments = Spree::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 :spree_payments, :number if index_exists?(:spree_payments, :number)
+      add_index :spree_payments, :number, unique: true
+    end
+  end
+end

data/spec/dummy/db/migrate/20180516191811_add_unique_index_on_number_to_spree_return_authorizations.spree.rb

@@ -0,0 +1,17 @@
+# This migration comes from spree (originally 20170320142750)
+class AddUniqueIndexOnNumberToSpreeReturnAuthorizations < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:spree_return_authorizations, :number, unique: true)
+      numbers = Spree::ReturnAuthorization.group(:number).having('sum(1) > 1').pluck(:number)
+      authorizations = Spree::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 :spree_return_authorizations, :number if index_exists?(:spree_return_authorizations, :number)
+      add_index :spree_return_authorizations, :number, unique: true
+    end
+  end
+end

data/spec/dummy/db/migrate/20180516191812_add_unique_index_on_number_to_spree_customer_returns.spree.rb

@@ -0,0 +1,17 @@
+# This migration comes from spree (originally 20170320145040)
+class AddUniqueIndexOnNumberToSpreeCustomerReturns < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:spree_customer_returns, :number, unique: true)
+      numbers = Spree::CustomerReturn.group(:number).having('sum(1) > 1').pluck(:number)
+      returns = Spree::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 :spree_customer_returns, :number if index_exists?(:spree_customer_returns, :number)
+      add_index :spree_customer_returns, :number, unique: true
+    end
+  end
+end

data/spec/dummy/db/migrate/20180516191813_add_unique_index_on_number_to_spree_reimbursements.spree.rb

@@ -0,0 +1,17 @@
+# This migration comes from spree (originally 20170320145518)
+class AddUniqueIndexOnNumberToSpreeReimbursements < ActiveRecord::Migration[5.0]
+  def change
+    unless index_exists?(:spree_reimbursements, :number, unique: true)
+      numbers = Spree::Reimbursement.group(:number).having('sum(1) > 1').pluck(:number)
+      reimbursements = Spree::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 :spree_reimbursements, :number if index_exists?(:spree_reimbursements, :number)
+      add_index :spree_reimbursements, :number, unique: true
+    end
+  end
+end

data/spec/dummy/db/migrate/20180516191814_add_missing_unique_indexes_for_unique_attributes.spree.rb

@@ -0,0 +1,38 @@
+# This migration comes from spree (originally 20170323151450)
+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 = "Spree::#{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

data/spec/dummy/db/migrate/20180516191815_add_index_on_stock_location_to_spree_customer_returns.spree.rb

@@ -0,0 +1,6 @@
+# This migration comes from spree (originally 20170329110859)
+class AddIndexOnStockLocationToSpreeCustomerReturns < ActiveRecord::Migration[5.0]
+  def change
+    add_index :spree_customer_returns, :stock_location_id
+  end
+end

data/spec/dummy/db/migrate/20180516191816_add_index_on_prototype_to_spree_option_type_prototype.spree.rb

@@ -0,0 +1,20 @@
+# This migration comes from spree (originally 20170329113917)
+class AddIndexOnPrototypeToSpreeOptionTypePrototype < ActiveRecord::Migration[5.0]
+  def change
+    duplicates = Spree::OptionTypePrototype.group(:prototype_id, :option_type_id).having('sum(1) > 1').size
+
+    duplicates.each do |f|
+      prototype_id, option_type_id = f.first
+      count = f.last - 1 # we want to leave one record
+      otp = Spree::OptionTypePrototype.where(prototype_id: prototype_id, option_type_id: option_type_id).last(count)
+      otp.map(&:destroy)
+    end
+
+    if index_exists? :spree_option_type_prototypes, [:prototype_id, :option_type_id]
+      remove_index :spree_option_type_prototypes, [:prototype_id, :option_type_id]
+      add_index :spree_option_type_prototypes, [:prototype_id, :option_type_id], unique: true, name: 'spree_option_type_prototypes_prototype_id_option_type_id'
+    end
+
+    add_index :spree_option_type_prototypes, :prototype_id
+  end
+end

data/spec/dummy/db/migrate/20180516191817_add_indexes_to_spree_option_value_variant.spree.rb

@@ -0,0 +1,20 @@
+# This migration comes from spree (originally 20170330082155)
+class AddIndexesToSpreeOptionValueVariant < ActiveRecord::Migration[5.0]
+  def change
+    duplicates = Spree::OptionValueVariant.group(:variant_id, :option_value_id).having('sum(1) > 1').size
+
+    duplicates.each do |f|
+      variant_id, option_value_id = f.first
+      count = f.last - 1 # we want to leave one record
+      ov = Spree::OptionValueVariant.where(variant_id: variant_id, option_value_id: option_value_id).last(count)
+      ov.map(&:destroy)
+    end
+
+    if index_exists? :spree_option_value_variants, [:variant_id, :option_value_id], name: "index_option_values_variants_on_variant_id_and_option_value_id"
+      remove_index :spree_option_value_variants, [:variant_id, :option_value_id]
+      add_index :spree_option_value_variants, [:variant_id, :option_value_id], unique: true, name: "index_option_values_variants_on_variant_id_and_option_value_id"
+    end
+
+    add_index :spree_option_value_variants, :variant_id
+  end
+end

data/spec/dummy/db/migrate/20180516191818_add_index_on_promotion_id_to_order_promotions.spree.rb

@@ -0,0 +1,6 @@
+# This migration comes from spree (originally 20170330132215)
+class AddIndexOnPromotionIdToOrderPromotions < ActiveRecord::Migration[5.0]
+  def change
+    add_index :spree_order_promotions, :promotion_id
+  end
+end

data/spec/dummy/db/migrate/20180516191819_add_indexes_for_property_prototype.spree.rb

@@ -0,0 +1,21 @@
+# This migration comes from spree (originally 20170331101758)
+class AddIndexesForPropertyPrototype < ActiveRecord::Migration[5.0]
+  def change
+    duplicates = Spree::PropertyPrototype.group(:prototype_id, :property_id).having('sum(1) > 1').size
+
+    duplicates.each do |f|
+      prototype_id, property_id = f.first
+      count = f.last - 1 # we want to leave one record
+      prototypes = Spree::PropertyPrototype.where(prototype_id: prototype_id, property_id: property_id).last(count)
+      prototypes.map(&:destroy)
+    end
+
+    if index_exists? :spree_property_prototypes, [:prototype_id, :property_id]
+      remove_index :spree_property_prototypes, [:prototype_id, :property_id]
+      add_index :spree_property_prototypes, [:prototype_id, :property_id], unique: true, name: 'index_property_prototypes_on_prototype_id_and_property_id'
+    end
+
+    add_index :spree_property_prototypes, :prototype_id
+    add_index :spree_property_prototypes, :property_id
+  end
+end

data/spec/dummy/db/migrate/20180516191820_add_index_for_prototype_id_to_prototype_taxons.spree.rb

@@ -0,0 +1,6 @@
+# This migration comes from spree (originally 20170331103334)
+class AddIndexForPrototypeIdToPrototypeTaxons < ActiveRecord::Migration[5.0]
+  def change
+    add_index :spree_prototype_taxons, :prototype_id
+  end
+end

data/spec/dummy/db/migrate/20180516191821_add_indexes_to_refunds.spree.rb

@@ -0,0 +1,7 @@
+# This migration comes from spree (originally 20170331110454)
+class AddIndexesToRefunds < ActiveRecord::Migration[5.0]
+  def change
+    add_index :spree_refunds, :payment_id
+    add_index :spree_refunds, :reimbursement_id
+  end
+end

data/spec/dummy/db/migrate/20180516191822_add_indexes_to_reimbursement_credits.spree.rb

@@ -0,0 +1,7 @@
+# This migration comes from spree (originally 20170331111757)
+class AddIndexesToReimbursementCredits < ActiveRecord::Migration[5.0]
+  def change
+    add_index :spree_reimbursement_credits, :reimbursement_id
+    add_index :spree_reimbursement_credits, [:creditable_id, :creditable_type], name: 'index_reimbursement_credits_on_creditable_id_and_type'
+  end
+end

data/spec/dummy/db/migrate/20180516191823_add_indexes_to_return_authorizations.spree.rb

@@ -0,0 +1,7 @@
+# This migration comes from spree (originally 20170331115246)
+class AddIndexesToReturnAuthorizations < ActiveRecord::Migration[5.0]
+  def change
+    add_index :spree_return_authorizations, :order_id
+    add_index :spree_return_authorizations, :stock_location_id
+  end
+end

data/spec/dummy/db/migrate/20180516191824_add_indexes_to_return_items.spree.rb

@@ -0,0 +1,12 @@
+# This migration comes from spree (originally 20170331120125)
+class AddIndexesToReturnItems < ActiveRecord::Migration[5.0]
+  def change
+    add_index :spree_return_items, :return_authorization_id
+    add_index :spree_return_items, :inventory_unit_id
+    add_index :spree_return_items, :reimbursement_id
+    add_index :spree_return_items, :exchange_variant_id
+    add_index :spree_return_items, :preferred_reimbursement_type_id
+    add_index :spree_return_items, :override_reimbursement_type_id
+  end
+end
+

data/spec/dummy/db/migrate/20180516191825_add_index_to_role_users.spree.rb

@@ -0,0 +1,19 @@
+# This migration comes from spree (originally 20170331121725)
+class AddIndexToRoleUsers < ActiveRecord::Migration[5.0]
+  def change
+
+    duplicates = Spree::RoleUser.group(:role_id, :user_id).having('sum(1) > 1').size
+
+    duplicates.each do |f|
+      role_id, user_id = f.first
+      count = f.last - 1 # we want to leave one record
+      roles = Spree::RoleUser.where(role_id: role_id, user_id: user_id).last(count)
+      roles.map(&:destroy)
+    end
+
+    if index_exists? :spree_role_users, [:role_id, :user_id]
+      remove_index :spree_role_users, [:role_id, :user_id]
+      add_index :spree_role_users, [:role_id, :user_id], unique: true
+    end
+  end
+end

data/spec/dummy/db/migrate/20180516191826_add_index_to_shipping_method_categories.spree.rb

@@ -0,0 +1,6 @@
+# This migration comes from spree (originally 20170331123625)
+class AddIndexToShippingMethodCategories < ActiveRecord::Migration[5.0]
+  def change
+    add_index :spree_shipping_method_categories, :shipping_category_id
+  end
+end

data/spec/dummy/db/migrate/20180516191827_add_index_to_shipping_method_zones.spree.rb

@@ -0,0 +1,21 @@
+# This migration comes from spree (originally 20170331123832)
+class AddIndexToShippingMethodZones < ActiveRecord::Migration[5.0]
+  def change
+    duplicates = Spree::ShippingMethodZone.group(:shipping_method_id, :zone_id).having('sum(1) > 1').size
+
+    duplicates.each do |f|
+      shipping_method_id, zone_id = f.first
+      count = f.last - 1 # we want to leave one record
+      zones = Spree::ShippingMethodZone.where(shipping_method_id: shipping_method_id, zone_id: zone_id).last(count)
+      zones.map(&:destroy)
+    end
+
+    if index_exists? :spree_shipping_method_zones, [:shipping_method_id, :zone_id]
+      remove_index :spree_shipping_method_zones, [:shipping_method_id, :zone_id]
+      add_index :spree_shipping_method_zones, [:shipping_method_id, :zone_id], unique: true
+    end
+
+    add_index :spree_shipping_method_zones, :zone_id
+    add_index :spree_shipping_method_zones, :shipping_method_id
+  end
+end

data/spec/dummy/db/migrate/20180516191828_add_index_to_spree_shipping_rates.spree.rb

@@ -0,0 +1,7 @@
+# This migration comes from spree (originally 20170331124251)
+class AddIndexToSpreeShippingRates < ActiveRecord::Migration[5.0]
+  def change
+    add_index :spree_shipping_rates, :shipment_id
+    add_index :spree_shipping_rates, :shipping_method_id
+  end
+end

data/spec/dummy/db/migrate/20180516191829_add_index_to_spree_stock_items.spree.rb

@@ -0,0 +1,6 @@
+# This migration comes from spree (originally 20170331124513)
+class AddIndexToSpreeStockItems < ActiveRecord::Migration[5.0]
+  def change
+    add_index :spree_stock_items, :stock_location_id
+  end
+end

data/spec/dummy/db/migrate/20180516191830_add_index_to_spree_stock_movement.spree.rb

@@ -0,0 +1,6 @@
+# This migration comes from spree (originally 20170331124924)
+class AddIndexToSpreeStockMovement < ActiveRecord::Migration[5.0]
+  def change
+    add_index :spree_stock_movements, [:originator_id, :originator_type], name: 'index_stock_movements_on_originator_id_and_originator_type'
+  end
+end

data/spec/dummy/db/migrate/20180516191831_change_indexes_on_friendly_id_slugs.spree.rb

@@ -0,0 +1,11 @@
+# This migration comes from spree (originally 20170413211707)
+class ChangeIndexesOnFriendlyIdSlugs < ActiveRecord::Migration[5.0]
+  def change
+    # Updating indexes to reflect changes in friendly_id v5.2
+    # See: https://github.com/norman/friendly_id/pull/694/commits/9f107f07ec9d2a58bda5a712b6e79a8d8013e0ab
+    remove_index :friendly_id_slugs, [:slug, :sluggable_type]
+    remove_index :friendly_id_slugs, [:slug, :sluggable_type, :scope]
+    add_index :friendly_id_slugs, [:slug, :sluggable_type], length: { name: 100, slug: 20, sluggable_type: 20 }
+    add_index :friendly_id_slugs, [:slug, :sluggable_type, :scope], length: { name: 100, slug: 20, sluggable_type: 20, scope: 20 }, unique: true
+  end
+end

data/spec/dummy/db/migrate/20180516191832_add_analytics_kind_to_spree_trackers.spree.rb

@@ -0,0 +1,6 @@
+# This migration comes from spree (originally 20170722102643)
+class AddAnalyticsKindToSpreeTrackers < ActiveRecord::Migration[5.1]
+  def change
+    add_column :spree_trackers, :kind, :integer, default: 0, null: false, index: true
+  end
+end

data/spec/dummy/db/migrate/20180516191833_rename_tracker_kind_field.spree.rb

@@ -0,0 +1,6 @@
+# This migration comes from spree (originally 20170727103056)
+class RenameTrackerKindField < ActiveRecord::Migration[5.1]
+  def change
+    rename_column :spree_trackers, :kind, :engine
+  end
+end