harpiya_core 4.3.0.alpha

data/db/migrate/20141009204607_add_store_id_to_orders.rb

@@ -0,0 +1,8 @@
+class AddStoreIdToOrders < ActiveRecord::Migration[4.2]
+  def change
+    add_column :harpiya_orders, :store_id, :integer
+    if Harpiya::Store.default.persisted?
+      Harpiya::Order.where(store_id: nil).update_all(store_id: Harpiya::Store.default.id)
+    end
+  end
+end

data/db/migrate/20141012083513_create_harpiya_taxons_prototypes.rb

@@ -0,0 +1,8 @@
+class CreateHarpiyaTaxonsPrototypes < ActiveRecord::Migration[4.2]
+  def change
+    create_table :harpiya_taxons_prototypes do |t|
+      t.belongs_to :taxon, index: true
+      t.belongs_to :prototype, index: true
+    end
+  end
+end

data/db/migrate/20141021194502_add_state_lock_version_to_order.rb

@@ -0,0 +1,5 @@
+class AddStateLockVersionToOrder < ActiveRecord::Migration[4.2]
+  def change
+    add_column :harpiya_orders, :state_lock_version, :integer, default: 0, null: false
+  end
+end

data/db/migrate/20141023005240_add_counter_cache_from_harpiya_variants_to_harpiya_stock_items.rb

@@ -0,0 +1,8 @@
+class AddCounterCacheFromHarpiyaVariantsToHarpiyaStockItems < ActiveRecord::Migration[4.2]
+  # This was unnecessary and was removed
+  def up
+  end
+
+  def down
+  end
+end

data/db/migrate/20141101231208_fix_adjustment_order_presence.rb

@@ -0,0 +1,13 @@
+class FixAdjustmentOrderPresence < ActiveRecord::Migration[4.2]
+  def change
+    say 'Fixing adjustments without direct order reference'
+    Harpiya::Adjustment.where(order: nil).find_each do |adjustment|
+      adjustable = adjustment.adjustable
+      if adjustable.is_a? Harpiya::Order
+        adjustment.update!(order_id: adjustable.id)
+      else
+        adjustment.update!(adjustable: adjustable.order)
+      end
+    end
+  end
+end

data/db/migrate/20141105213646_update_classifications_positions.rb

@@ -0,0 +1,9 @@
+class UpdateClassificationsPositions < ActiveRecord::Migration[4.2]
+  def up
+    Harpiya::Taxon.all.each do |taxon|
+      taxon.classifications.each_with_index do |c12n, i|
+        c12n.set_list_position(i + 1)
+      end
+    end
+  end
+end

data/db/migrate/20141120135441_add_guest_token_index_to_harpiya_orders.rb

@@ -0,0 +1,5 @@
+class AddGuestTokenIndexToHarpiyaOrders < ActiveRecord::Migration[4.2]
+  def change
+    add_index :harpiya_orders, :guest_token
+  end
+end

data/db/migrate/20141215232040_remove_token_permissions_table.rb

@@ -0,0 +1,6 @@
+class RemoveTokenPermissionsTable < ActiveRecord::Migration[4.2]
+  def change
+    # The MoveOrderTokenFromTokenizedPermission migration never dropped this.
+    drop_table :harpiya_tokenized_permissions
+  end
+end

data/db/migrate/20141215235502_remove_extra_products_slug_index.rb

@@ -0,0 +1,5 @@
+class RemoveExtraProductsSlugIndex < ActiveRecord::Migration[4.2]
+  def change
+    remove_index :harpiya_products, name: :permalink_idx_unique
+  end
+end

data/db/migrate/20141217215630_update_product_slug_index.rb

@@ -0,0 +1,6 @@
+class UpdateProductSlugIndex < ActiveRecord::Migration[4.2]
+  def change
+    remove_index :harpiya_products, :slug if index_exists?(:harpiya_products, :slug)
+    add_index :harpiya_products, :slug, unique: true
+  end
+end

data/db/migrate/20141218025915_rename_identifier_to_number_for_payment.rb

@@ -0,0 +1,5 @@
+class RenameIdentifierToNumberForPayment < ActiveRecord::Migration[4.2]
+  def change
+    rename_column :harpiya_payments, :identifier, :number
+  end
+end

data/db/migrate/20150118210639_create_harpiya_store_credits.rb

@@ -0,0 +1,24 @@
+class CreateHarpiyaStoreCredits < ActiveRecord::Migration[4.2]
+  def change
+    create_table :harpiya_store_credits do |t|
+      t.references :user
+      t.references :category
+      t.references :created_by
+      t.decimal :amount, precision: 8, scale: 2, default: 0.0, null: false
+      t.decimal :amount_used, precision: 8, scale: 2, default: 0.0, null: false
+      t.text :memo
+      t.datetime :deleted_at
+      t.string :currency
+      t.decimal :amount_authorized, precision: 8, scale: 2, default: 0.0, null: false
+      t.integer :originator_id
+      t.string :originator_type
+      t.integer :type_id
+      t.timestamps null: false, precision: 6
+    end
+
+    add_index :harpiya_store_credits, :deleted_at
+    add_index :harpiya_store_credits, :user_id
+    add_index :harpiya_store_credits, :type_id
+    add_index :harpiya_store_credits, [:originator_id, :originator_type], name: :harpiya_store_credits_originator
+  end
+end

data/db/migrate/20150118211500_create_harpiya_store_credit_categories.rb

@@ -0,0 +1,8 @@
+class CreateHarpiyaStoreCreditCategories < ActiveRecord::Migration[4.2]
+  def change
+    create_table :harpiya_store_credit_categories do |t|
+      t.string :name
+      t.timestamps null: false, precision: 6
+    end
+  end
+end

data/db/migrate/20150118212051_create_harpiya_store_credit_events.rb

@@ -0,0 +1,17 @@
+class CreateHarpiyaStoreCreditEvents < ActiveRecord::Migration[4.2]
+  def change
+    create_table :harpiya_store_credit_events do |t|
+      t.integer  :store_credit_id,    null: false
+      t.string   :action,             null: false
+      t.decimal  :amount,             precision: 8,  scale: 2
+      t.string   :authorization_code, null: false
+      t.decimal  :user_total_amount,  precision: 8, scale: 2, default: 0.0, null: false
+      t.integer  :originator_id
+      t.string   :originator_type
+      t.datetime :deleted_at
+      t.timestamps null: false, precision: 6
+    end
+    add_index :harpiya_store_credit_events, :store_credit_id
+    add_index :harpiya_store_credit_events, [:originator_id, :originator_type], name: :harpiya_store_credit_events_originator
+  end
+end

data/db/migrate/20150118212101_create_harpiya_store_credit_types.rb

@@ -0,0 +1,10 @@
+class CreateHarpiyaStoreCreditTypes < ActiveRecord::Migration[4.2]
+  def change
+    create_table :harpiya_store_credit_types do |t|
+      t.string :name
+      t.integer :priority
+      t.timestamps null: false, precision: 6
+    end
+    add_index :harpiya_store_credit_types, :priority
+  end
+end

data/db/migrate/20150121022521_remove_environment_from_payment_method.rb

@@ -0,0 +1,6 @@
+class RemoveEnvironmentFromPaymentMethod < ActiveRecord::Migration[4.2]
+  def up
+    Harpiya::PaymentMethod.where('environment != ?', Rails.env).update_all(active: false)
+    remove_column :harpiya_payment_methods, :environment
+  end
+end

data/db/migrate/20150122145607_add_resellable_to_return_items.rb

@@ -0,0 +1,5 @@
+class AddResellableToReturnItems < ActiveRecord::Migration[4.2]
+  def change
+    add_column :harpiya_return_items, :resellable, :boolean, default: true, null: false
+  end
+end

data/db/migrate/20150122202432_add_code_to_harpiya_promotion_categories.rb

@@ -0,0 +1,5 @@
+class AddCodeToHarpiyaPromotionCategories < ActiveRecord::Migration[4.2]
+  def change
+    add_column :harpiya_promotion_categories, :code, :string
+  end
+end

data/db/migrate/20150128032538_remove_environment_from_tracker.rb

@@ -0,0 +1,8 @@
+class RemoveEnvironmentFromTracker < ActiveRecord::Migration[4.2]
+  class Harpiya::Tracker < Harpiya::Base; end
+
+  def up
+    Harpiya::Tracker.where('environment != ?', Rails.env).update_all(active: false)
+    remove_column :harpiya_trackers, :environment
+  end
+end

data/db/migrate/20150128060325_remove_harpiya_configurations.rb

@@ -0,0 +1,16 @@
+class RemoveHarpiyaConfigurations < ActiveRecord::Migration[4.2]
+  def up
+    drop_table "harpiya_configurations"
+  end
+
+  def down
+    create_table "harpiya_configurations", force: true do |t|
+      t.string   "name"
+      t.string   "type",       limit: 50
+      t.datetime "created_at"
+      t.datetime "updated_at"
+    end
+
+    add_index "harpiya_configurations", ["name", "type"], name: "index_harpiya_configurations_on_name_and_type"
+  end
+end

data/db/migrate/20150216173445_add_index_to_harpiya_stock_items_variant_id.rb

@@ -0,0 +1,13 @@
+class AddIndexToHarpiyaStockItemsVariantId < ActiveRecord::Migration[4.2]
+  def up
+    unless index_exists? :harpiya_stock_items, :variant_id
+      add_index :harpiya_stock_items, :variant_id
+    end
+  end
+
+  def down
+    if index_exists? :harpiya_stock_items, :variant_id
+      remove_index :harpiya_stock_items, :variant_id
+    end
+  end
+end

data/db/migrate/20150309161154_ensure_payments_have_numbers.rb

@@ -0,0 +1,13 @@
+class EnsurePaymentsHaveNumbers < ActiveRecord::Migration[4.2]
+  def change
+    add_index :harpiya_payments, :number unless index_exists?(:harpiya_payments, :number)
+    Harpiya::Payment.where(number: nil).find_each do |payment|
+      begin
+        payment.save! # to generate a new number we need to save the record
+      rescue ActiveRecord::RecordNotSaved
+        Rails.logger.error("Payment with ID = #{payment.id} couldn't be saved")
+        Rails.logger.error(payment.errors.full_messages.to_sentence)
+      end
+    end
+  end
+end

data/db/migrate/20150314013438_add_missing_indexes_on_harpiya_tables.rb

@@ -0,0 +1,67 @@
+class AddMissingIndexesOnHarpiyaTables < ActiveRecord::Migration[4.2]
+  def change
+    if data_source_exists?(:harpiya_promotion_rules_users) && !index_exists?(:harpiya_promotion_rules_users,
+                                                                     [:user_id, :promotion_rule_id],
+                                                                     name: 'index_promotion_rules_users_on_user_id_and_promotion_rule_id')
+      add_index :harpiya_promotion_rules_users,
+                [:user_id, :promotion_rule_id],
+                name: 'index_promotion_rules_users_on_user_id_and_promotion_rule_id'
+    end
+
+    if data_source_exists?(:harpiya_products_promotion_rules) && !index_exists?(:harpiya_products_promotion_rules,
+                                                                        [:promotion_rule_id, :product_id],
+                                                                        name: 'index_products_promotion_rules_on_promotion_rule_and_product')
+      add_index :harpiya_products_promotion_rules,
+                [:promotion_rule_id, :product_id],
+                name: 'index_products_promotion_rules_on_promotion_rule_and_product'
+    end
+
+    unless index_exists? :harpiya_orders, :canceler_id
+      add_index :harpiya_orders, :canceler_id
+    end
+
+    unless index_exists? :harpiya_orders, :store_id
+      add_index :harpiya_orders, :store_id
+    end
+
+    if data_source_exists?(:harpiya_orders_promotions) && !index_exists?(:harpiya_orders_promotions, [:promotion_id, :order_id])
+      add_index :harpiya_orders_promotions, [:promotion_id, :order_id]
+    end
+
+    if data_source_exists?(:harpiya_properties_prototypes) && !index_exists?(:harpiya_properties_prototypes, :prototype_id)
+      add_index :harpiya_properties_prototypes, :prototype_id
+    end
+
+    if data_source_exists?(:harpiya_properties_prototypes) && !index_exists?(:harpiya_properties_prototypes,
+                                                                     [:prototype_id, :property_id],
+                                                                     name: 'index_properties_prototypes_on_prototype_and_property')
+      add_index :harpiya_properties_prototypes,
+                [:prototype_id, :property_id],
+                name: 'index_properties_prototypes_on_prototype_and_property'
+    end
+
+    if data_source_exists?(:harpiya_taxons_prototypes) && !index_exists?(:harpiya_taxons_prototypes, [:prototype_id, :taxon_id])
+      add_index :harpiya_taxons_prototypes, [:prototype_id, :taxon_id]
+    end
+
+    if data_source_exists?(:harpiya_option_types_prototypes) && !index_exists?(:harpiya_option_types_prototypes, :prototype_id)
+      add_index :harpiya_option_types_prototypes, :prototype_id
+    end
+
+    if data_source_exists?(:harpiya_option_types_prototypes) && !index_exists?(:harpiya_option_types_prototypes,
+                                                                       [:prototype_id, :option_type_id],
+                                                                       name: 'index_option_types_prototypes_on_prototype_and_option_type')
+      add_index :harpiya_option_types_prototypes,
+                [:prototype_id, :option_type_id],
+                name: 'index_option_types_prototypes_on_prototype_and_option_type'
+    end
+
+    if data_source_exists?(:harpiya_option_values_variants) && !index_exists?(:harpiya_option_values_variants,
+                                                                       [:option_value_id, :variant_id],
+                                                                       name: 'index_option_values_variants_on_option_value_and_variant')
+      add_index :harpiya_option_values_variants,
+                [:option_value_id, :variant_id],
+                name: 'index_option_values_variants_on_option_value_and_variant'
+    end
+  end
+end

data/db/migrate/20150317174308_remove_duplicated_indexes_from_multi_columns.rb

@@ -0,0 +1,18 @@
+class RemoveDuplicatedIndexesFromMultiColumns < ActiveRecord::Migration[4.2]
+  def change
+    remove_index :harpiya_adjustments, name: "index_adjustments_on_order_id"
+    remove_index :harpiya_option_types_prototypes, :prototype_id
+    add_index :harpiya_option_types_prototypes, :option_type_id
+    remove_index :harpiya_option_values_variants, name: 'index_option_values_variants_on_option_value_and_variant'
+    remove_index :harpiya_option_values_variants, :variant_id
+    add_index :harpiya_option_values_variants, :option_value_id
+    remove_index :harpiya_orders, :user_id
+    remove_index :harpiya_orders_promotions, [:order_id, :promotion_id]
+    add_index :harpiya_orders_promotions, :order_id
+    remove_index :harpiya_products_promotion_rules, name: "index_products_promotion_rules_on_promotion_rule_id"
+    remove_index :harpiya_promotion_rules_users, name: "index_promotion_rules_users_on_user_id"
+    remove_index :harpiya_properties_prototypes, :prototype_id
+    remove_index :harpiya_stock_items, :stock_location_id
+    remove_index :harpiya_taxons_prototypes, :prototype_id
+  end
+end

data/db/migrate/20150324104002_remove_user_index_from_harpiya_state_changes.rb

@@ -0,0 +1,14 @@
+class RemoveUserIndexFromHarpiyaStateChanges < ActiveRecord::Migration[4.2]
+  def up
+    if index_exists? :harpiya_state_changes, :user_id
+      remove_index :harpiya_state_changes, :user_id
+    end
+
+  end
+
+  def down
+    unless index_exists? :harpiya_state_changes, :user_id
+      add_index :harpiya_state_changes, :user_id
+    end
+  end
+end

data/db/migrate/20150515211137_fix_adjustment_order_id.rb

@@ -0,0 +1,70 @@
+class FixAdjustmentOrderId < ActiveRecord::Migration[4.2]
+  def change
+    say 'Populate order_id from adjustable_id where appropriate'
+    execute(<<-SQL.squish)
+      UPDATE
+        harpiya_adjustments
+      SET
+        order_id = adjustable_id
+      WHERE
+        adjustable_type = 'Harpiya::Order'
+      ;
+    SQL
+
+    # Submitter of change does not care about MySQL, as it is not officially supported.
+    # Still harpiya officials decided to provide a working code path for MySQL users, hence
+    # submitter made a AR code path he could validate on PostgreSQL.
+    #
+    # Whoever runs a big enough MySQL installation where the AR solution hurts:
+    # Will have to write a better MySQL specific equivalent.
+    if Harpiya::Order.connection.adapter_name.eql?('MySQL')
+      Harpiya::Adjustment.where(adjustable_type: 'Harpiya::LineItem').find_each do |adjustment|
+        adjustment.update_columns(order_id: Harpiya::LineItem.find(adjustment.adjustable_id).order_id)
+      end
+    else
+      execute(<<-SQL.squish)
+        UPDATE
+          harpiya_adjustments
+        SET
+          order_id =
+            (SELECT order_id FROM harpiya_line_items WHERE harpiya_line_items.id = harpiya_adjustments.adjustable_id)
+        WHERE
+          adjustable_type = 'Harpiya::LineItem'
+      SQL
+    end
+
+    say 'Fix schema for harpiya_adjustments order_id column'
+    change_table :harpiya_adjustments do |t|
+      t.change :order_id, :integer, null: false
+    end
+
+    # Improved schema for postgresql, uncomment if you like it:
+    #
+    # # Negated Logical implication.
+    # #
+    # # When adjustable_type is 'Harpiya::Order' (p) the adjustable_id must be order_id (q).
+    # #
+    # # When adjustable_type is NOT 'Harpiya::Order' the adjustable id allowed to be any value (including of order_id in
+    # # case foreign keys match). XOR does not work here.
+    # #
+    # # Postgresql does not have an operator for logical implication. So we need to build the following truth table
+    # # via AND with OR:
+    # #
+    # #  p q | CHECK = !(p -> q)
+    # #  -----------
+    # #  t t | t
+    # #  t f | f
+    # #  f t | t
+    # #  f f | t
+    # #
+    # # According to de-morgans law the logical implication q -> p is equivalent to !p || q
+    # #
+    # execute(<<-SQL.squish)
+    #   ALTER TABLE ONLY harpiya_adjustments
+    #    ADD CONSTRAINT fk_harpiya_adjustments FOREIGN KEY (order_id)
+    #      REFERENCES harpiya_orders(id) ON UPDATE RESTRICT ON DELETE RESTRICT,
+    #    ADD CONSTRAINT check_harpiya_adjustments_order_id CHECK
+    #      (adjustable_type <> 'Harpiya::Order' OR order_id = adjustable_id);
+    # SQL
+  end
+end

data/db/migrate/20150522071831_add_position_to_harpiya_payment_methods.rb

@@ -0,0 +1,5 @@
+class AddPositionToHarpiyaPaymentMethods < ActiveRecord::Migration[4.2]
+  def change
+    add_column :harpiya_payment_methods, :position, :integer, default: 0
+  end
+end

data/db/migrate/20150522181728_add_deleted_at_to_friendly_id_slugs.rb

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

data/db/migrate/20150609093816_increase_scale_on_pre_tax_amounts.rb

@@ -0,0 +1,16 @@
+class IncreaseScaleOnPreTaxAmounts < ActiveRecord::Migration[4.2]
+  def change
+    # set pre_tax_amount on shipments to discounted_amount - included_tax_total
+    # so that the null: false option on the shipment pre_tax_amount doesn't generate
+    # errors.
+    #
+    execute(<<-SQL)
+      UPDATE harpiya_shipments
+      SET pre_tax_amount = (cost + promo_total) - included_tax_total
+      WHERE pre_tax_amount IS NULL;
+    SQL
+
+    change_column :harpiya_line_items, :pre_tax_amount, :decimal, precision: 12, scale: 4, default: 0.0, null: false
+    change_column :harpiya_shipments, :pre_tax_amount, :decimal, precision: 12, scale: 4, default: 0.0, null: false
+  end
+end