aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_dispatch/routing
Commit message (Collapse)AuthorAgeFilesLines
* Copy literal route constraints to defaults - fixes #3571 and #6224.Andrew White2012-05-111-0/+19
|
* Don't ignore nil positional arguments for url helpers - fixes #6196.Andrew White2012-05-101-1/+1
|
* Refactor Generator class to not rely on in-place editing the controllerAndrew White2012-05-091-5/+10
|
* Fix bug when url_for changes controller.Nikita Beloglazov2012-05-091-2/+2
|
* Fix that optimized named routes should also work as singleton methods on the ↵Jeremy Kemper2012-05-061-1/+2
| | | | url_helpers module
* Force given path to http methods in mapper to skip canonical action checkingCarlos Antonio da Silva2012-05-041-2/+3
| | | | | | | | | | | | | | | This fixes the following scenario: resources :contacts do post 'new', action: 'new', on: :collection, as: :new end Where the /new path is not generated because it's considered a canonical action, part of the normal resource actions: new_contacts POST /contacts(.:format) contacts#new Fixes #2999
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-05-041-1/+1
|\
| * Corrected the name of the module that should be included to get the url helpers.Anuj Dutta2012-05-031-1/+1
| |
* | RouteSet: optimize routes generation when globbing is usedBogdan Gusiev2012-05-031-2/+5
| |
* | use extract_options!Paul McMahon2012-05-031-1/+2
| |
* | Merge pull request #6138 from bogdan/routesJosé Valim2012-05-031-19/+10
|\ \ | | | | | | RouteSet: remove some code dups
| * | RouteSet: remove some code dupsBogdan Gusiev2012-05-021-19/+10
| | |
* | | Reset the request parameters after a constraints checkAndrew White2012-05-021-0/+2
|/ / | | | | | | | | | | | | | | | | | | | | | | | | A callable object passed as a constraint for a route may access the request parameters as part of its check. This causes the combined parameters hash to be cached in the environment hash. If the constraint fails then any subsequent access of the request parameters will be against that stale hash. To fix this we delete the cache after every call to `matches?`. This may have a negative performance impact if the contraint wraps a large number of routes as the parameters hash is built by merging GET, POST and path parameters. Fixes #2510.
* / ActionPack routes: remove some useless code.Bogdan Gusiev2012-05-021-26/+4
|/
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-05-011-8/+6
|\
| * Reword polymorphic routes + mounted engine rdocMark Rushakoff2012-04-291-8/+6
| | | | | | | | Also correctly indent its example code.
* | Restore interpolation of path option in redirect routesAndrew White2012-04-291-0/+10
| |
* | Escape interpolated params when redirecting - fixes #5688Andrew White2012-04-291-1/+7
|/
* Shadowing variable warning removedArun Agrawal2012-04-261-3/+3
|
* Allow loading external route files from the routerJose and Yehuda2012-04-252-0/+18
| | | | | | | | | This feature enables the ability to load an external routes file from the router via: draw :filename External routes files go in +config/routes+. This feature works in both engines and applications.
* Remove default match without specified methodJose and Yehuda2012-04-241-2/+12
| | | | | | | | | | | | | | | | In the current router DSL, using the +match+ DSL method will match all verbs for the path to the specified endpoint. In the vast majority of cases, people are currently using +match+ when they actually mean +get+. This introduces security implications. This commit disallows calling +match+ without an HTTP verb constraint by default. To explicitly match all verbs, this commit also adds a :via => :all option to +match+. Closes #5964
* Refactor hash creation in routesetCarlos Antonio da Silva2012-04-241-3/+4
|
* RouteSet: decomplecting a way to handle positional argsBogdan Gusiev2012-04-241-26/+20
|
* RouteSet: simplify routes helpers generation codeBogdan Gusiev2012-04-241-13/+21
|
* Add missing requires in routesBogdan Gusiev2012-04-231-0/+1
|
* Fix formattingOscar Del Ben2012-04-181-2/+2
|
* let's keep the slash in the return value instead of the path variable [ci skip]Vijay Dev2012-04-181-2/+2
|
* Warn about do end syntax in redirect optionOscar Del Ben2012-04-171-0/+3
|
* Fix redirect with block exampleAlexey Vakhov2012-04-171-3/+3
|
* Revert "Merge pull request #5750 from ahoward/master"Santiago Pastorino2012-04-081-10/+1
| | | | | | | This reverts commit 174cf8b22064c3999dfa685014d5b31ac993bb54, reversing changes made to 7ecd6a731bd60665bc6de94095137f0b2c4ada2a. The reverted commit improved the performance in the wrong place, now we have added this 6ddbd1844a6fd6aca2992f5f75c9f605cf89808f improvement.
* documents the contract for the argument of AD::Routing::RouteSet#url_forXavier Noria2012-04-051-0/+1
|
* Make url_for work when option is nil and default_url_options hasRafael Mendonça França2012-04-051-1/+1
| | | | stringified keys
* Fix url_for when options is nilRafael Mendonça França2012-04-051-1/+1
| | | | | RouteSet#url_for was modifying the options hash that belong to the caller
* url_for: avoid a nil check. unroll a loop. handle most common case 1st.ara.t.howard2012-04-051-18/+13
|
* carefully and quickly merge url_for optionsara.t.howard2012-04-041-1/+17
|
* Updated/changed useless tr/gsubsJurriaan Pruis2012-04-031-2/+2
|
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-03-271-0/+19
|\
| * add missing do [ci skip]Vijay Dev2012-03-271-1/+1
| |
| * added shallow_prefix option description in documantationLukasz Sarnacki2012-03-271-0/+19
| | | | | | | | It is in response for confusion in issue #5301
* | Allow a defining custom member field on resourcesJamie Macey2012-03-251-4/+5
|/ | | | | | | | | | By default, resources routes are created with :resource/:id. A model defining to_param can make prettier urls by using something more readable than an integer ID, but since the route picks it up as :id you wind up with awkward User.find_by_username(params[:id]) calls. By overriding the key to be used in @request.params you can be more obvious in your intent.
* Add missing requireSantiago Pastorino2012-03-171-0/+1
|
* Remove unused requireSantiago Pastorino2012-03-171-1/+0
|
* Route root helper shortcutBrian Cardarella2012-03-031-0/+5
| | | | Allow the root route helper to accept just a string
* don't pass unnecessary argumentSergey Nartimov2012-03-021-2/+2
|
* Optimize url helpers.Sergey Nartimov + José Valim2012-03-022-11/+13
|
* Optimize path helpers.José Valim2012-03-023-12/+64
|
* Merge pull request #5219 from ↵Aaron Patterson2012-02-291-0/+2
|\ | | | | | | | | mattfawcett/2781-fix-querystring-route-constraint-for-testing Fix for #2781
| * Fix the assert_recognizes test method so that it works when there areMatt Fawcett2012-02-241-0/+2
| | | | | | | | constraints on the querystring. Issue #2781
* | Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-02-251-1/+1
|\ \ | | | | | | | | | | | | Conflicts: actionmailer/CHANGELOG.md
| * | Fix typo in match :to docsJustin Woodbridge2012-02-231-1/+1
| | |