Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | | | | | push parent up to the superclass | Aaron Patterson | 2013-10-09 | 3 | -7/+11 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We always want a linked list back to the root node. | |||||
| * | | | | | | | convert JoinBase to a tree and remove the Node class | Aaron Patterson | 2013-10-09 | 2 | -42/+30 | |
| | | | | | | | | ||||||
| * | | | | | | | names are guaranteed to be symbols | Aaron Patterson | 2013-10-08 | 1 | -1/+1 | |
| | | | | | | | | ||||||
| * | | | | | | | eliminate function that is only used in one place | Aaron Patterson | 2013-10-08 | 1 | -7/+2 | |
| | | | | | | | | ||||||
| * | | | | | | | we will always detect the same node, so just pass the node in | Aaron Patterson | 2013-10-08 | 1 | -7/+2 | |
| | | | | | | | | ||||||
| * | | | | | | | we do not need to to_s the name all the time | Aaron Patterson | 2013-10-08 | 1 | -5/+3 | |
| | | | | | | | | ||||||
| * | | | | | | | parent is guaranteed to be the same | Aaron Patterson | 2013-10-08 | 1 | -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 allocations | Aaron Patterson | 2013-10-08 | 1 | -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 tree | Aaron Patterson | 2013-10-08 | 1 | -7/+7 | |
| | | | | | | | | ||||||
| * | | | | | | | walk the tree rather than generating a hash and walking it | Aaron Patterson | 2013-10-08 | 1 | -21/+10 | |
| | | | | | | | | ||||||
| * | | | | | | | do not convert the tree to a list just for the first node | Aaron Patterson | 2013-10-08 | 1 | -1/+1 | |
| | | | | | | | | ||||||
| * | | | | | | | generate the association hash from the top down | Aaron Patterson | 2013-10-08 | 1 | -17/+15 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This prevents us from walking back up the parent tree once for each leaf. | |||||
| * | | | | | | | store join parts in a tree | Aaron Patterson | 2013-10-08 | 1 | -16/+45 | |
| | | | | | | | | ||||||
* | | | | | | | | Merge pull request #12485 from arunagw/code-removal | Rafael Mendonça França | 2013-10-09 | 2 | -35/+0 | |
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | viewpathtests file moved | |||||
| * | | | | | | | | Moved viewpathstest into actionview/test/actionpack/controller | Arun Agrawal | 2013-10-09 | 1 | -0/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These test were not running at all, Now it will run as a tests part. | |||||
| * | | | | | | | | FakeController is not required in AV | Arun Agrawal | 2013-10-09 | 1 | -35/+0 | |
|/ / / / / / / / | ||||||
* | | | | | | | | Merge pull request #12482 from francisgo/patch-11 | Yves Senn | 2013-10-09 | 1 | -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 Go | 2013-10-10 | 1 | -1/+1 | |
|/ / / / / / / / | ||||||
* | | | | | | | | Merge pull request #12481 from arunagw/fixes | Rafael Mendonça França | 2013-10-09 | 6 | -10/+11 | |
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | Few fixes which includes revert fixes | |||||
| * | | | | | | | | Added ActionView in rake about [ci skip] | Arun Agrawal | 2013-10-09 | 1 | -5/+6 | |
| | | | | | | | | | ||||||
| * | | | | | | | | Minor align BR tags with HTML void - with reference to the generators code ↵ | AvnerCohen | 2013-10-09 | 2 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | change - https://github.com/rails/rails/commit/f038d4cc5fe1ac21f92c7e32c61ff85a0c816f3f | |||||
| * | | | | | | | | Using flat_map instead of map and flatten | Arun Agrawal | 2013-10-09 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | original commit 8998441967a8cfc6e4302c29664ab9d0acd77704 Reverted here ec8ef1e1055c4e1598da13f49d30261f07f4a9b4 | |||||
| * | | | | | | | | Typo fix [ci skip] | Arun Agrawal | 2013-10-09 | 1 | -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 Agrawal | 2013-10-09 | 1 | -1/+1 | |
|/ / / / / / / / | ||||||
* | | | | | | | | Merge pull request #12283 from JuanitoFatas/rails-on-rack | Yves Senn | 2013-10-09 | 1 | -1/+5 | |
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | [ci skip] Add explanation on Rack::Sendfile middleware in rails_on_rack.md. | |||||
| * | | | | | | | | [ci skip] Add explanation on Rack::Sendfile and fix serve static files setting. | Juanito Fatas | 2013-10-04 | 1 | -1/+5 | |
| | | | | | | | | | ||||||
* | | | | | | | | | Merge pull request #12479 from benpickles/unshift-middleware | José Valim | 2013-10-09 | 3 | -0/+14 | |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Expose MiddlewareStack#unshift to environment configuration | |||||
| * | | | | | | | | | Expose MiddlewareStack#unshift to environment configuration. | Ben Pickles | 2013-10-09 | 3 | -0/+14 | |
| | |_|/ / / / / / | |/| | | | | | | | ||||||
* | | | | | | | | | add dots after `Fixes #YYYYY` in actionpack CHANGELOG. [ci skip] | Yves Senn | 2013-10-09 | 1 | -12/+12 | |
| | | | | | | | | | ||||||
* | | | | | | | | | Merge pull request #12083 from BlueHotDog/fixing_respond_with | Yves Senn | 2013-10-09 | 4 | -0/+25 | |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Fixing repond_with working directly on the options hash | |||||
| * | | | | | | | | | Fixing repond_with working directly on the options hash | BlueHotDog | 2013-10-09 | 4 | -0/+25 | |
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes an issue where the respond_with worked directly with the given options hash, so that if a user relied on it after calling respond_with, the hash wouldn't be the same. Fixes #12029 | |||||
* | | | | | | | | | Merge pull request #12473 from claudiob/document-after-touch | Yves Senn | 2013-10-09 | 2 | -2/+6 | |
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | | Add documentation for after_touch [ci skip] | |||||
| * | | | | | | | | Add documentation for after_touch [ci skip] | claudiob | 2013-10-08 | 2 | -2/+6 | |
| | |/ / / / / / | |/| | | | | | | ||||||
* | | | | | | | | :scissors: | Rafael Mendonça França | 2013-10-09 | 2 | -2/+2 | |
| | | | | | | | | ||||||
* | | | | | | | | Push define_accessors to class level since we don't need anythig from | Rafael Mendonça França | 2013-10-08 | 3 | -8/+8 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the instance | |||||
* | | | | | | | | Use the reflection name instead of the accessor | Rafael Mendonça França | 2013-10-08 | 2 | -1/+2 | |
| | | | | | | | | ||||||
* | | | | | | | | Move add_counter_cache_methods to class level | Rafael Mendonça França | 2013-10-08 | 1 | -2/+2 | |
| | | | | | | | | ||||||
* | | | | | | | | Move define_constructors to class level | Rafael Mendonça França | 2013-10-08 | 1 | -3/+2 | |
| | | | | | | | | ||||||
* | | | | | | | | Move define_writers to class level since we don't need anything from the | Rafael Mendonça França | 2013-10-08 | 2 | -3/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | instance | |||||
* | | | | | | | | Move define_readers to class method since it doesn't need nothing from | Rafael Mendonça França | 2013-10-08 | 2 | -5/+3 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the instance | |||||
* | | | | | | | | Fix the indentation | Rafael Mendonça França | 2013-10-08 | 1 | -12/+13 | |
|/ / / / / / / | ||||||
* | | | | | | | join parts in `build` are *always* new, so don't bother checking | Aaron Patterson | 2013-10-08 | 1 | -1/+2 | |
| | | | | | | | ||||||
* | | | | | | | stop looking up the reflection multiple times | Aaron Patterson | 2013-10-08 | 1 | -6/+5 | |
| | | | | | | | ||||||
* | | | | | | | Merge pull request #12470 from nameisvijay/activemodel_example_fix | Rafael Mendonça França | 2013-10-08 | 1 | -0/+1 | |
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | adding load_path to include active_model in the load path of example | |||||
| * | | | | | | | adding load_path to include active_model in the load path of example | Vijay Ubuntu | 2013-10-09 | 1 | -0/+1 | |
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | similar to activerecord/examples/simple.rb | |||||
* | | | | | | | cache join associations and join base on the stack | Aaron Patterson | 2013-10-08 | 1 | -4/+11 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | No reason to decompose the join objects on every iteration | |||||
* | | | | | | | use the API rather than accessing underlying data structures | Aaron Patterson | 2013-10-08 | 1 | -1/+1 | |
| | | | | | | | ||||||
* | | | | | | | associations is always a hash | Aaron Patterson | 2013-10-08 | 1 | -19/+13 | |
| | | | | | | | ||||||
* | | | | | | | keys are always symbols | Aaron Patterson | 2013-10-08 | 1 | -2/+4 | |
| | | | | | | | ||||||
* | | | | | | | we can get a list of reflections from the join parts, so let's remove | Aaron Patterson | 2013-10-08 | 1 | -3/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | the @reflections ivar. |