aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | Merge branch 'master' into nested_has_many_throughJon Leighton2010-11-27171-1857/+3057
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activerecord/CHANGELOG activerecord/lib/active_record/associations.rb
| * \ \ \ \ \ Merge branch 'master' into nested_has_many_throughJon Leighton2010-11-17196-1793/+3968
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/associations/has_many_through_association.rb activerecord/test/cases/associations/has_many_through_associations_test.rb
| * \ \ \ \ \ \ Merge branch 'master' into nested_has_many_throughJon Leighton2010-11-0852-185/+381
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/associations.rb
| * | | | | | | | Update new tests in cascaded_eager_loading_test.rb to work with the modified ↵Jon Leighton2010-10-311-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixtures in this branch
| * | | | | | | | Fix bug with 0bb85ed9ffa9808926b46e8f7e59cab5b85ac19f which missed out a ↵Jon Leighton2010-10-311-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixtures declaration in cascaded_eager_loading_test.rb
| * | | | | | | | Fix naughty trailing whitespaceJon Leighton2010-10-3121-259/+259
| | | | | | | | |
| * | | | | | | | Merge branch 'master' into nested_has_many_throughJon Leighton2010-10-3116-43/+121
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/associations/has_many_association.rb activerecord/lib/active_record/associations/through_association_scope.rb
| * \ \ \ \ \ \ \ \ Merge branch 'master' into nested_has_many_throughJon Leighton2010-10-28116-632/+6546
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activerecord/CHANGELOG activerecord/lib/active_record/association_preload.rb activerecord/lib/active_record/associations.rb activerecord/test/schema/schema.rb
| * | | | | | | | | | Rename nested_has_many_through_associations_test.rb because it does not only ↵Jon Leighton2010-10-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | concern has_many associations
| * | | | | | | | | | Add explicit ordering in relations_test.rb, as the lack of this was causing ↵Jon Leighton2010-10-191-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | failures against postgres
| * | | | | | | | | | Add explicit ordering to nested_has_many_through_associations_test.rb as ↵Jon Leighton2010-10-191-32/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this was causing failures under postgres
| * | | | | | | | | | Remove obsolete require to active_record/associations/nested_has_many_throughJon Leighton2010-10-191-1/+0
| | | | | | | | | | |
| * | | | | | | | | | Fix typoJon Leighton2010-10-191-1/+1
| | | | | | | | | | |
| * | | | | | | | | | Remove obsolete autoloadJon Leighton2010-10-191-1/+0
| | | | | | | | | | |
| * | | | | | | | | | Add a CHANGELOG entry about nested through associationsJon Leighton2010-10-191-0/+2
| | | | | | | | | | |
| * | | | | | | | | | Merge branch 'master' into nested_has_many_throughJon Leighton2010-10-1967-1408/+2386
| |\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Add some API documentation about nested through associationsJon Leighton2010-10-191-8/+52
| | | | | | | | | | | |
| * | | | | | | | | | | Add some comments for ThroughReflection#through_reflection_chainJon Leighton2010-10-191-1/+10
| | | | | | | | | | | |
| * | | | | | | | | | | Remove various comments and code which were just being used during the ↵Jon Leighton2010-10-195-199/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | development of nested through association support (OMFGZ, I might just have nearly finished this\!
| * | | | | | | | | | | Add explicit tests for the nested through association changes in reflection.rbJon Leighton2010-10-195-2/+80
| | | | | | | | | | | |
| * | | | | | | | | | | Fix broken testJon Leighton2010-10-191-1/+5
| | | | | | | | | | | |
| * | | | | | | | | | | Small refactoringJon Leighton2010-10-191-6/+2
| | | | | | | | | | | |
| * | | | | | | | | | | Support the :primary_key option on a through reflection in a nested through ↵Jon Leighton2010-10-196-7/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | association
| * | | | | | | | | | | Bugfix/refactoringJon Leighton2010-10-193-11/+20
| | | | | | | | | | | |
| * | | | | | | | | | | Remove unused methodsJon Leighton2010-10-191-11/+0
| | | | | | | | | | | |
| * | | | | | | | | | | Support for :primary_key option on the source reflection of a through ↵Jon Leighton2010-10-1910-18/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | association, where the source is a has_one or has_many
| * | | | | | | | | | | Respect the :primary_key option on the through_reflection of (non-nested) ↵Jon Leighton2010-10-199-11/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | through associations
| * | | | | | | | | | | Properly support conditions on any of the reflections involved in a nested ↵Jon Leighton2010-10-1916-98/+235
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | through association
| * | | | | | | | | | | Refactoring: replace the mix of variables like @finder_sql, @counter_sql, ↵Jon Leighton2010-10-1711-154/+138
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | etc with just a single scope hash (created on initialization of the proxy). This is now used consistently across all associations. Therefore, all you have to do to ensure finding/counting etc is done correctly is implement the scope correctly.
| * | | | | | | | | | | Make sure nested through associations are read onlyJon Leighton2010-10-156-9/+82
| | | | | | | | | | | |
| * | | | | | | | | | | Fix small bug which was shown by the last commitJon Leighton2010-10-151-1/+1
| | | | | | | | | | | |
| * | | | | | | | | | | Add assertions for nested through associations loaded by includes with ↵Jon Leighton2010-10-151-81/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | conditions (uses the single-query strategy). Currently one failure to fix.
| * | | | | | | | | | | Support preloading nested through associations (using the default ↵Jon Leighton2010-10-152-122/+216
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | multi-query strategy)
| * | | | | | | | | | | Remove unnecessary requires from nested_has_many_through_associations_test.rbJon Leighton2010-10-141-6/+0
| | | | | | | | | | | |
| * | | | | | | | | | | Add test_has_one_through_has_one_through_with_belongs_to_source_reflectionJon Leighton2010-10-147-4/+31
| | | | | | | | | | | |
| * | | | | | | | | | | Add test_has_many_through_belongs_to_with_has_many_through_source_reflection ↵Jon Leighton2010-10-142-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (which already passes)
| * | | | | | | | | | | Add test_has_many_through_has_many_through_with_belongs_to_source_reflection ↵Jon Leighton2010-10-142-1/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (which already works)
| * | | | | | | | | | | Added ↵Jon Leighton2010-10-145-4/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test_has_many_through_has_many_with_has_many_through_habtm_source_reflection and make it pass
| * | | | | | | | | | | Fix bug in previous refactoringJon Leighton2010-10-141-6/+6
| | | | | | | | | | | |
| * | | | | | | | | | | Merge branch 'master' into nested_has_many_throughJon Leighton2010-10-14160-1320/+8646
| |\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/associations.rb activerecord/test/cases/associations/cascaded_eager_loading_test.rb
| * | | | | | | | | | | | Add ↵Jon Leighton2010-10-133-4/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test_has_many_through_has_and_belongs_to_many_with_has_many_source_reflection and make it work
| * | | | | | | | | | | | Refactoring ThroughAssociationScope#construct_through_joinsJon Leighton2010-10-131-54/+62
| | | | | | | | | | | | |
| * | | | | | | | | | | | First bit of support for habtm in through assocs - ↵Jon Leighton2010-10-136-30/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test_has_many_through_has_many_with_has_and_belongs_to_many_source_reflection now passes
| * | | | | | | | | | | | Hook ThroughAssociationScope up to use the AliasTracker classJon Leighton2010-10-131-9/+30
| | | | | | | | | | | | |
| * | | | | | | | | | | | Some small tweaks on the last commitJon Leighton2010-10-122-19/+12
| | | | | | | | | | | | |
| * | | | | | | | | | | | Extract aliasing code from JoinDependency and JoinAssociation into a ↵Jon Leighton2010-10-122-41/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | separate AliasTracker class. This can then be used by ThroughAssociationScope as well.
| * | | | | | | | | | | | Add a commented, failing test for using a habtm in a has many through ↵Jon Leighton2010-10-125-55/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | association. I want to refactor how aliasing works first.
| * | | | | | | | | | | | Add test_has_many_through_has_one_through_with_has_many_source_reflectionJon Leighton2010-10-122-1/+19
| | | | | | | | | | | | |
| * | | | | | | | | | | | Adding test_has_many_through_has_one_with_has_many_through_source_reflection ↵Jon Leighton2010-10-125-3/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | and modifying ThroughAssociationScope to make it work correctly.
| * | | | | | | | | | | | Rename some tests for consistencyJon Leighton2010-10-121-3/+3
| | | | | | | | | | | | |