Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

tonycoco

tonycoco

Not a member

tonycoco's latest activity

Friday, February 11 2011
6393 action_dispatch/http/request.rb missing a require was updated in Ruby on Rails
  • tonycoco
    tonycoco commented at 10:52 PM

    +1 I've seen this too with the 3.0.4 release too. A quick fix is to add the mentioned "require 'active_support/core_ext/module/deprecation'" before you require ...
Wednesday, February 02 2011
6313 Capture not working when passing pre-escaped blocks (like, JSON)... was updated in Ruby on Rails
  • tonycoco
    tonycoco commented at 12:59 AM

    Upon further review of the newest Rails code. The capture method works as advertised. I'll continue to look at it, but I think this one can be closed out.
Wednesday, January 19 2011
6313 Capture not working when passing pre-escaped blocks (like, JSON)... was created in Ruby on Rails
  • tonycoco
    tonycoco created the ticket at 12:58 AM

    A quick look at the current code for reference...

    # File actionpack/lib/action_view/helpers/capture_helper.rb, line 38
def capture(*args)
  value = nil
  buffe...
Tuesday, June 01 2010
3945 helper :all on by default was updated in Ruby on Rails
  • tonycoco
    tonycoco commented at 9:55 PM

    +1, the solution in Rails 2 was much more elegant and made sense (if you want it, we got it, if you don't, comment it out). Yehuda's solution looks very Java-es...
Friday, April 23 2010
4452 Making ActiveSupport::Cache consistent was updated in Ruby on Rails
  • tonycoco
    tonycoco commented at 5:41 PM

    This looks like an excellent patch. I'll have to dig around a bit more in the patch, but it looks to be very concise and has refactored many parts where it was ...
Tuesday, March 24 2009
1626 fieldWithErrors shouldn't use a div was updated in Ruby on Rails
  • tonycoco
    tonycoco commented at 8:09 PM

    This is a great addition. I have looked at adding the class on the actual element in the past, but using a span around works for me just fine. +1 for sure. We h...