aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/associations/association_collection.rb
Commit message (Collapse)AuthorAgeFilesLines
* Rename AssociationProxy#loaded to loaded! as it mutates the associationJon Leighton2011-01-301-1/+1
|
* Abstract load_target conditional logicJon Leighton2011-01-301-1/+1
|
* Let's be less blasé about method visibility on association proxiesJon Leighton2011-01-301-37/+38
|
* We shouldn't be using scoped.scoping { ... } to build associated records, as ↵Jon Leighton2011-01-301-8/+5
| | | | this can affect validations/callbacks/etc inside the record itself [#6252 state:resolved]
* Nested attributes and in-memory changed values #first and #[] behaviour ↵Alexey Nayden2011-01-281-1/+1
| | | | consistency fix
* push !loaded? conditional upAaron Patterson2011-01-141-10/+8
|
* return early in case the left or right side lists are emptyAaron Patterson2011-01-141-5/+4
|
* only find_target can raise the exception, so isolate the rescue around that callAaron Patterson2011-01-141-5/+9
|
* we always have a target, so stop checkingAaron Patterson2011-01-141-1/+1
|
* we should use [] instead of Array.newAaron Patterson2011-01-141-1/+1
|
* @target is always a list, so stop doing is_a? checksAaron Patterson2011-01-141-1/+1
|
* move complex logic to it's own methodAaron Patterson2011-01-141-15/+19
|
* loaded? will not raise an AR::RecordNotFound exception, so move the rescue ↵Aaron Patterson2011-01-141-4/+4
| | | | inside the conditional
* include_in_memory? should check against @target list in case of new records. ↵Aaron Patterson2011-01-121-2/+2
| | | | [#6257 state:resolved]
* use array maths rather than *argsAaron Patterson2011-01-121-2/+4
|
* just use respond_to? and super rather than aliasing around methodsAaron Patterson2011-01-121-1/+1
|
* Raise ActiveRecord::RecordNotSaved if an AssociationCollection fails to be ↵Jon Leighton2011-01-111-1/+5
| | | | replaced
* Not really worth having the HasAssociation module for just a single methodJon Leighton2011-01-071-2/+0
|
* Construct an actual ActiveRecord::Relation object for the association scope, ↵Jon Leighton2011-01-071-17/+10
| | | | rather than a hash which is passed to apply_finder_options. This allows more flexibility in how the scope is created, for example because scope.where(a, b) and scope.where(a).where(b) mean different things.
* no need to send a symbol to send()Aaron Patterson2011-01-051-1/+1
|
* Create the association scope directly rather than going through with_scopeJon Leighton2011-01-031-1/+1
|
* Use the association directly in other places tooJon Leighton2011-01-031-5/+2
|
* Let AssociationCollection#find use #scoped to do its finding. Note that I am ↵Jon Leighton2011-01-031-39/+42
| | | | removing test_polymorphic_has_many_going_through_join_model_with_disabled_include, since this specifies different behaviour for an association than for a regular scope. It seems reasonable to expect scopes and association proxies to behave in roughly the same way rather than having subtle differences.
* Rename AssociationProxy#foreign_key_present to foreign_key_present?Jon Leighton2011-01-031-1/+1
|
* Refactor BelongsToAssociation to allow BelongsToPolymorphicAssociation to ↵Jon Leighton2010-12-311-4/+0
| | | | inherit from it
* Specify the STI type condition using SQL IN rather than a whole load of ORs. ↵Jon Leighton2010-12-311-1/+1
| | | | Required a fix to ActiveRecord::Relation#merge for properly merging create_with_value. This also fixes a situation where the type condition was appearing twice in the resultant SQL query.
* I N C E P T I O N: flatten_deeper works around a bug in Ruby 1.8.2.Aaron Patterson2010-12-271-2/+2
|
* Refactor we_can_set_the_inverse_on_this? to use a less bizarre name amongst ↵Jon Leighton2010-12-261-4/+2
| | | | other things
* Add a HasAssociation module for common code for has_* associationsJon Leighton2010-12-261-0/+2
|
* Associations: DRY up the code which is generating conditions, and make it ↵Jon Leighton2010-12-261-1/+1
| | | | all use arel rather than SQL strings
* scoped.where_values_hash is never nilJon Leighton2010-12-261-1/+1
|
* When a has_many association is not :uniq, appending the same record multiple ↵Jon Leighton2010-12-231-2/+2
| | | | times should append it to the @target multiple times [#5964 state:resolved]
* no use for set, no need to to_ary, reduce extra objectsAaron Patterson2010-12-201-3/+2
|
* use array arithmetic rather than create setsAaron Patterson2010-12-201-4/+2
|
* Refactor create_record and build_record in AssociationCollectionJon Leighton2010-12-201-17/+5
|
* Delete create, create! and create_record from HasManyThroughAssociation in ↵Jon Leighton2010-12-201-12/+14
| | | | exchange for more generic versions in AssociationCollection
* Specify insert_record with NotImplementedError in AssociationCollection, to ↵Jon Leighton2010-12-201-0/+11
| | | | indicate that subclasses should implement it. Also add save_record to reduce duplication.
* Replace rudimentary named_scope with scope. [#6052 state:resolved]Pavel Gorbokon2010-12-151-8/+8
| | | | | | | * rename method names (actually in tests) * rename instance variable @_named_scopes_cache to @_scopes_cache * rename references in doc comments * don't touch CHANGELOG :)
* Partialy revert f1c13b0dd7b22b5f6289ca1a09f1d7a8c7c8584bJosé Valim2010-11-281-11/+11
|
* nil check unneeded ht. exvivaSantiago Pastorino2010-11-251-1/+1
|
* Use ternary instead explicit return.Emilio Tagua2010-11-251-2/+1
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Remove return, we are already returning self.Emilio Tagua2010-11-251-6/+6
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* class inheritable attributes is used no more! all internal use of class ↵Josh Kalderimis2010-11-201-1/+1
| | | | | | inheritable has been changed to class_attribute. class inheritable attributes has been deprecated. Signed-off-by: José Valim <jose.valim@gmail.com>
* super automatically passes on the implicit blockAaron Patterson2010-11-161-5/+1
|
* use unless instead of if !Aaron Patterson2010-11-161-1/+1
|
* removing space errorsAaron Patterson2010-11-161-3/+3
|
* reloading an association will properly set attributes of instantiated ↵Aaron Patterson2010-11-161-1/+3
| | | | objects. Thanks Brian Palmer [#5802 state:resolved]
* refactoring uniq methodAaron Patterson2010-11-161-7/+4
|
* if association is already loaded and if a find operation is performed on the ↵Neeraj Singh2010-11-151-1/+2
| | | | | | already loaded association list with an empty hash then do not perform another sql [#5972 state:resolved]
* Execute less operationsSantiago Pastorino2010-11-091-2/+1
|
generated by cgit v1.2.3 (git 2.25.1) at 2024-08-29 03:28:17 +0000