aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/associations/association_collection.rb
Commit message (Collapse)AuthorAgeFilesLines
* 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
|
* use persisted? instead of new_record? wherever possibleDavid Chelimsky2010-11-091-10/+11
| | | | | | | | | | | - persisted? is the API defined in ActiveModel - makes it easier for extension libraries to conform to ActiveModel APIs without concern for whether the extended object is specifically ActiveRecord [#5927 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Refactoring: replace the mix of variables like @finder_sql, @counter_sql, ↵Jon Leighton2010-10-301-37/+29
| | | | 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.
* Refactor AssociationCollection#include? with objects in memoryCarlos Antonio da Silva2010-10-021-4/+3
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* AssociationCollection#include? working properly for objects added with build ↵Marcelo Giorgi2010-09-301-0/+13
| | | | method [#3472 state:resolved]
* Refactor association_collection uniq method.Emilio Tagua2010-09-221-4/+3
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Use scoped here to get the scoped whereSantiago Pastorino2010-09-051-5/+3
|
* While creating a new record using has_many create method default scope of ↵Neeraj Singh2010-08-191-1/+6
| | | | | | | | | | | child should be respected. author.posts.create should take into account default_scope defined on post. [#3939: state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* fixed joining of attributes when using find_or_create_by with multiple ↵Jon Buda2010-07-281-1/+1
| | | | | | attributes through an association Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* No need to check for :uniqNeeraj Singh2010-07-261-6/+3
| | | | | | [#5170 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Make use of tap to return a previously used varSantiago Pastorino2010-07-211-4/+4
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* association load target shouldn't replace recordsSzetobo2010-07-211-3/+4
| | | | | | | | from db if it is already loaded by nested attributes assignment [#5053 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* In nested_attributes when association is not loaded and associationSubba Rao Pasupuleti2010-07-211-1/+8
| | | | | | | | record is saved and then in memory record attributes should be saved [#5053 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Removed warnings when a variable is shadowedŁukasz Strzałkowski2010-07-191-3/+3
|
* Remove unintentional API changes. [#1108]Pratik Naik2010-07-141-9/+4
|
* count method should not take options if it is operated on has_many ↵Neeraj Singh2010-07-131-2/+5
| | | | | | | | association which has finder_sql or counter_sql [#2395 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* fix ActiveRecord `destroy_all` so it returns destroyed recordsMislav Marohnić2010-07-081-3/+4
| | | | Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Don't remove scheduled destroys when loading an association. [#4642 ↵James Le Cuirot2010-07-011-1/+6
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* AssociationCollection#create_by_*, find_or_create_by_* work properly now. ↵Aaron Patterson2010-06-291-0/+11
| | | | | | [#1108 state:resolved] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Don't define block, just yield if block is given.Emilio Tagua2010-06-241-7/+7
|
* Merge remote branch 'rails/master'Xavier Noria2010-06-201-1/+5
|\ | | | | | | | | Conflicts: actionpack/lib/abstract_controller/base.rb
| * Don't overwrite unsaved updates when loading an association but preserve the ↵James Le Cuirot2010-06-201-1/+5
| | | | | | | | | | | | | | | | order of the loaded records. Reapplied from before but now allows already-saved records to be refreshed. [#4830 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | Adds title to activerecord/lib/active_record/associations/*Rizwan Reza2010-06-161-0/+2
|/
* Revert "Don't overwrite unsaved updates when loading an association but ↵José Valim2010-06-111-1/+1
| | | | | | | | preserve the order of the loaded records. [#4642 state:open]" This commit introduced a regression described in ticket [#4830]. This reverts commit 0265c708b9696c3943518ad5f3dabdc22c5eba11.
* Don't overwrite unsaved updates when loading an association but preserve the ↵James Le Cuirot2010-06-091-1/+1
| | | | | | order of the loaded records. [#4642 state:resolved] Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* Properly cache association_collection#scopes calls having argumentsPratik Naik2010-06-041-1/+2
|
* Ensure not to load the entire association when bulk updating existing ↵Pratik Naik2010-04-141-10/+10
| | | | records using nested attributes
* Avoid deprecated String#to_a by using Array.wrap(...) instead of Array(...)Jeremy Kemper2010-04-101-1/+2
|
* Some doc updates reflecting the new query APIPratik Naik2010-04-081-1/+1
|
* Reset named scope cache whenever the @target is resetPratik Naik2010-04-071-2/+9
|
* Memoize association.named_scope callsPratik Naik2010-04-051-0/+3
|
* keep AssociationProxy#to_a to avoid warning in Array(post.author), and ↵Xavier Noria2010-03-121-0/+1
| | | | | | | | delegate it to @target in association collections [#4165 state:committed] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Get rid of construct_count_options_from_argsPratik Naik2010-01-191-3/+4
|
* Inherit named scope class Scope from RelationPratik Naik2010-01-181-2/+0
|
* Make scopes use relations under the hoodPratik Naik2010-01-161-3/+6
|
* Add Relation#includes to be an equivalent of current finder option :includePratik Naik2010-01-031-1/+1
|
* Get rid of Model.construct_finder_arel_with_includes. Use ↵Pratik Naik2010-01-031-1/+1
| | | | construct_finder_arel instead
* Get rid of DeprecatedCallbacks in ActiveRecord::Associations and finally ↵José Valim2009-12-301-1/+8
| | | | remove it.
* Migrate all the calculation methods to RelationPratik Naik2009-12-291-1/+1
|
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-06 05:56:08 +0000