From 8957f5db5d543b465af2ec55339628476ed17dc2 Mon Sep 17 00:00:00 2001 From: Pratik Naik Date: Sun, 27 Dec 2009 18:32:10 +0530 Subject: [PATCH] Make Model.all just a wrapper for find(:all) --- activerecord/lib/active_record/base.rb | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/activerecord/lib/active_record/base.rb b/activerecord/lib/active_record/base.rb index 0766dfe752..ed031088a7 100755 --- a/activerecord/lib/active_record/base.rb +++ b/activerecord/lib/active_record/base.rb @@ -669,16 +669,10 @@ module ActiveRecord #:nodoc: find(:last, *args) end - # Returns an ActiveRecord::Relation object. You can pass in all the same arguments to this method as you can - # to find(:all). + # A convenience wrapper for find(:all, *args). You can pass in all the + # same arguments to this method as you can to find(:all). def all(*args) - options = args.extract_options! - - if options.empty? && !scoped?(:find) - arel_table.to_a - else - construct_finder_arel_with_includes(options).to_a - end + find(:all, *args) end # Executes a custom SQL query against your database and returns all the results. The results will