Permalink
Browse files

Added Base.exists?(id) that'll return true if an object of the class …

…with the given id exists #854 [stian@grytoyr.net]

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@913 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
  • Loading branch information...
1 parent 9776bfc commit 2f944123f32ad37efc84f13de25350beb7f8fed6 @dhh dhh committed Mar 17, 2005
Showing with 16 additions and 0 deletions.
  1. +2 −0 activerecord/CHANGELOG
  2. +7 −0 activerecord/lib/active_record/base.rb
  3. +7 −0 activerecord/test/finder_test.rb
View
2 activerecord/CHANGELOG
@@ -1,5 +1,7 @@
*SVN*
+* Added Base.exists?(id) that'll return true if an object of the class with the given id exists #854 [stian@grytoyr.net]
+
* Added optionally allow for nil or empty strings with validates_numericality_of #801 [Sebastian Kanthak]
* Fixed problem with using slashes in validates_format_of regular expressions #801 [Sebastian Kanthak]
View
7 activerecord/lib/active_record/base.rb
@@ -326,6 +326,13 @@ def find(*args)
end
end
+ # Returns true if the given +id+ represents the primary key of a record in the database, false otherwise.
+ # Example:
+ # Person.exists?(5)
+ def exists?(id)
+ !find_first("#{primary_key} = #{sanitize(id)}").nil?
+ end
+
# This method is deprecated in favor of find with the :conditions option.
# Works like find, but the record matching +id+ must also meet the +conditions+.
# +RecordNotFound+ is raised if no record can be found matching the +id+ or meeting the condition.
View
7 activerecord/test/finder_test.rb
@@ -10,6 +10,13 @@ def test_find
assert_equal(@topics["first"]["title"], Topic.find(1).title)
end
+ def test_exists
+ assert (Topic.exists?(1))
+ assert !(Topic.exists?(45))
+ assert !(Topic.exists?("foo"))
+ assert !(Topic.exists?([1,2]))
+ end
+
def test_find_by_array_of_one_id
assert_kind_of(Array, Topic.find([ 1 ]))
assert_equal(1, Topic.find([ 1 ]).length)

0 comments on commit 2f94412

Please sign in to comment.