Ensure indices don't flip order in schema.rb [#1266 state:committed]

Signed-off-by: NDavid Heinemeier Hansson <david@loudthinking.com>

activerecord/CHANGELOG

 *2.2.1 [RC2 or 2.2 final]*
 
+* Ensure indices don't flip order in schema.rb #1266 [Jordi Bunster]
+
 * Fixed that serialized strings should never be type-casted (i.e. turning "Yes" to a boolean) #857 [Andreas Korth]
 
 

activerecord/lib/active_record/schema_dumper.rb

@@ -159,13 +159,19 @@ def default_string(value)
       end
       
       def indexes(table, stream)
-        indexes = @connection.indexes(table)
-        indexes.each do |index|
-          stream.print "  add_index #{index.table.inspect}, #{index.columns.inspect}, :name => #{index.name.inspect}"
-          stream.print ", :unique => true" if index.unique
+        if (indexes = @connection.indexes(table)).any?
+          add_index_statements = indexes.map do |index|
+            statment_parts = [ ('add_index ' + index.table.inspect) ]
+            statment_parts << index.columns.inspect
+            statment_parts << (':name => ' + index.name.inspect)
+            statment_parts << ':unique => true' if index.unique
+
+            '  ' + statment_parts.join(', ')
+          end
+
+          stream.puts add_index_statements.sort.join("\n")
           stream.puts
         end
-        stream.puts unless indexes.empty?
       end
   end
-end
+end
\ No newline at end of file