Changeset 8208

Timestamp:

11/25/07 22:34:46 (2 years ago)

Author:

david

Message:

Allow validates_acceptance_of to use a real attribute instead of only virtual (so you can record that the acceptance occured) (closes #7457) [ambethia]

Files:

• trunk/activerecord/CHANGELOG (modified) (1 diff)
• trunk/activerecord/lib/active_record/validations.rb (modified) (2 diffs)
• trunk/activerecord/test/validations_test.rb (modified) (2 diffs)

trunk/activerecord/CHANGELOG

@@ -1 +1 @@
-*SVN*
+
+* Allow validates_acceptance_of to use a real attribute instead of only virtual (so you can record that the acceptance occured) #7457 [ambethia]
-
-* DateTimes use Ruby's default calendar reform setting. #10201 [Geoff Buesing]

trunk/activerecord/lib/active_record/validations.rb

@@ -415 +415 @@
-      #   end
-      #
-The terms_of_service attribute is entirely virtual. No database column is needed. This check is performed only if
-
+If the database column does not exist, the terms_of_service attribute is entirely virtual. This check is
+performed only if 
-      #
-      # Configuration options:
@@ -434 +434 @@
-        configuration.update(attr_names.extract_options!)
-
-
+.reject { |name| column_names.include? name.to_s }
-
-        validates_each(attr_names,configuration) do |record, attr_name, value|

trunk/activerecord/test/validations_test.rb

@@ -24 +24 @@
-class UniqueReply < Reply
-  validates_uniqueness_of :content, :scope => 'parent_id'
+end
+
+class PlagiarizedReply < Reply
+  validates_acceptance_of :author_name
-end
-
@@ -291 +295 @@
-    t.terms_of_service = "I agree."
-    assert t.save
+  end
+
+  def test_validates_acceptance_of_as_database_column
+    reply = PlagiarizedReply.create("author_name" => "Dan Brown")
+    assert_equal "Dan Brown", reply["author_name"]
-  end
-