aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_dispatch/journey/routes.rb
Commit message (Collapse)AuthorAgeFilesLines
* `ast` is no longer to be `nil` since #33118Ryuta Kamizono2019-04-051-1/+0
|
* Merge pull request #33054 from jboler/masterRafael França2018-07-031-0/+1
|\ | | | | Fix route eager loading
| * Only build routes simulator when there is an ASTJonathan Boler2018-06-041-0/+1
| |
* | Fix bug with eager_load in development environmentDan Jensen2018-06-111-1/+1
|/ | | | | Modifies the routes simulator to allow for empty RouteSets, which are created when secondary Engines are loaded.
* Use frozen string literal in actionpack/Kir Shatrov2017-07-291-0/+2
|
* Revert "Merge pull request #29540 from kirs/rubocop-frozen-string"Matthew Draper2017-07-021-1/+0
| | | | | This reverts commit 3420a14590c0e6915d8b6c242887f74adb4120f9, reversing changes made to afb66a5a598ce4ac74ad84b125a5abf046dcf5aa.
* Enforce frozen string in RubocopKir Shatrov2017-07-011-0/+1
|
* move route allocation to a factory method on the mapping objectAaron Patterson2015-08-151-8/+1
| | | | | | I would like to change the signature of the Route constructor. Since the mapping object has all the data required to construct a Route object, move the allocation to a factory method.
* only keep one hash of named routesAaron Patterson2015-08-141-4/+1
| | | | | The outer router object already keeps a hash of named routes, so we should just use that.
* rm add_route2Aaron Patterson2015-08-141-7/+1
| | | | | refactor the tests with a backwards compatible method call so we can rm add_route2 from the journey router
* pass pass the mapping object down the add_route stackAaron Patterson2015-08-141-2/+12
| | | | | then we can let the mapping object derive stuff that the Route object needs.
* extract required_defaults from the conditions hash before constructing the routeAaron Patterson2015-06-081-2/+2
| | | | | this way we can remove the strange "respond_to?" conditional in the `matches?` loop
* ActionDispatch::Journey::Routes#empty? test casesValentine Valyaeff2015-05-191-1/+1
|
* Added ActionDispatch::Journey::Routes#empty?juggernaut-2015-05-181-0/+4
|
* Partition routes during setup.Guo Xiang Tan2015-02-261-7/+12
| | | | | | | Partitioning of all the routes is currently being done during the first request. Since there is no need to clear the cache for `partitioned_routes` when adding a new route. We can move the partitioning of the routes during setup time.
* Clear named routes when routes.rb is reloadedAndrew White2013-07-211-0/+1
| | | | | | | | Fix an issue where Journey was failing to clear the named routes hash when the routes were reloaded and since it doesn't overwrite existing routes then if a route changed but wasn't renamed it kept the old definition. This was being masked by the optimised url helpers so it only became apparent when passing an options hash to the url helper.
* access `@path` and `@routes` via reader methods in journeyGosha Arinich2013-01-071-2/+2
|
* refactor Journey::RoutesGosha Arinich2013-01-071-12/+11
| | | | | * prefer do-end for multiline blocks * prefer or-equals over returns with checks
* update AD::Journey to follow Rails coding conventionsFrancesco Rodriguez2012-12-201-8/+8
|
* :nodoc: Journey because is not part of the public API [ci skip]Francesco Rodriguez2012-12-191-5/+4
|
* Integrate Journey into Action DispatchAndrew White2012-12-191-0/+77
Move the Journey code underneath the ActionDispatch namespace so that we don't pollute the global namespace with names that may be used for models. Fixes rails/journey#49.