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.

Tekin

Tekin

Developer · Not a member

Tekin's latest activity

Tuesday, November 25 2008
1394 compute_asset_host bombs when asset_host = Proc and called from a Mailer was updated in Ruby on Rails
  • Tekin
    Tekin commented at 11:38 AM

    Here are some tests, including one that fails.

    Although my patch fixes the problem, long term, I'm not convinced it's the right solution. The second argument wa...
Thursday, November 20 2008
1394 compute_asset_host bombs when asset_host = Proc and called from a Mailer was updated in Ruby on Rails
  • Tekin
    Tekin commented at 11:27 PM

    Just looking into this a bit more, although it's still not right that compute_asset_host falls over in mailer templates when asset_host is set to a 2 arguement ...
Tuesday, November 18 2008
1394 compute_asset_host bombs when asset_host = Proc and called from a Mailer was updated in Ruby on Rails
  • Tekin
    Tekin commented at 6:35 PM

    Yeah, sorry, had been a long day. Will get a proper patch in asap.
Monday, November 17 2008
1394 compute_asset_host bombs when asset_host = Proc and called from a Mailer was created in Ruby on Rails
  • Tekin
    Tekin created the ticket at 9:59 PM

    compute_asset_host is falling over when used in Mailer views under certain conditions.

    If you set asset_host to a 2 arguement Proc, compute_asset_host calls the...
Monday, November 10 2008
837 Specify the disabled attribute for option tags was updated in Ruby on Rails
  • Tekin
    Tekin commented at 10:50 PM

    Thanks Michael.

    Yeah, I'm not entirely happy with the API either but not sure what the alternative would be without drastic changes. In the mean time, I'll crea...
Saturday, November 08 2008
837 Specify the disabled attribute for option tags was updated in Ruby on Rails
  • Tekin
    Tekin commented at 5:06 PM

    Up to date patch with support added to select and collection_select helpers.
Thursday, November 06 2008
1102 Added :counter_sql to valid keys for HABTM was updated in Ruby on Rails
  • Tekin
    Tekin commented at 9:09 PM

    No worries!

    Here's an updated patch with a couple of tests. Hope that does the trick.
  • Tekin
    Tekin commented at 8:25 PM

    OK, will try get some simply tests added to the patch.
  • Tekin
    Tekin commented at 8:03 PM

    Here you go!
Wednesday, September 24 2008
831 count on HABTMs broken by Ruby 1.8.7 Array.count was updated in Ruby on Rails
  • Tekin
    • Tag changed from 2.1, activerecord, bug, edge, patch, tests to 2.1, activerecord, bug, edge, patch, tests
    by Tekin at 1:50 PM

    :counter_sql has not been added to the valid keys hash in create_has_and_belongs_to_many_reflections on line 1574 of associations.rb so :counter_sql is being re...