r1607@asus: jeremy | 2005-07-02 23:36:52 -0700

 Rollback MysqlAdapter#select speedup; fix add_limit_offset


git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@1625 5ecf4fe2-1ee6-0310-87b1-e25e094e27de

1 files changed, 7 insertions, 7 deletions

diff --git a/activerecord/lib/active_record/connection_adapters/mysql_adapter.rb b/activerecord/lib/active_record/connection_adapters/mysql_adapter.rb
index 82ebf6bb4a..edfdc44f83 100755
--- a/activerecord/lib/active_record/connection_adapters/mysql_adapter.rb
+++ b/activerecord/lib/active_record/connection_adapters/mysql_adapter.rb
@@ -174,11 +174,11 @@ module ActiveRecord
       end
 
       def add_limit_offset!(sql, options)
-        unless options[:limit].blank?
-          unless options[:offset].blank?
-            sql << " LIMIT #{options[:offset]}, #{options[:limit]}"
-          else
+        if options[:limit]
+          if options[:offset].blank?
             sql << " LIMIT #{options[:limit]}"
+          else
+            sql << " LIMIT #{options[:offset]}, #{options[:limit]}"
           end
         end
       end
@@ -206,9 +206,9 @@ module ActiveRecord
           @connection.query_with_result = true
           result = execute(sql, name)
           rows = []
-          #all_fields_initialized = result.fetch_fields.inject({}) { |all_fields, f| all_fields[f.name] = nil; all_fields }
-          #result.each_hash { |row| rows << all_fields_initialized.dup.update(row) }
-          result.each_hash { |row| rows << row }
+          all_fields_initialized = result.fetch_fields.inject({}) { |all_fields, f| all_fields[f.name] = nil; all_fields }
+          result.each_hash { |row| rows << all_fields_initialized.merge(row) }
+          #result.each_hash { |row| rows << row }
           result.free
           rows
         end