Merge branch 'fix_migration_builds' into 'master'

Prevent job_id removing

I would prefer to keep field job_id in the builds table. It will allow us much room for manoeuvre.
Of course it is not the best practice to change migration because it can lead to different database states for users but we will remove this filed soon anyway. We will need to put condition there if this field exist.

See merge request !142

1 files changed, 1 insertions, 4 deletions

diff --git a/db/migrate/20150528011012_remove_unused_fields.rb b/db/migrate/20150528011012_move_job_name_to_build.rb
index 2895169..512ad47 100644
--- a/db/migrate/20150528011012_remove_unused_fields.rb
+++ b/db/migrate/20150528011012_move_job_name_to_build.rb
@@ -1,13 +1,10 @@
-class RemoveUnusedFields < ActiveRecord::Migration
+class MoveJobNameToBuild < ActiveRecord::Migration
   def up
     select_all("SELECT id, name FROM jobs").each do |job|
       execute("UPDATE builds SET name = '#{quote_string(job["name"])}' WHERE job_id = #{job["id"]}")
     end
-
-    remove_column :builds, :job_id, :integer
   end
 
   def down
-    add_column :builds, :job_id, :integer
   end
 end