aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/associations
Commit message (Collapse)AuthorAgeFilesLines
* 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-162-5/+5
|
* 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]
* Remove unused varSantiago Pastorino2010-11-141-1/+1
|
* Fix bug where size of through association is not correct after adding a ↵Jason Cheow2010-11-141-1/+0
| | | | | | | | has_many association (occurs only before main object has been reloaded). [#5968 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Execute less operationsSantiago Pastorino2010-11-091-2/+1
|
* use persisted? instead of new_record? wherever possibleDavid Chelimsky2010-11-097-25/+26
| | | | | | | | | | | - 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>
* Association Proxy should not undefine the default respond_to_missingsblackstone2010-11-041-1/+1
|
* Refactoring: replace the mix of variables like @finder_sql, @counter_sql, ↵Jon Leighton2010-10-3010-157/+144
| | | | 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]
* no need of nil checkNeeraj Singh2010-09-291-1/+1
|
* Refactor association_collection uniq method.Emilio Tagua2010-09-221-4/+3
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Perf: refactor method.Emilio Tagua2010-09-221-11/+8
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* do not pass nil values to arelAaron Patterson2010-09-211-1/+1
|
* Use scoped here to get the scoped whereSantiago Pastorino2010-09-051-5/+3
|
* mark SQL literals as SQL literalsAaron Patterson2010-08-251-1/+1
|
* marking sql literals as sql literalsAaron Patterson2010-08-231-1/+1
|
* 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>
* Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵Santiago Pastorino2010-08-143-10/+10
| | | | 's/[ \t]*$//' -i {} \;)
* Removing most of the symbol to proc usage in Active RecordPrem Sichanugrist2010-08-131-1/+1
| | | | This will hopefully make Active Record run a bit more faster.
* removing references to arel constantsAaron Patterson2010-08-131-1/+1
|
* avoiding symbol to proc againAaron Patterson2010-08-131-1/+1
|
* do not use arel constants directlyAaron Patterson2010-08-131-1/+1
|
* symbol to proc is slow, we should avoid itAaron Patterson2010-08-131-1/+1
|
* dry up the hash dup and avoid sending nil valuesAaron Patterson2010-08-112-11/+15
|
* standarizes the use of the article "an" for "SQL" and "SQLite"Xavier Noria2010-08-051-1/+1
|
* ensuring that description does not exceed 100 columnsNeeraj Singh2010-08-022-6/+8
|
* Don't increment and then decrement the same counter when re-assigning a ↵Tarmo Tänav2010-07-291-1/+1
| | | | | | | | belongs_to association [#2786 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.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>
* Ensure all join table attributes will be in the same timestamp.José Valim2010-07-211-11/+21
|
* Timestamp columns of HABTM join table should record timestampsNeeraj Singh2010-07-211-1/+5
| | | | [#5161 state:resolved]
* 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>
* id is a public methodSantiago Pastorino2010-07-211-2/+2
|
* Ensure that primary_keys of HABTM records is not double quotedNeeraj Singh2010-07-201-2/+2
| | | | [#5152 state:reslved]
* 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>
* string IDs are now quoted correctly [#5064 state:resolved]Will St. Clair + Neeraj Singh2010-07-131-1/+1
| | | | 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>
* splitting a really long line into multiple lines which is easy on eyesNeeraj Singh2010-06-291-1/+5
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Don't define block, just yield if block is given.Emilio Tagua2010-06-241-7/+7
|