aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/dispatch/routing_test.rb
Commit message (Collapse)AuthorAgeFilesLines
* Use nested scope for routes defined at the :resources scope level (as in ↵Andrew White2010-08-241-0/+7
| | | | | | Rails 2.3) Signed-off-by: José Valim <jose.valim@gmail.com>
* Allow format to be skipped. This is used internally by mount.José Valim2010-08-241-1/+1
|
* Finally fix the bug where symbols and strings were not having the same ↵José Valim2010-08-241-6/+38
| | | | | | | behavior in the router. If you were using symbols before for methods like match/get/post/put/delete, it is likely that this commit will break your routes. Everything should behave the same if you are using strings, if not, please open up a ticket.
* Ensure shortcuts inside resources also generates helpers.José Valim2010-08-241-4/+4
|
* Fix how routes inside namespaces are generated.José Valim2010-08-241-3/+3
|
* Allow actions which start with A-Za-z_ onlySantiago Pastorino2010-08-241-1/+4
|
* Fix a bug where symbols and strings were not behaving the same in the ↵José Valim2010-08-241-3/+3
| | | | router. [#5431 state:resolved]
* When custom resource actions are specified using strings add the default ↵Andrew White2010-08-241-0/+22
| | | | | | | | name and action if the string is a valid ruby method name. [#5431 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Cache the symbolized path parameters using a instance variable in the ↵Andrew White2010-08-221-0/+21
| | | | | | | | | | | request object rather than the environment hash. This it to prevent stale parameters in later routing constraints/redirects as only the normal path parameters are set by Rack::Mount. Also if a constraint proc arity is more than one, pass the symbolized path parameters as the first argument to match redirect proc args and provide easier access. [#5157 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Don't add the standard https port when using redirect in routes.rb and ↵Andrew White2010-08-201-0/+18
| | | | | | | | ensure that request.scheme returns https when using a reverse proxy. [#5408 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Allow symbols for :path resource(s) option [#5306 state:resolved]Andrew White2010-08-201-0/+19
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Move regexps in options hash to :constraints hash so that they are pushed ↵Andrew White2010-08-191-0/+22
| | | | | | into the scope [#5208 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Move edit route before show route so that it will have precedence if the :id ↵Andrew White2010-08-191-0/+16
| | | | | | parameter allows slashes [#5409 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Don't pluralize resource methods [#4704 state:resolved]Javier Martín2010-08-181-0/+62
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Allow member actions (get, etc) to accept strings, with testAndre Arko2010-08-171-1/+1
|
* adds URL to the body generated by the redirect macro in the routes mapper as ↵Xavier Noria2010-08-091-30/+25
| | | | per the RFC, extracts common test pattern into a test macro, adds a test to cover the :status option
* Routes should respect namespace even if action is a regexp and just ↵José Valim2010-07-121-3/+15
| | | | controller is given to to.
* When a dynamic :controller segment is present in the path add a Regexp ↵Andrew White2010-07-071-11/+25
| | | | | | | | | | | constraint that allow matching on multiple path segments. Using a namespace block isn't compatible with dynamic routes so we raise an ArgumentError if we detect a :module present in the scope. [#5052 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Refactor handling of :only and :except options. The rules are:Andrew White2010-07-061-47/+190
| | | | | | | | | | | 1. Don't inherit when specified as an option on a resource 2. Don't push into scope when specified as an option on a resource 2. Resources pull in :only or :except options from scope 3. Either :only or :except in nested scope overwrites parent scope [#5048 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Recognize should also work with route is wrapped in a constraint.José Valim2010-07-061-1/+4
|
* Fix syntax of routing tests so they actually runAndrew White2010-07-041-12/+18
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Refactor resource options and scoping. Resource classes are now only ↵Andrew White2010-07-041-2/+81
| | | | | | responsible for controlling how they are named. All other options passed to resources are pushed out to the scope. Signed-off-by: José Valim <jose.valim@gmail.com>
* Fix routes with :controller segment when namespaced [#5034 state:resolved]José Valim2010-07-021-5/+19
|
* A couple enhancements to the router:José Valim2010-06-301-0/+27
| | | | | | | | * Allow to use the get :symbol shortcut outside resources scopes as well; * Fix a bug where :action was not being picked from given options; * Giving :as option inside a resource now changes just the relative name instead of the full name;
* Merge :constraints from scope into resource options [#2694 state:resolved]Andrew White2010-06-281-0/+43
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Remove invalid conditions from route [#4989 state:resolved]Andrew White2010-06-281-0/+12
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Support optional static segements as well [#4832 state:resolved]Andrew White2010-06-261-0/+34
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Fixed normalize_path in Routing::Mapper to handle optional prefix segments ↵Paul Barry2010-06-261-0/+46
| | | | | | with static and dynamic parts Signed-off-by: José Valim <jose.valim@gmail.com>
* Don't force pluralization of controller name when defining a resource [#4980 ↵Andrew White2010-06-261-0/+9
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Failing test for whether glob parameters accept regexpsAndrew White2010-06-251-0/+9
| | | | | | [#4970 state:committed] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Add failing test case for parameters with periodsAndrew White2010-06-251-0/+27
| | | | | | [#2536 state:resolved] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Fix namespaced redirects [#4389 state:resolved]Andrew White2010-06-241-0/+14
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Add the :path option to match routes when given as symbols. This is ↵Carlos Antonio da Silva2010-06-231-1/+20
| | | | | | | | | | specially useful in http helpers for generating routes in scenarios like: resources :users, :path => 'usuarios' do get :search, :on => :collection, :path => 'pesquisar' end Signed-off-by: José Valim <jose.valim@gmail.com>
* Provive resources_path_names helpers in the router.José Valim2010-06-231-0/+11
|
* Allow namespace accept options in routes [#4936 state:resolved]Carlos Antonio da Silva2010-06-221-0/+12
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Deprecate :name_prefix in the new router DSL.José Valim2010-06-221-3/+3
|
* Don't use module to work out shallow name prefix and path as it may not ↵Andrew White2010-06-201-1/+16
| | | | | | accurately reflect the actual namespace [#4899 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Accept an object for :constraints option [#4904 state:resolved]Andrew White2010-06-201-0/+8
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Custom resource routes should be scopedAndrew White2010-06-191-0/+47
| | | | | | [#3765] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* fix for :shallow in router not generating helpers for create, update, and ↵Josh Kalderimis2010-06-181-1/+39
| | | | | | | | destroy actions when :only or :except are used [#4900 state:committed] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Add shallow routing option to new router [#3765 status:committed]Andrew White2010-06-171-0/+125
| | | | Signed-off-by: David Heinemeier Hansson <david@loudthinking.com>
* Add more cases to previous commit [#4394 state:resolved]José Valim2010-06-081-2/+9
|
* Make sure namespaces are nested within resourcesJan De Poorter2010-06-081-0/+15
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Revert "Add shallow routes to the new router" for now. Needs more work.David Heinemeier Hansson2010-06-081-39/+0
| | | | This reverts commit 67a60ee314f53abcde78f8ecd2a1f7c9ef8264e1.
* Fix resources ignoring scope optionsAndrew White2010-06-081-4/+30
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Add support for actions on a new resource to the new routing DSL [#4328 ↵Andrew White2010-06-081-2/+48
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Add shallow routes to the new router [Closes #3765]Diego Carrion2010-06-071-0/+39
|
* member on resource should not expect an ID.José Valim2010-06-071-3/+2
|
* Router accepts member routes on resource. [#4624 state:resolved]Rizwan Reza2010-06-061-0/+13
|
* Allow root to be given in the resources scope without need to specify :on => ↵José Valim2010-05-171-1/+11
| | | | collection.