Andrew Kaspick

So everything committed to prototype-ujs will be merged for the next rails point release? I just thought it would have been merged for the final rails 3 release...

To avoid any confusion, I didn't mean to say "original" patch there, just meant to say "patch" by itself... the one that was pushed to prototype-ujs.

I noticed that the rails.js file that comes with rails 3 is still out of date and doesn't include this fix...

http://github.com/rails/rails/blob/master/railtie...

Adding a patch for the 2.3 branch as well.

This patch needs to be backported to rails 2.3.x as the issue exists there as well. Do I need to provide an additional patch for that branch as well?

class Test
  extend ActiveSupport::Memoizable

protected
  def test
  end
  memoize :test
end

Before this patch, the test method would be accessible as a publ...

Fair enough. I just wanted to make sure that's what the plan for the submit button was. Besides that, Larry's patch is what mine would be then too.

I'm away traveling so am a bit staged with getting online. I noticed Larry attached a new patch. I have a new patch as well, but don't do the "|| false" part as...

Storing on the form makes much more sense. I'll get another patch together soon. Thanks.

Thanks Larry. It appeared to be working for me locally, but looking at the code for serializeElements in prototype, it does seem to do a comparison with the 'na...