aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Fix wrong variable name used in the select_day method documentationJefferson Queiroz Venerando2013-10-101-2/+2
| | | The variable name created in the example is `my_date`, the methods were using `my_time` instead.
* Respect `SCRIPT_NAME` when using `redirect` with a relative pathAndrew White2013-10-103-0/+136
| | | | | | | | | | | | | | | | Example: # application routes.rb mount BlogEngine => '/blog' # engine routes.rb get '/admin' => redirect('admin/dashboard') This now redirects to the path `/blog/admin/dashboard`, whereas before it would've generated an invalid url because there would be no slash between the host name and the path. It also allows redirects to work where the application is deployed to a subdirectory of a website. Fixes #7977
* 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-098-59/+43
|\
| * 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-093-16/+15
| | | | | | | | instance
* | Merge pull request #12488 from CoryFoy/masterRafael Mendonça França2013-10-091-6/+16
|\ \ | | | | | | [ci skip] Modifies the initialization guide to make the initializers process clearer
| * | Modifies the initialization guide to make the initializers process clearer ↵Cory Foy2013-10-091-6/+16
| | | | | | | | | | | | and remove a TODO. [ci skip]
* | | 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.
| * | convert JoinBase to a tree and remove the Node classAaron Patterson2013-10-092-42/+30
| | |
| * | names are guaranteed to be symbolsAaron Patterson2013-10-081-1/+1
| | |
| * | eliminate function that is only used in one placeAaron Patterson2013-10-081-7/+2
| | |
| * | we will always detect the same node, so just pass the node inAaron Patterson2013-10-081-7/+2
| | |
| * | we do not need to to_s the name all the timeAaron Patterson2013-10-081-5/+3
| | |
| * | parent is guaranteed to be the sameAaron Patterson2013-10-081-4/+1
| | | | | | | | | | | | | | | since we are searching a tree, the current level should be consistent, so we don't have to inspect the parent class / table
| * | reduce number of comparisons and array allocationsAaron Patterson2013-10-081-11/+10
| | | | | | | | | | | | | | | Just inspecting the children nodes will reduce the number of comparisons we have to do (since it is limited to the child size)
| * | rename the variable to make more sense with a treeAaron Patterson2013-10-081-7/+7
| | |
| * | walk the tree rather than generating a hash and walking itAaron Patterson2013-10-081-21/+10
| | |
| * | do not convert the tree to a list just for the first nodeAaron Patterson2013-10-081-1/+1
| | |
| * | generate the association hash from the top downAaron Patterson2013-10-081-17/+15
| | | | | | | | | | | | | | | This prevents us from walking back up the parent tree once for each leaf.
| * | store join parts in a treeAaron Patterson2013-10-081-16/+45
| | |
* | | Merge pull request #12485 from arunagw/code-removalRafael Mendonça França2013-10-092-35/+0
|\ \ \ | | | | | | | | viewpathtests file moved
| * | | Moved viewpathstest into actionview/test/actionpack/controllerArun Agrawal2013-10-091-0/+0
| | | | | | | | | | | | | | | | These test were not running at all, Now it will run as a tests part.
| * | | FakeController is not required in AVArun Agrawal2013-10-091-35/+0
|/ / /
* | | Merge pull request #12482 from francisgo/patch-11Yves Senn2013-10-091-1/+1
|\ \ \ | | | | | | | | [Validations Guide] Add full stop to the end of sentence [ci skip]
| * | | [Validations Guide] Add full stop to the end of sentence [ci skip]Francis Go2013-10-101-1/+1
|/ / /
* | | Merge pull request #12481 from arunagw/fixesRafael Mendonça França2013-10-096-10/+11
|\ \ \ | | | | | | | | Few fixes which includes revert fixes
| * | | Added ActionView in rake about [ci skip]Arun Agrawal2013-10-091-5/+6
| | | |
| * | | Minor align BR tags with HTML void - with reference to the generators code ↵AvnerCohen2013-10-092-2/+2
| | | | | | | | | | | | | | | | change - https://github.com/rails/rails/commit/f038d4cc5fe1ac21f92c7e32c61ff85a0c816f3f
| * | | Using flat_map instead of map and flattenArun Agrawal2013-10-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | original commit 8998441967a8cfc6e4302c29664ab9d0acd77704 Reverted here ec8ef1e1055c4e1598da13f49d30261f07f4a9b4
| * | | Typo fix [ci skip]Arun Agrawal2013-10-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixing the typo which is formed a not required link. Check here http://api.rubyonrails.org/classes/ActionController/Base.html under paramters section keeping it under tt tag gets reverted here ec8ef1e1055c4e1598da13f49d30261f07f4a9b4
| * | | observers are not part of ActiveModel [ci skip]Arun Agrawal2013-10-091-1/+1
|/ / /