Permalink
Browse files

Merge pull request #11263 from neerajdotname/columns_for_remove

Removed deprecated method `columns_for_remove`
  • Loading branch information...
2 parents 9adbbe9 + 0e8027f commit f775a8456a85a6ff838ed37de1aa7bcde9fc9c32 @senny senny committed Jul 3, 2013
@@ -1,3 +1,7 @@
+* Removed deprecated `columns_for_remove` from `SchemaStatements`.
+
+ *Neeraj Singh*
+
* Remove deprecated `SchemaStatements#distinct`.
*Francesco Rodriguez*
@@ -803,12 +803,6 @@ def index_name_for_remove(table_name, options = {})
index_name
end
- def columns_for_remove(table_name, *column_names)
- ActiveSupport::Deprecation.warn("columns_for_remove is deprecated and will be removed in the future")
- raise ArgumentError.new("You must specify at least one column name. Example: remove_columns(:people, :first_name)") if column_names.blank?
- column_names.map {|column_name| quote_column_name(column_name) }
- end
-
def rename_table_indexes(table_name, new_name)
indexes(new_name).each do |index|
generated_index_name = index_name(table_name, column: index.columns)

0 comments on commit f775a84

Please sign in to comment.