aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test/cases/associations
Commit message (Collapse)AuthorAgeFilesLines
* Don't use mass-assignment protection when setting foreign keys or ↵Jon Leighton2011-05-121-0/+33
| | | | association conditions on singular associations. Fixes #481 (again).
* Merge pull request #512 from pivotalneutron/fix_eager_load_with_calculationsJon Leighton2011-05-111-1/+3
|\ | | | | Bug fixes for calculations with includes
| * Bug fixes:Fadzril Muhamad & Joseph Palermo2011-05-121-1/+3
| | | | | | | | | | | | - If doing a count on a relation that has an :include and a :join, it does a distinct even though it should not. - When doing a count on a relation that has an :include, it always falls back to a old style left join when performing the count. Looks like it was broken here: https://github.com/rails/rails/commit/b9599502c9e738a5a1513e75d08f8d40ed408265
* | Add missing requiresSantiago Pastorino2011-05-111-0/+1
| |
* | Don't remove the target if it has already been destroyedAndrew White2011-05-111-0/+9
|/
* Don't use select() values from the join model of a through association. ↵Jon Leighton2011-05-111-0/+4
| | | | Fixes #508.
* Bring back obj.association_loaded? as a deprecated method. Fixes #472.Jon Leighton2011-05-111-0/+9
|
* Add test to specify that attributes from an association's conditions should ↵Jon Leighton2011-05-101-4/+23
| | | | be assigned without mass-assignment protection when a record is built on the association.
* Don't use mass-assignment protection when applying the ↵Jon Leighton2011-05-101-0/+9
| | | | scoped.scope_for_create. Fixes #481.
* Add failing tests according to #479.José Valim2011-05-101-0/+15
|
* Add failing tests, according to #480.José Valim2011-05-101-0/+14
|
* Merge pull request #399 from ernie/join_conditions_on_joinJon Leighton2011-05-051-0/+11
|\ | | | | Add association join conditions to JOIN clause, not WHERE
| * Add a test for outer joins with conditions where value IS NULLErnie Miller2011-05-051-0/+5
| |
| * Add join conditions to JOIN clause, not WHEREErnie Miller2011-05-051-0/+6
| |
* | Fix for lighthouse #6741Nick Howard2011-05-011-0/+24
|/ | | | | | | - adds tests for find_or_create_by and find_or_initialize_by on has_many associations - changes the behavior of ActiveRecord::Associations::CollectionProxy#method_missing to differ to ActiveRecord::FinderMethods#find_or_instantiator_by_attributes for arg processing and saving so find_or_create_by's api on associations will be consistent w/ the api for model classes.
* assert_difference can take a callable piece of code rather than just evaling ↵Aaron Patterson2011-05-011-4/+4
| | | | a string
* fixing more test warnings in 1.9.3Aaron Patterson2011-04-303-6/+6
|
* Added test case and fix for proper eager loading associationsPavel Forkert2011-04-221-0/+10
|
* Fix test_associate_existing in has_many_through_associations_test on mysql ↵Jon Leighton2011-04-171-1/+1
| | | | and postgresql
* Refactor test to avoid hackeryJon Leighton2011-04-151-4/+4
|
* Deprecated support for passing hashes and relations to default_scope, in ↵Jon Leighton2011-04-122-14/+14
| | | | favour of defining a 'default_scope' class method in the model. See the CHANGELOG for more details.
* Using Object#in? and Object#either? in various placesPrem Sichanugrist2011-04-111-2/+3
| | | | There're a lot of places in Rails source code which make a lot of sense to switching to Object#in? or Object#either? instead of using [].include?.
* Added new #update_column method.Sebastian Martinez2011-03-275-14/+14
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Revert "Removed #update_attribute method. New #update_column method."Sebastian Martinez2011-03-275-14/+14
| | | | | | This reverts commit 45c233ef819dc7b67e259dd73f24721fec28b8c8. Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Removed #update_attribute method. New #update_column method.Sebastian Martinez2011-03-265-14/+14
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* adding missing requireAaron Patterson2011-03-221-0/+1
|
* oracle, y u defy meJon Leighton2011-03-161-1/+1
|
* Fix tests under postgres - we should always put conditions in the WHERE part ↵Jon Leighton2011-03-121-1/+1
| | | | not in ON constraints because postgres requires that the table has been joined before the condition references it.
* Add a test for STI on the through where the through is nested, and change ↵Jon Leighton2011-03-051-0/+9
| | | | the code which support this
* When preloading has_and_belongs_to_many associations, we should only ↵Jon Leighton2011-03-041-0/+16
| | | | instantiate one AR object per actual record in the database. (Even when IM is off.)
* Merge branch 'master' into nested_has_many_throughJon Leighton2011-03-0416-504/+1182
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activerecord/CHANGELOG activerecord/lib/active_record/association_preload.rb activerecord/lib/active_record/associations.rb activerecord/lib/active_record/associations/class_methods/join_dependency.rb activerecord/lib/active_record/associations/class_methods/join_dependency/join_association.rb activerecord/lib/active_record/associations/has_many_association.rb activerecord/lib/active_record/associations/has_many_through_association.rb activerecord/lib/active_record/associations/has_one_association.rb activerecord/lib/active_record/associations/has_one_through_association.rb activerecord/lib/active_record/associations/through_association_scope.rb activerecord/lib/active_record/reflection.rb activerecord/test/cases/associations/has_many_through_associations_test.rb activerecord/test/cases/associations/has_one_through_associations_test.rb activerecord/test/cases/reflection_test.rb activerecord/test/cases/relations_test.rb activerecord/test/fixtures/memberships.yml activerecord/test/models/categorization.rb activerecord/test/models/category.rb activerecord/test/models/member.rb activerecord/test/models/reference.rb activerecord/test/models/tagging.rb
| * Rewrote AssociationPreload.Jon Leighton2011-02-282-16/+15
| |
| * Use proper objects to do the work to build the associations (adding methods, ↵Jon Leighton2011-02-211-8/+12
| | | | | | | | callbacks etc) rather than calling a whole bunch of methods with rather long names.
| * Don't use skip, just don't run anything, we don't have skip in Ruby 1.8Emilio Tagua2011-02-181-4/+2
| |
| * Should use "=" instead "replace" after this commit: ↵Emilio Tagua2011-02-181-1/+1
| | | | | | | | 1644663ba7f678d178deab2bf1629dc05626f85b
| * No need to test agaisnt target.Emilio Tagua2011-02-181-1/+1
| |
| * Merge remote branch 'rails/master' into identity_mapEmilio Tagua2011-02-187-109/+40
| |\ | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/associations/association.rb activerecord/lib/active_record/fixtures.rb
| | * Split AssociationProxy into an Association class (and subclasses) which ↵Jon Leighton2011-02-186-109/+12
| | | | | | | | | | | | manages the association, and a CollectionProxy class which is *only* a proxy. Singular associations no longer have a proxy. See CHANGELOG for more.
| | * Allow building and then later saving has_many :through records, such that ↵Jon Leighton2011-02-181-0/+18
| | | | | | | | | | | | the join record is automatically saved too. This requires the :inverse_of option to be set on the source association in the join model. See the CHANGELOG for details. [#4329 state:resolved]
| | * Ensure that association_ids uses the correct attribute where the association ↵Jon Leighton2011-02-181-0/+10
| | | | | | | | | | | | is a has_many :through with a :primary_key option on the source reflection. [#6376 state:resolved]
| * | Run tests without IdentityMap when IM=false is given.Emilio Tagua2011-02-153-9/+13
| | |
| * | Merge remote branch 'rails/master' into identity_mapEmilio Tagua2011-02-1514-323/+801
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activerecord/examples/performance.rb activerecord/lib/active_record/association_preload.rb activerecord/lib/active_record/associations.rb activerecord/lib/active_record/associations/association_proxy.rb activerecord/lib/active_record/autosave_association.rb activerecord/lib/active_record/base.rb activerecord/lib/active_record/nested_attributes.rb activerecord/test/cases/relations_test.rb
| | * Add interpolation of association conditions back in, in the form of proc { ↵Jon Leighton2011-02-145-2/+24
| | | | | | | | | | | | ... } rather than instance_eval-ing strings
| | * Add missing requireSantiago Pastorino2011-02-131-0/+1
| | |
| | * Fix table name collision due to incorrect alias count on certain joins.Ernie Miller2011-02-121-0/+7
| | | | | | | | | | | | | | | | | | [#6423 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
| | * require tag since we need it for this testAaron Patterson2011-02-071-0/+1
| | |
| | * Correctly update counter caches on deletion for has_many :through [#2824 ↵Jon Leighton2011-02-072-10/+56
| | | | | | | | | | | | state:resolved]. Also fixed a bunch of other counter cache bugs in the process, as once I fixed this one others started appearing like nobody's business.
| | * Support the :dependent option on has_many :through associations. For ↵Jon Leighton2011-02-071-0/+100
| | | | | | | | | | | | historical and practical reasons, :delete_all is the default deletion strategy employed by association.delete(*records), despite the fact that the default strategy is :nullify for regular has_many. Also, this only works at all if the source reflection is a belongs_to. For other situations, you should directly modify the through association.
| | * Make record.association.destroy(*records) on habtm and hm:t only delete ↵Jon Leighton2011-02-072-12/+30
| | | | | | | | | | | | records in the join table. This is to make the destroy method more consistent across the different types of associations. For more details see the CHANGELOG entry.
| | * the connection pool caches table_exists? callsAaron Patterson2011-02-071-0/+2
| | |