aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/lib
diff options
context:
space:
mode:
authorPrem Sichanugrist <s@sikachu.com>2012-02-03 11:47:47 -0500
committerPrem Sichanugrist <s@sikachu.com>2012-02-03 14:26:34 -0500
commit567ac65b423c30c24aa6c0c0522858e3c240eb26 (patch)
tree4b94f14e9987ba0174a24054f4ff67dd9151561d /actionpack/test/lib
parent4ca633e4663b62653ee017e5fd02dd86f06d1200 (diff)
downloadrails-567ac65b423c30c24aa6c0c0522858e3c240eb26.tar.gz
rails-567ac65b423c30c24aa6c0c0522858e3c240eb26.tar.bz2
rails-567ac65b423c30c24aa6c0c0522858e3c240eb26.zip
Fix override API response bug in respond_with
Default responder was only using the given respond block when user requested for HTML format, or JSON/XML format with valid resource. This fix the responder so that it will use the given block regardless of the validity of the resource. Note that in this case you'll have to check for object's validity by yourself in the controller. Fixes #4796
Diffstat (limited to 'actionpack/test/lib')
-rw-r--r--actionpack/test/lib/controller/fake_models.rb10
1 files changed, 10 insertions, 0 deletions
diff --git a/actionpack/test/lib/controller/fake_models.rb b/actionpack/test/lib/controller/fake_models.rb
index f2362714d7..bbb4cc5ef3 100644
--- a/actionpack/test/lib/controller/fake_models.rb
+++ b/actionpack/test/lib/controller/fake_models.rb
@@ -34,6 +34,16 @@ end
class GoodCustomer < Customer
end
+class ValidatedCustomer < Customer
+ def errors
+ if name =~ /Sikachu/i
+ []
+ else
+ [{:name => "is invalid"}]
+ end
+ end
+end
+
module Quiz
class Question < Struct.new(:name, :id)
extend ActiveModel::Naming