aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord
Commit message (Collapse)AuthorAgeFilesLines
* 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
|
* Add test_has_many_through_has_one_throughJon Leighton2010-10-122-2/+17
|
* Support has_one through assocs as the source associationJon Leighton2010-10-127-20/+71
|
* A load of tests that need to be writtenJon Leighton2010-10-121-0/+63
|
* Refactoring to remove duplication introduced by the last commitJon Leighton2010-10-101-160/+145
|
* Add support for nested through associations in JoinAssociation. Hence ↵Jon Leighton2010-10-0911-94/+175
| | | | Foo.joins(:bar) will work for through associations. There is some duplicated code now, which will be refactored.
* Merge branch 'master' into nested_has_many_throughJon Leighton2010-10-0615-67/+147
|\
| * create fewer objects, call fewer methods in extract_pg_identifier_from_nameAaron Patterson2010-10-041-3/+3
| |
| * * + flatten is not required in >= Ruby 1.8.7Aaron Patterson2010-10-041-1/+1
| |
| * converting inject([]) to mapAaron Patterson2010-10-031-4/+3
| |
| * speed up duplicate migration detectionAaron Patterson2010-10-031-6/+5
| |
| * reduce the number of calls to camelizeAaron Patterson2010-10-031-3/+4
| |
| * convertion MigrationProxy to a Struct, initialize instance variablesAaron Patterson2010-10-031-7/+6
| |
| * dry up some migration logicAaron Patterson2010-10-031-2/+4
| |
| * no need to differentiate between nil and false in this caseAaron Patterson2010-10-031-2/+2
| |
| * lengths will be nil or an arrayAaron Patterson2010-10-031-2/+2
| |
| * remove a few function callsAaron Patterson2010-10-031-3/+5
| |
| * be kind to the garbage collector and reuse our visitor objectAaron Patterson2010-10-031-1/+4
| |
| * avoid creating objects when we canAaron Patterson2010-10-033-14/+7
| |
| * Rely on Rack::Session stores API for more compatibility across the Ruby world.José Valim2010-10-031-2/+5
| |
| * Refactor AssociationCollection#include? with objects in memoryCarlos Antonio da Silva2010-10-021-4/+3
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
| * Arel::Sql::Engine.new does not do anything anymoreAaron Patterson2010-09-302-2/+2
| |
| * type_name should check for blank because people may have messed up databasesAaron Patterson2010-09-302-1/+15
| |
| * build_where should be privateAaron Patterson2010-09-301-2/+2
| |
| * type_name is never a blank string, so use faster .nil? callAaron Patterson2010-09-301-1/+1
| |
| * AssociationCollection#include? working properly for objects added with build ↵Marcelo Giorgi2010-09-304-0/+39
| | | | | | | | method [#3472 state:resolved]
| * speeding up object instantiation by eliminating instance_evalAaron Patterson2010-09-301-7/+21
| |
| * [#5441 state:resolved] refactoring code to determine aggregate columnAaron Patterson2010-09-301-11/+14
| |
| * group clause must be more specificAaron Patterson2010-09-301-1/+1
| |
| * Tests proving #5441oleg dashevskii2010-09-301-1/+11
| |
* | Refactoring JoinDependency and friends. This improves the code (IMO) ↵Jon Leighton2010-10-0610-209/+356
| | | | | | | | including adding some explanatory comments, but more importantly structures it in such a way as to allow a JoinAssociation to produce an arbitrary number of actual joins, which will be necessary for nested has many through support. Also added 3 tests covering functionality which existed but was not previously covered.
* | A failing test for a nested has many through association loaded via ↵Jon Leighton2010-10-051-0/+6
| | | | | | | | Foo.joins(:bar)
* | Fix the tests (I have actually verified that these are also the 'right' ↵Jon Leighton2010-10-036-24/+24
| | | | | | | | fixes, rather than just making the tests pass again)
* | Add support for table aliasing, with a test that needs aliasing in order to ↵Jon Leighton2010-10-027-15/+75
| | | | | | | | work correctly. This test incidentally provides a more complicated test case (4 inner joins, 2 using polymorphism).
* | Integrate nested support into ThroughAssociationScope, using my concept of ↵Jon Leighton2010-10-025-59/+107
| | | | | | | | generating a 'chain' of reflections to be joined. It seems to work at the moment, all existing tests are passing. There may be further complications as we add more test cases for nested associations, though.
* | Started implementing nested :through associations by using the existing ↵Jon Leighton2010-10-014-32/+60
| | | | | | | | structure of ThroughAssociationScope rather than layering a module over the top
* | Prevent ↵Jon Leighton2010-09-301-2/+2
| | | | | | | | test_has_many_through_a_has_many_through_association_on_through_reflection failing for me due to ordering of the results
* | Merge branch 'master' into nested_has_many_through_2Jon Leighton2010-09-3033-224/+350
|\|
| * Fix 'rake db:create' is ignore encoding when using postgres [#5717 ↵yalab2010-09-301-1/+1
| | | | | | | | | | | | state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
| * Added config.app_generators to allow configuring application's generators ↵Piotr Sarnacki2010-09-301-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | from railties. With config.generators becomes a way to configure generators for current instance only. For example: module Blog class Engine < Rails::Engine config.generators do |g| g.orm :active_record end config.app_generators do |g| g.test_framework :rspec end end end such definition sets :active_record as orm for engine and :rspec as test_framework for application. The values set with app_generators can be overwritten in application using config.generators as you would normally do: module MyApp class Application < Rails::Application config.generators do |g| g.test_framework :test_unit end end end
| * double negative is not goodNeeraj Singh2010-09-301-1/+1
| |
| * no need of nil checkNeeraj Singh2010-09-291-1/+1
| |
| * Use detect instead select to avoid "sh: [mysqldump]: command not found".Emilio Tagua2010-09-291-1/+1
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>