Ticket #8976: exists_should_use_count.patch

test/base_test.rb

@@ -1704 +1704 @@
-    assert_equal %("#{t.written_on.to_s(:db)}"), t.attribute_for_inspect(:written_on)
-    assert_equal '"This is some really long content, longer than 50 ch..."', t.attribute_for_inspect(:content)
-  end
+  
+  def test_exists_should_work_with_id
+    t = topics(:first)
+    assert Topic.exists?(t.id)
+  end
+  
+  def test_exists_should_work_with_conditions
+    t = topics(:first)
+    assert Topic.exists?(:title => t.title)
+  end
+  
+  def test_exists_should_return_false
+    assert !Topic.exists?(800)
+  end
-
-  private
-    def assert_readers(model, exceptions)

lib/active_record/base.rb

@@ -447 +447 @@
-      #   Person.exists?(:name => "David")
-      #   Person.exists?(['name LIKE ?', "%#{query}%"])
-      def exists?(id_or_conditions)
-!find(:first, :conditions => expand_id_conditions(id_or_conditions)).nil?
+count(:conditions => expand_id_conditions(id_or_conditions)) > 0
-      rescue ActiveRecord::ActiveRecordError
-        false
-      end