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.

Gabe da Silveira

Gabe da Silveira

Lead Developer · Not a member

Gabe da Silveira's latest activity

Monday, August 10 2009
2510 url_for ignores routing priority was updated in Ruby on Rails
  • Gabe da Silveira
    Gabe da Silveira commented at 4:28 PM

    So you deprecated it, but A) my patch would have still cleanly applied and B) the original issue still exists. At least it should be wontfix.
  • Gabe da Silveira
    Gabe da Silveira commented at 4:46 AM

    I don't understand why you wouldn't just apply this in the meantime since it fixes an legitimate issue. It's not really invalid.
1196 counter_cache not decrementing on delete was updated in Ruby on Rails
  • Gabe da Silveira
    Gabe da Silveira commented at 12:20 AM

    Okay, after integrating bitsweat's test changes it was revealed that the actual problem footprint is much smaller. (The original patch conflated a buggy test wi...
Sunday, August 09 2009
1196 counter_cache not decrementing on delete was updated in Ruby on Rails
  • Gabe da Silveira
    Gabe da Silveira commented at 9:21 PM

    Rebased the test cases against 2-3-stable and verified that they do indeed fail.

    I've attached a test.
2719 has_many :through should work if the join model uses has_one relationship was updated in Ruby on Rails
  • Gabe da Silveira
    Gabe da Silveira commented at 9:00 PM

    Crap. The deprecation snuck into master diff as well, which I don't think it should be according to my impression from koz. Without the deprecation:
  • Gabe da Silveira
    Gabe da Silveira commented at 8:57 PM

    Here are updated patches for 2-3-stable and master. Handy-dandy one-stop location:
2326 script.aculo.us Sortable.create drops draggable despite option.only was updated in Ruby on Rails
2481 Console/log output for malformed XML requests is not very helpful was updated in Ruby on Rails
  • Gabe da Silveira
    Gabe da Silveira commented at 12:21 PM

    +1 on fixing this issue. I've been annoyed by this for quite some time. Using puts is not the right approach, but we ought to be able to do something about it.....
2790 ActiveRecord :include doesn't work with foreign keys was updated in Ruby on Rails
  • Gabe da Silveira
    Gabe da Silveira commented at 12:09 PM

    not reproducible for me either. this would be a major bug that people would notice quickly. Must be some local interaction you're having Johnny.
2510 url_for ignores routing priority was updated in Ruby on Rails
  • Gabe da Silveira
    Gabe da Silveira commented at 12:01 PM

    verified the bug on 2-3-stable.

    The problem is that the prioritization algorithm for routes, which depends on the number of matching components, does not preser...