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.

Wincent Colaiuta's latest activity

Wednesday, November 03 2010
4858 ActionMailer is html escaping ampersand (&) in Urls in plain text messages was updated in Ruby on Rails
  • Wincent Colaiuta
    Wincent Colaiuta commented at 8:57 PM

    I think adding new syntax is a non-solution to this problem which just wall-papers over the underlying issue.

    The fact is that it doesn't make sense to HTML-esc...

Sunday, October 10 2010
5539 Fix misleading advice to add 'memcache' to Gemfile was updated in Ruby on Rails
  • Wincent Colaiuta
    Wincent Colaiuta commented at 12:21 PM

    it told you to require memcache (the library), not to add "memcache" (the string) to your Gemfile.

    So, you're saying that "Please add it to your Gemfile" mean...

  • Wincent Colaiuta
    Wincent Colaiuta commented at 10:41 AM

    No, it did tell you to put "memcache" in your Gemfile; read what it says:

    You don't have memcache installed in your application. Please add it to your Gemfile

    ...
Monday, September 27 2010
5098 Rails 3 Beta 4 ActiveRecord 5X slower than Rails 2.3.5 was updated in Ruby on Rails
  • Wincent Colaiuta
    Wincent Colaiuta commented at 6:09 PM

    Did you do any benchmarks for cases other than Post.find(1). Seems an incredibly narrow basis for declaring performance parity.

    Having said that, there's probab...

Sunday, September 12 2010
5618 [PATCH] Swap conditional order for better readability was created in Ruby on Rails
  • Wincent Colaiuta
    Wincent Colaiuta created the ticket at 12:37 AM

    "if X else Y" constructs are generally easier to read than "unless Y else X" constructs.

Saturday, September 04 2010
4858 ActionMailer is html escaping ampersand (&) in Urls in plain text messages was updated in Ruby on Rails
  • Wincent Colaiuta
    Wincent Colaiuta commented at 2:07 PM

    I've had to use raw in a few places in my non-HTML templates for this reason.

    It's not limited to templates with the .text.erb extension. I've also had to do it...

Friday, September 03 2010
5539 [PATCH] Fix misleading advice to add 'memcache' to Gemfile was created in Ruby on Rails
  • Wincent Colaiuta
    Wincent Colaiuta created the ticket at 12:27 AM

    Commit 57144388f removed the hard-coded dependency on the memcache-client gem, and added a warning advising people to install it if needed. The problem is, howe...

Tuesday, August 24 2010
5440 ActiveRecord::record_timestamps broken in RC2 (regression from RC1) was updated in Ruby on Rails
  • Wincent Colaiuta
    Wincent Colaiuta commented at 2:53 PM

    Thanks for the quick fix.

  • Wincent Colaiuta
    Wincent Colaiuta commented at 2:22 PM

    Here's a test case, but I can't actually run it yet because Bundler is giving me hell. Hopefully it's right. In the meantime I'll continue to fight with Bundler...

  • Wincent Colaiuta
    • Assigned user set to José Valim
    by Wincent Colaiuta at 1:40 PM

    Output from git bisect reveals that this commit is the culprit:

    30ea923040ded944209c98383389b6c9aafe806a is the first bad commit
    commit 30ea923040ded944209c98...