aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_dispatch/routing
Commit message (Collapse)AuthorAgeFilesLines
* 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
| | |
* | | consistently mention first patch, then putXavier Noria2012-02-241-7/+7
| | | | | | | | | | | | | | | | | | There was a mix, sometimes patch first, sometimes put first. Use always patch first, since this is going to be the primary verb for updates.
* | | uses PATCH for the forms of persisted records, and routes PATCH and PUT to ↵Xavier Noria2012-02-241-9/+8
| |/ |/| | | | | the update action of resources
* | Avoid inspecting the whole route set, closes #1525José Valim2012-02-231-0/+6
| |
* | Merge pull request #5084 from johndouthat/patch-1Aaron Patterson2012-02-221-2/+1
|\ \ | | | | | | Remove reference to rails_legacy_mapper, which isn't compatible with 3.2...
| * | Remove reference to rails_legacy_mapper, which isn't compatible with 3.2 #5022johndouthat2012-02-181-2/+1
| |/
* | Fix routes inspection orderRafael Mendonça França2012-02-221-2/+2
| |
* | Add config.default_method_for_update to support PATCHDavid Lee2012-02-221-54/+71
|/ | | | | | | | | | | | | | | | PATCH is the correct HTML verb to map to the #update action. The semantics for PATCH allows for partial updates, whereas PUT requires a complete replacement. Changes: * adds config.default_method_for_update you can set to :patch * optionally use PATCH instead of PUT in resource routes and forms * adds the #patch verb to routes to detect PATCH requests * adds #patch? to Request * changes documentation and comments to indicate support for PATCH This change maintains complete backwards compatibility by keeping :put as the default for config.default_method_for_update.
* Fix GH #4720. Routing problem with nested namespace and already camelized ↵kennyj2012-02-101-0/+5
| | | | controller option.
* Fix match docsRafael Mendonça França2012-02-071-1/+1
|
* Fixed force_ssl redirects to include original query paramsRyan McGeary2012-02-071-0/+1
| | | | | | | | | | `ActionController.force_ssl` redirects http URLs to their https equivalent; however, when a URL contains a query string, the resulting redirect lacked the original query string. Conflicts: actionpack/lib/action_controller/metal/force_ssl.rb
* Fix url_for method's behavior when it is called with :controller option ↵kennyj2012-02-071-1/+1
| | | | | | which starts with "/" from multiple nested controller. Closes #3864
* Note the ways #match may be calledJeremy Kemper2012-02-051-0/+3
|
* Revert "Deprecated multi args to http route methods"Jeremy Kemper2012-02-051-11/+0
| | | | | | | | | | Too painful to lose the compact shorthand form! This reverts commit e848c52535fa0f9488cdbdb3f1cedc7c7c02d643. Conflicts: actionpack/lib/action_dispatch/routing/mapper.rb