aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record
Commit message (Collapse)AuthorAgeFilesLines
* Explicitly exit with status "1" for create and drop failuresJay Hayes2013-11-112-10/+29
| | | | | | | | | | | | | | | | * A non-zero exit status allows subsequent shell commands to be chained together such as: `rake db:reset test:prepare && rspec && cap deploy` (if you're feeling brave :) * Any exceptions raised during the `create` and `drop` tasks are caught in order to print a "pretty" message to the user. Unfortunately doing so prevents rake from aborting with a non-zero exit status to the shell. * Therefore we re-raise the exceptions after the "pretty" message and re-catch them in the task. * From the task we explicitly exit with a non-zero status. This method was chosen (rather than just letting rake fail from the exception) so that the backtrace is suppressed and the output to stderr is unchanged. * Update activerecord CHANGELOG
* Extract a function to determine if the default value is a functionRafael Mendonça França2013-10-141-1/+5
|
* Push default_function to superclass to avoid method checkRafael Mendonça França2013-10-143-15/+18
|
* Dump the default function when the primary key is uuidRafael Mendonça França2013-10-141-0/+1
| | | | Fixes #12489
* Merge pull request #12486 from cjolly/newline-eof-structure-sqlRafael Mendonça França2013-10-131-0/+1
|\ | | | | newline at end of structure.sql file
| * newline at end of structure.sql fileChad Jolly2013-10-131-0/+1
| |
* | Merge pull request #12518 from vipulnsward/remove_count_optionsRafael Mendonça França2013-10-134-19/+16
|\ \ | | | | | | `Relation#count` doesn't use options anymore.
| * | Stop accepting `options` for `Relation#average`, `Relation#minimum`, ↵Vipul A M2013-10-142-11/+11
| | | | | | | | | | | | `Relation#maximum`, `Relation#calculate`, `perform_calculation`, `NullRelation#calculate` as they isn't used anymore.
| * | `Relation#count` doesn't use options anymore.Vipul A M2013-10-133-8/+5
| | |
* | | Add nodoc to methodRafael Mendonça França2013-10-131-1/+1
| | |
* | | Merge pull request #12508 from ↵Rafael Mendonça França2013-10-131-2/+12
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | jetthoughts/12415_generate_subqueries_for_relation_from_binding_params Generate subquery for Relation passed as array condition for where Conflicts: activerecord/CHANGELOG.md
| * | | Generate subquery for Relation passed as array condition for wherePaul Nikitochkin2013-10-131-2/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of executing 2 queries for fetching records filtered by array condition with Relation, added generation of subquery to current query. This behaviour will be consistent when passes Relation as hash condition to where Closes: #12415
* | | | 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
| |/ / |/| |
* | | Merge pull request #11791 from versioncontrol/includes_with_persistent_selectRafael Mendonça França2013-10-121-1/+1
|\ \ \ | |/ / |/| | Includes with persistent select, fixes #11773
| * | Fixes #11773 when using includes combined with select, the select statement ↵Edo Balvers2013-10-081-1/+1
| | | | | | | | | | | | was overwritten.
* | | 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-102-1/+31
| | | | | | | | | | | | Merge JoinDependency objects as outer joins
* | | stuff the join dependency object in the "anything goes" hash.Aaron Patterson2013-10-104-6/+8
| | |
* | | stop splatting things back and forthAaron Patterson2013-10-102-2/+2
| | |
* | | 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
* | | | no need to to_symAaron Patterson2013-10-091-1/+1
| | | |
* | | | Merge branch 'tree'Aaron Patterson2013-10-095-109/+105
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * tree: (22 commits) remove dead code add some convenient methods for avoiding array allocations hide join_constraints inside the JoinDependency object speed up match? expose the root node and call it just skip the join if it's already there speed up finding existing nodes make node search more efficient remove == so we can see where walking up parents occurs push parent up to the superclass convert JoinBase to a tree and remove the Node class names are guaranteed to be symbols eliminate function that is only used in one place we will always detect the same node, so just pass the node in we do not need to to_s the name all the time parent is guaranteed to be the same reduce number of comparisons and array allocations rename the variable to make more sense with a tree walk the tree rather than generating a hash and walking it do not convert the tree to a list just for the first node ...
| * | | remove dead codeAaron Patterson2013-10-091-13/+0
| | | |
| * | | add some convenient methods for avoiding array allocationsAaron Patterson2013-10-092-3/+6
| | | |
| * | | hide join_constraints inside the JoinDependency objectAaron Patterson2013-10-092-8/+7
| | | |
| * | | speed up match?Aaron Patterson2013-10-091-0/+1
| | | |
| * | | expose the root node and call itAaron Patterson2013-10-091-21/+11
| | | |
| * | | just skip the join if it's already thereAaron Patterson2013-10-091-14/+10
| | | | | | | | | | | | | | | | Remove the annoying `find_or_build` stuff.
| * | | speed up finding existing nodesAaron Patterson2013-10-091-3/+1
| | | | | | | | | | | | | | | | | | | | The node must be a child of the parent passed in, so only search children of the parent node
| * | | make node search more efficientAaron Patterson2013-10-094-3/+43
| | | | | | | | | | | | | | | | | | | | | | | | Rather than search every node in the tree, comparing that node and all of its parents every time, start at the root from both sides and work our way down the tree
| * | | remove == so we can see where walking up parents occursAaron Patterson2013-10-094-25/+15
| | | |
| * | | push parent up to the superclassAaron Patterson2013-10-093-7/+11
| | | | | | | | | | | | | | | | We always want a linked list back to the root node.