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.

Arthur Zapparoli

Arthur Zapparoli

Backend Developer · Not a member

Arthur Zapparoli's latest activity

Monday, August 10 2009
3017 Fixed typo in test name and CHANGELOG was updated in Ruby on Rails
Sunday, August 09 2009
2949 Rails 2.3.3 ActiveRecord generates invalid SQL conditions for named_scope wit... was updated in Ruby on Rails
  • Arthur Zapparoli
    • Tag changed from 2.3.3, activerecord, bugmash, conditions, joins, named_scope, sql to 2.3.3, activerecord, bugmash, conditions, joins, named_scope, sql, verified
    by Arthur Zapparoli at 8:41 PM

    +1 for Tristan's patch. Applies cleanly to 2-3-stable and master.

    Nice catch! :)
  • Arthur Zapparoli
    Arthur Zapparoli commented at 5:50 PM

    Here's the SQL generated for Comment.enabled.post_desktop

    SELECT `comments`.* FROM `comments` INNER JOIN `posts` ON `posts`.id = `comments`.post_id WHERE ((`co...
  • Arthur Zapparoli
    Arthur Zapparoli commented at 5:40 PM

    Ok, this is really weird. I think it is related directly to the model name (or only the first letter). I was trying to create a failing test patch, but, after p...
  • Arthur Zapparoli
    Arthur Zapparoli commented at 5:33 AM

    Verified this generates invalid SQL on 2-3-stable

    Besides working with conditions Array, it also works with:

      class Widget < ActiveRecord::Base
    belongs_to...
3026 [Patch] Remove duplicated tests from belongs_to_association_test was created in Ruby on Rails
  • Arthur Zapparoli
    Arthur Zapparoli created the ticket at 8:33 PM

    I've attached patch which removes two duplicated tests from belongs_to_association_test.rb.

    Patch applies cleanly for 2-3-stable and master.
2990 None standard ID column on has_many :through was updated in Ruby on Rails
  • Arthur Zapparoli
    Arthur Zapparoli commented at 8:05 PM

    +1 for the idea, -1 for the patch

    Dira: I agree with José Valim, and think this patch needs to be applied using the existing AR models on the test suite.

    Also, ...
3025 Model#human_attribute_name does not accept symbols was updated in Ruby on Rails
  • Arthur Zapparoli
    Arthur Zapparoli commented at 6:46 PM

    I've attached a patch containing the fix for this ticket
358 find with a has_many condition fails was updated in Ruby on Rails
  • Arthur Zapparoli
    Arthur Zapparoli commented at 7:58 AM

    +1 for this patch

    Vladimir's patch solve the issue and applies cleanly to 2-3-stable
1719 ActiveSupport::Testing::SetupAndTeardown::run_with_callbacks_and_mocha does ... was updated in Ruby on Rails
  • Arthur Zapparoli
    Arthur Zapparoli commented at 5:45 AM

    not reproducible in 2-3-stable

    created the bug_test.rb file above, and it reproduced a test Error, instead of killing the test run.