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.

Yaroslav Markin's latest activity

Saturday, December 27 2008
1280 error_messages_for doesn't html escape it's messages was updated in Ruby on Rails
  • Yaroslav Markin
    Yaroslav Markin commented at 4:13 PM

    If applying this, we should probably also patch error_message_on. That is active_record_helper.rb @L123@

1648 Introduce dynamic scopers (scoped_by_foo_and_bar) was updated in Ruby on Rails
  • Yaroslav Markin
    Yaroslav Markin commented at 3:23 PM

    Fixed typo in commit message

  • Yaroslav Markin
    Yaroslav Markin created the ticket at 3:22 PM

    Patch adds dynamic scopers support (basically the same thing as with dynamic finders, but with calling AR::Base.scoped())

    I don't think tests are good enough, s...

1647 Refactor AR::Base#new_record? was created in Ruby on Rails
  • Yaroslav Markin
    Yaroslav Markin created the ticket at 2:04 PM

    See also: #1219

    Patch simplifies new_record? method.

    Sorry that I missed this refactoring when sumbitting #1219

1109 object allocation during garbage collection phase was updated in Ruby on Rails
  • Yaroslav Markin
    Yaroslav Markin commented at 1:48 PM

    Can you reproduce this with latest rails (2.2.2)?

1219 new_record? returns nil on records retrieved from database was updated in Ruby on Rails
  • Yaroslav Markin
    • Assigned user set to DHH
    by Yaroslav Markin at 1:31 PM
  • Yaroslav Markin
    • Tag changed from activerecord, create, joins to activerecord, create, patch, tiny
    by Yaroslav Markin at 1:28 PM

    Patch attached, I believe that it should return false on existing records as well, unless we have a good reason not to

Friday, December 26 2008
1634 number_to_human_size helper: fix/introduce pluralization, add 'format' property was updated in Ruby on Rails
  • Yaroslav Markin
    Yaroslav Markin commented at 12:22 PM

    Needs more discussion.

    Since it fixes a bug anyway, in case we don't come up with a better solution, we should apply this to 2.3.

1389 Error with date_select when a default_locale is set was updated in Ruby on Rails
  • Yaroslav Markin
    Yaroslav Markin commented at 10:41 AM

    @bruno :'en-US' is not the default locale anymore, did you try :en?

Thursday, December 25 2008
1545 Error_messages_for i18n not working as expected was updated in Ruby on Rails