'refinery_settings' already exists about refinerycms-blog

[thebravoman]

I had to modify

    unless ActiveRecord::Base.connection.table_exists? 'refinery_settings'
      create_table :refinery_settings do |t|
        t.string    :name
        t.text      :value
        t.boolean   :destroyable,             :default => true
        t.string    :scoping
        t.boolean   :restricted,              :default => false
        t.string    :form_value_type

        t.timestamps
      end

      add_index :refinery_settings, :name
    end

# This migration comes from refinery_settings (originally 20130414130143)
class AddSlugToRefinerySettings < ActiveRecord::Migration[4.2]
  def change
  	unless ActiveRecord::Base.connection.column_exists?(:refinery_settings, :slug)
	    add_column :refinery_settings, :slug, :string, :unique => true
	  end
  end
end

# This migration comes from refinery_settings (originally 20130422105953)
class AddTitleToRefinerySettings < ActiveRecord::Migration[4.2]
  def change
  	unless ActiveRecord::Base.connection.column_exists?(:refinery_settings, :title)
    	add_column :refinery_settings, :title, :string
   	end
  end
end

Should I get this into a PR? Comments if I am missing something here?

[bricesanchez]

I don't think it's the things to do but i've seen this in refinerycms-blog:

refinerycms-blog/lib/generators/refinery/blog/blog_generator.rb

Line 12 in fcce6f6

rake("refinery_settings:install:migrations")

Perhaps we should add a constraint for this line.

Did you install refinerycms-settings before the blog?