diff options
author | Jeremy Kemper <jeremy@bitsweat.net> | 2008-02-27 23:05:59 +0000 |
---|---|---|
committer | Jeremy Kemper <jeremy@bitsweat.net> | 2008-02-27 23:05:59 +0000 |
commit | 8352287c725c09ad64db39bd54a69efdae3cdce4 (patch) | |
tree | e12fb75cfe042552b2762192f4e5ff9a4b0e4b11 /activerecord/test | |
parent | 33736bc18a9733d95953f6eaec924db10badc908 (diff) | |
download | rails-8352287c725c09ad64db39bd54a69efdae3cdce4.tar.gz rails-8352287c725c09ad64db39bd54a69efdae3cdce4.tar.bz2 rails-8352287c725c09ad64db39bd54a69efdae3cdce4.zip |
Fix edge case with colons in times interpreted as bind variables by requiring that bind vars start with a letter. Closes #10519 [matrix9180]
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@8936 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/finder_test.rb | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/activerecord/test/cases/finder_test.rb b/activerecord/test/cases/finder_test.rb index d17712e1bf..49015133da 100644 --- a/activerecord/test/cases/finder_test.rb +++ b/activerecord/test/cases/finder_test.rb @@ -339,6 +339,8 @@ class FinderTest < ActiveRecord::TestCase def test_named_bind_variables assert_equal '1', bind(':a', :a => 1) # ' ruby-mode assert_equal '1 1', bind(':a :a', :a => 1) # ' ruby-mode + + assert_nothing_raised { bind("'+00:00'", :foo => "bar") } assert_kind_of Firm, Company.find(:first, :conditions => ["name = :name", { :name => "37signals" }]) assert_nil Company.find(:first, :conditions => ["name = :name", { :name => "37signals!" }]) |