aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_dispatch/routing/mapper.rb
Commit message (Collapse)AuthorAgeFilesLines
...
* Remove implicit dependency on pathnameJosé Valim2012-05-251-3/+3
|
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-05-151-11/+0
|\ | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/asset_tag_helper.rb
| * Removing ==Examples and last blank lines of docs from actionpackFrancesco Rodriguez2012-05-151-11/+0
| |
* | add tests and external file backtrace for Routing::Mapper#drawKornelius Kalnbach2012-05-151-3/+4
|/
* Copy literal route constraints to defaults - fixes #3571 and #6224.Andrew White2012-05-111-0/+19
|
* 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
* 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.
* Shadowing variable warning removedArun Agrawal2012-04-261-3/+3
|
* Allow loading external route files from the routerJose and Yehuda2012-04-251-0/+16
| | | | | | | | | 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
* Add missing requires in routesBogdan Gusiev2012-04-231-0/+1
|
* 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
* Optimize path helpers.José Valim2012-03-021-1/+5
|
* 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
* | 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 match docsRafael Mendonça França2012-02-071-1/+1
|
* 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
* Fix typo in routing documentationWen-Tien Chang2012-02-011-1/+1
|
* initialized the @segment_keys instance variableAaron Patterson2012-01-231-1/+4
|
* updated the warning signatureAaron Patterson2012-01-231-1/+1
|
* Deprecated multi args to http route methodsAaron Patterson2012-01-231-0/+11
|
* stop splatting so much. We don't need :star:args everywhereAaron Patterson2012-01-201-7/+6
|
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2011-12-291-0/+7
|\
| * Documented about using :path option for resourcesKevin Moore2011-12-281-0/+7
| |
* | Correctly display rack apps with dynamic constraints in RoutesInspectorPiotr Sarnacki2011-12-261-1/+1
|/ | | | | | | | | | | | | If you used dynamic constraint like that: scope :constraint => MyConstraint.new do mount RackApp => "/foo" end routes were not displayed correctly when using `rake routes`. This commit fixes it. If you want nice display of dynamic constraints in `rake routes` output, please just override to_s method in your constraint's class.
* remove checking for non-empty string before calling to_symVasiliy Ermolovich2011-12-241-2/+1
|
* Do not raise an exception if an invalid route was generated automatically.José Valim2011-12-161-3/+11
|
* Fixing incorrect documentation Aviv Ben-Yosef2011-11-301-2/+2
| | | `path_names` can only be used for affecting `new` and `edit`
* Revert "copy options keys to the right place so that undo will work correctly"Aaron Patterson2011-11-191-4/+10
| | | | This reverts commit 3178cc9a80262d3bf7754f3507ef60243b46634f.
* Warning removed for shadowing variableArun Agrawal2011-11-191-1/+1
|
* copy options keys to the right place so that undo will work correctlyAaron Patterson2011-11-181-10/+4
|
* no need for type checkingAaron Patterson2011-11-181-1/+1
|
* cleaning up variable names to match method parameter namesAaron Patterson2011-11-181-6/+8
|
* stop doing is_a? checks on the resource typeAaron Patterson2011-11-181-4/+4
|
* move constants to methods since nothing else is using themAaron Patterson2011-11-181-6/+5
|
* pushing hash validation upAaron Patterson2011-11-181-13/+14
|
* breaking match down to smaller methodsAaron Patterson2011-11-181-19/+22
|
* some refactoring of the match methodAaron Patterson2011-11-181-25/+17
|