aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib/action_dispatch/routing
Commit message (Collapse)AuthorAgeFilesLines
* 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
* Fix typo in routing documentationWen-Tien Chang2012-02-011-1/+1
|
* Merge pull request #3775 from karevn/masterAaron Patterson2012-01-241-1/+2
|\ | | | | Please pull my changes - they fix a rare problem with tests framework
| * Fix: when using subdomains and constraints, request params were not passed ↵karevn2011-11-281-1/+2
| | | | | | | | to constraints callback
* | initialized the @segment_keys instance variableAaron Patterson2012-01-231-1/+4
| |
* | Added custom regexps to ASTs that have literal nodes on either side ofAaron Patterson2012-01-231-1/+20
| | | | | | | | symbol nodes. Fixes #4585
* | 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
| |
* | We should finalize the routes as soon as possible and not wait for the ↵José Valim2012-01-061-2/+0
| | | | | | | | controller to be loaded.
* | Override respond_to? since we are also overriding method_missing.José Valim2012-01-031-0/+4
| |
* | Clean up routes inclusion and add some comments for the next soul that ↵José Valim2012-01-032-41/+49
| | | | | | | | decides to adventure on this code.
* | be explicit about where helpers are installedAaron Patterson2011-12-301-1/+1
| |
* | just add the writer rather than adding both and removing oneAaron Patterson2011-12-291-2/+1
| |
* | stop using __send__ and just module eval in the extensionsAaron Patterson2011-12-291-3/+2
| |
* | we know the classes will be a list, so *tell* it to respond to eachAaron Patterson2011-12-291-3/+3
| | | | | | | | rather than casting
* | don't need the begin / endAaron Patterson2011-12-291-21/+17
| |
* | avoid extra method calls by just defining the delegateAaron Patterson2011-12-291-2/+3
| |
* | don't use instance eval, just reference variables so we don't have toAaron Patterson2011-12-291-10/+11
| | | | | | | | worry about "inspect" marshalling
* | 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
| | |
* | | removing dead code.Aaron Patterson2011-12-291-12/+3
| | |
* | | decouple initialize from clear!. Initialize ivars in initialize, clearAaron Patterson2011-12-291-5/+5
| | | | | | | | | | | | ivars in clear!
* | | modules don't have any instance methodsAaron Patterson2011-12-291-3/+1
| | |