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.

Michael Koziarski's latest activity

Sunday, July 12 2009
2373 record sensitive procs for to_xml was updated in Ruby on Rails
  • Michael Koziarski
    Michael Koziarski commented at 4:24 AM

    Remove the ternary operator, it just obfuscates things:

    if proc.arity == 1
    proc.call ... else
    proc.call ... end

    Also update the docs so that it explains why you...

Friday, July 10 2009
2852 accepts_nested_attributes_for creating new associated record when one already... was updated in Ruby on Rails
Monday, July 06 2009
2765 after_create callback not working as it should was updated in Ruby on Rails
  • Michael Koziarski
    Michael Koziarski commented at 10:06 AM

    Sounds good, can you attach that as a diff (assuming it's failing
    :)). If it doesn't fail then we have a whole other problem :)

Sunday, July 05 2009
2764 Supporting partial updates for serialized columns was updated in Ruby on Rails
  • Michael Koziarski
    Michael Koziarski commented at 4:58 AM

    Yeah, I'm kinda opposed to making those kind of changes, but would be
    happy to look at a patch which achieved it.

    Fundamentally if you're directly working with ...

2858 Mocha release 0.9.6 heads-up was updated in Ruby on Rails
  • Michael Koziarski
    Michael Koziarski commented at 4:56 AM

    Jeremy's the guy who did all the work the last time we had to deal
    with test/unit, so he's probably the best one to comment on any
    unified approach.

    But yeah, f...

1560 link_to in ActionView::TestCase uses ActionController::UrlRewriter's url_for ... was updated in Ruby on Rails
  • Michael Koziarski
    Michael Koziarski commented at 3:58 AM

    At the same time it could be worthwhile attempting to rationalise the 3 different implementations of url_for to make sure that they code they share, isn't repea...

  • Michael Koziarski
    • State changed from invalid to open
    by Michael Koziarski at 3:57 AM

    change UrlWriter#url_for to handle Strings as options?

    This seems like the right place to handle this for me.

Saturday, July 04 2009
2765 after_create callback not working as it should was updated in Ruby on Rails
  • Michael Koziarski
    Michael Koziarski commented at 2:20 AM

    Can you provide us with a failing test in the activerecord test suite rather than a stand alone app? That'd make it much easier to reproduce and verify

Friday, July 03 2009
2251 AssociationCollection#destroy should only delete join table records was updated in Ruby on Rails
  • Michael Koziarski
    Michael Koziarski commented at 1:06 AM

    OK, I'm in agreement with will here. Can someone upload a patch to do
    this, and then I think we're good to go.

    (or does it do this already?)

Thursday, July 02 2009
2858 Mocha release 0.9.6 heads-up was updated in Ruby on Rails
  • Michael Koziarski
    Michael Koziarski commented at 10:40 AM

    My brain's a bit fried by this head-cold so I'm not sure on yehuda's
    changes either. Will check them out properly in the morning, but odds
    are this lighthouse e...