From bdf462926d0ff77f002735d02ece82416cd98515 Mon Sep 17 00:00:00 2001
From: Rob Harrington <oeoeaio@gmail.com>
Date: Fri, 24 Apr 2015 13:05:01 +1000
Subject: [PATCH] Ensure that producer properties have a position

---
 ...ult_and_not_null_to_producer_properties_position.rb |  9 +++++++++
 db/schema.rb                                           | 10 +++++-----
 2 files changed, 14 insertions(+), 5 deletions(-)
 create mode 100644 db/migrate/20150424025907_add_default_and_not_null_to_producer_properties_position.rb

diff --git a/db/migrate/20150424025907_add_default_and_not_null_to_producer_properties_position.rb b/db/migrate/20150424025907_add_default_and_not_null_to_producer_properties_position.rb
new file mode 100644
index 0000000000..8c5a2d1f2c
--- /dev/null
+++ b/db/migrate/20150424025907_add_default_and_not_null_to_producer_properties_position.rb
@@ -0,0 +1,9 @@
+class AddDefaultAndNotNullToProducerPropertiesPosition < ActiveRecord::Migration
+  def change
+    ProducerProperty.where(position: nil).each do |producer_property|
+      producer_property.update_attribute(:position, 0)
+    end
+
+    change_column :producer_properties, :position, :integer, null: false, default: 0
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
index 627dd8aa8c..cb8b503583 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -11,7 +11,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20150422014819) do
+ActiveRecord::Schema.define(:version => 20150424025907) do
 
   create_table "adjustment_metadata", :force => true do |t|
     t.integer "adjustment_id"
@@ -373,9 +373,9 @@ ActiveRecord::Schema.define(:version => 20150422014819) do
     t.string   "value"
     t.integer  "producer_id"
     t.integer  "property_id"
-    t.integer  "position"
-    t.datetime "created_at",  :null => false
-    t.datetime "updated_at",  :null => false
+    t.integer  "position",    :default => 0, :null => false
+    t.datetime "created_at",                 :null => false
+    t.datetime "updated_at",                 :null => false
   end
 
   add_index "producer_properties", ["position"], :name => "index_producer_properties_on_position"
@@ -618,9 +618,9 @@ ActiveRecord::Schema.define(:version => 20150422014819) do
     t.string   "email"
     t.text     "special_instructions"
     t.integer  "distributor_id"
-    t.integer  "order_cycle_id"
     t.string   "currency"
     t.string   "last_ip_address"
+    t.integer  "order_cycle_id"
     t.integer  "cart_id"
   end