Rails Ruby on Rails

Comments and changes to this ticket

• You flagged this item as spam.
  

  Wojciech Wnętrzak February 11th, 2011 @ 10:01 AM

  Additional patch with test

  • test_for_post_request_when_argument_of_respond_with_is_array_of_resources.diff 1.6 KB

• You flagged this item as spam.
  

  Wojciech Wnętrzak February 11th, 2011 @ 10:20 AM

  I forgot to add require 'active_support/core_ext/array/wrap' for patch with fix.
  This patch should be used instead.

  • fix_for_respond_with_method_when_argument_is_array_of_resources.diff 1.3 KB
• 

  José Valim February 11th, 2011 @ 01:12 PM

  • State changed from “new” to “invalid”
  • Assigned user set to “José Valim”
  • Importance changed from “” to “Low”

  Rails allows you to use respond_with([@admin, @post]) but Rails interprets it as a collection. The same as if you do: respond_with(Article.all).

  That said, if you want to say you are redirecting to a nested resource, you have to do: respond_with(@admin, @post). Therefore, [[arg1, arg2]] is not wrong and the test that fails after your change is correct.

• You flagged this item as spam.
  

  Wojciech Wnętrzak February 11th, 2011 @ 01:37 PM

  One more question:
  Let's say we have responder in create action like:
  

    respond_with([:admin, @post])
  

  And object @post has errors - it wont be detected and instead of 422, we get 301 - is it intentional?

  Also note about similarity of method respond_with to polymorphic_url is little misleading - there we can pass only array if we need nesting.
  Documentation

• 

  José Valim February 11th, 2011 @ 01:51 PM

  Yes, it is intentional in the sense you should use respond_with(@admin, @post) instead. I agree with you about the note being misleading, I have just removed it. Thanks!