aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/associations
Commit message (Collapse)AuthorAgeFilesLines
...
| | * | `Relation#count` doesn't use options anymore.Vipul A M2013-10-132-5/+3
| | | |
| * | | Merge pull request #12511 from ↵Rafael Mendonça França2013-10-131-3/+3
| |\ \ \ | | | | | | | | | | | | | | | | | | | | jetthoughts/informative_raise_message_for_incorrect_association Make missed association exception message more informative
| | * | | Make missed association exception message more informativePaul Nikitochkin2013-10-131-3/+3
| | |/ / | | | | | | | | | | | | | | | | Add target class name, which should have missed association on preload, into exception message to simplify detecting problem part.
| * / / Drop unused iterator varVipul A M2013-10-131-1/+1
| |/ /
* | | move column_names_with_alias on to the alias cache objectAaron Patterson2013-10-144-37/+22
| | |
* | | keep a cache on the alias objectAaron Patterson2013-10-142-16/+21
| | |
* | | move alias building to the table nodeAaron Patterson2013-10-142-8/+8
| | |
* | | let the Table class build the ARel alias nodesAaron Patterson2013-10-141-4/+9
| | |
* | | store aliases in a better structureAaron Patterson2013-10-141-4/+17
| | |
* | | remove delegation to `chain`Aaron Patterson2013-10-141-3/+1
| | |
* | | rm JoinHelper from JoinAssociationAaron Patterson2013-10-141-3/+1
| | |
* | | unhook `join_constraints` from the @tables ivarAaron Patterson2013-10-132-3/+3
| | |
* | | remove delegate methods that are not usedAaron Patterson2013-10-131-1/+1
| | |
* | | push the node->AR cache up one levelAaron Patterson2013-10-132-16/+12
| | |
* | | remove dead codeAaron Patterson2013-10-111-5/+0
| | |
* | | remove useless codeAaron Patterson2013-10-111-24/+20
| | | | | | | | | | | | | | | turns out that the activerecord object will always have the same id as the row from which it was constructed
* | | the parent hash keeps the records unique alreadyAaron Patterson2013-10-111-4/+3
| | |
* | | keep track of AR objects we've made as we walk the treeAaron Patterson2013-10-111-35/+22
| | | | | | | | | | | | | | | Remove duplicate removal code because we avoid adding duplicates as we walk the tree the first time
* | | push up the loaded target checkingAaron Patterson2013-10-111-9/+13
| | |
* | | we always have a model nowAaron Patterson2013-10-111-1/+1
| | |
* | | extract conditional to the callerAaron Patterson2013-10-111-11/+13
| | |
* | | we always have a reflection and it is on the nodeAaron Patterson2013-10-111-2/+2
| | |
* | | fix variable name. we're constructing a model, not an associationAaron Patterson2013-10-111-12/+12
| | |
* | | reflections know if they are collections, so askAaron Patterson2013-10-111-6/+5
| | | | | | | | | | | | | | | We don't need to poke at the macro to figure out if we should look for a cached record or not
* | | belongs_to can be treated the same way as has_oneAaron Patterson2013-10-111-12/+5
| | |
* | | cache the primary key / typcast lookup on the stackAaron Patterson2013-10-111-9/+9
| | |
* | | remove dead code / make duplicate code more obviousAaron Patterson2013-10-111-4/+4
| | |
* | | pull parent and alias tacker from the nodes.Aaron Patterson2013-10-114-20/+23
| | | | | | | | | | | | For now, we'll set the tables on the nodes manually.
* | | decouple the `join_constraints` method from the parent pointerAaron Patterson2013-10-112-2/+8
|/ /
* | no reason to do this column addition and subtraction businessAaron Patterson2013-10-111-1/+1
| |
* | refactor a little so we can remove some methodsAaron Patterson2013-10-102-4/+1
| |
* | the parent node is on the stack, so use itAaron Patterson2013-10-101-4/+4
| | | | | | | | | | we don't need to walk back up the node's parents when we have the parent on the stack
* | pass the parent node to the construct methodAaron Patterson2013-10-101-7/+6
| |
* | nodes do not need to be sortedAaron Patterson2013-10-101-1/+1
| |
* | delete unused codeAaron Patterson2013-10-102-50/+0
| | | | | | | | | | Now that we merge trees from the top down, we don't need to search through the whole tree for particular nodes, just walk it and merge.
* | merge JoinDependency as outer joinsAaron Patterson2013-10-101-0/+30
| | | | | | | | Merge JoinDependency objects as outer joins
* | stuff the join dependency object in the "anything goes" hash.Aaron Patterson2013-10-101-2/+2
| |
* | stop splatting things back and forthAaron Patterson2013-10-101-1/+1
| |
* | simplify each method.Aaron Patterson2013-10-101-8/+2
| | | | | | | | Stop writing terrible code Aaron. This is Ruby, not Scheme
* | stop creating a bunch of relation objects when we can slap all the joins on ↵Aaron Patterson2013-10-092-8/+3
| | | | | | | | at once
* | Merge branch 'builder-instances'Rafael Mendonça França2013-10-097-57/+42
|\ \
| * | Method visibility will not make difference hereRafael Mendonça França2013-10-094-8/+0
| | |
| * | Remove builder instancesRafael Mendonça França2013-10-092-22/+10
| | | | | | | | | | | | | | | All the job can be done at class level so we can avoid some object allocation
| * | Extract the scope building to a class methodRafael Mendonça França2013-10-092-10/+17
| | |
| * | Remove unneeded readerRafael Mendonça França2013-10-091-2/+0
| | |
| * | Move wrap_scope to class levelRafael Mendonça França2013-10-091-2/+2
| | |
| * | Move macro to class levelRafael Mendonça França2013-10-094-5/+5
| | |
| * | Make validate_options a class methodRafael Mendonça França2013-10-091-3/+3
| | |
| * | Make valid_options a class methodRafael Mendonça França2013-10-096-7/+7
| | |
| * | Define the association extensions without need to have a builderRafael Mendonça França2013-10-092-14/+14
| | | | | | | | | | | | instance