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.

Lawrence Pit

Lawrence Pit

Not a member

Lawrence Pit's latest activity

Tuesday, May 11 2010
3744 ActiveSupport::JSON.encode bogus CircularReferenceError was updated in Ruby on Rails
  • Lawrence Pit
    Lawrence Pit commented at 11:56 AM

    This can be closed, the line currently reads (as suggested above as patch) :

       if @seen.any? { |object| object.equal?(value) }

    Edited by Rohit Arondekar for ...
Friday, May 07 2010
4437 [Patch] XML Mini Parser NoMethodError :getc when application running through ... was updated in Ruby on Rails
  • Lawrence Pit
    Lawrence Pit commented at 1:55 AM
  • Lawrence Pit
    Lawrence Pit commented at 1:49 AM

    Jeremy, attached patch with test.
  • Lawrence Pit
    Lawrence Pit commented at 1:48 AM

    Mat and Gael, it /is/ possible that you get this issue in production as well, because any rack middleware that adds itself and wraps the input which strictly co...
Thursday, May 06 2010
4437 [Patch] XML Mini Parser NoMethodError :getc when application running through ... was updated in Ruby on Rails
  • Lawrence Pit
    • Tag changed from active_resource, passenger, rack, xml_mini to actiondispatch, active_resource, passenger, rack, rails3, xml_mini
    • Assigned user set to Jeremy Kemper
    by Lawrence Pit at 12:46 PM

    This shouldn't be solved by patching the xmlmini classes.

    The issue only occurs in development mode. This is because Rack adds the middleware Rack::Lint when in...
Monday, May 03 2010
4525 Favor %{} in all code instead of (deprecated) {{}} as interpolation syntax fo... was created in Ruby on Rails
  • Lawrence Pit
    Lawrence Pit created the ticket at 7:47 AM

    Attached patch removes use of the {{}} syntax, which is deprecated by i18n. All rails code and doc examples should promote using %{}.
4521 HABTM: ability to link and unlink records with :autosave => true was updated in Ruby on Rails
  • Lawrence Pit
    Lawrence Pit commented at 4:12 AM
  • Lawrence Pit
    Lawrence Pit commented at 3:03 AM

    And another updated version. Now without an extra association class (and without an extra module). This version also supports linking / unlinking of records to ...
  • Lawrence Pit
    Lawrence Pit commented at 12:53 AM

    Updated version with extra tests to show association target isn't loaded when not necessary.
Sunday, May 02 2010
4521 HABTM: ability to link and unlink records with :autosave => true was updated in Ruby on Rails
  • Lawrence Pit
    • Assigned user cleared.
    by Lawrence Pit at 11:27 PM

    1) The current behavior, with or without :autosave => true, is that when you add or delete a record to a collection association, the link is immediately saved t...