Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Remove more warnings shadowing outer local variable. | Emilio Tagua | 2010-09-27 | 1 | -8/+8 | |
| | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
* | Remove more warnings shadowing outer local variable. | Emilio Tagua | 2010-09-27 | 1 | -8/+8 | |
| | ||||||
* | Remove duplicated test. | Emilio Tagua | 2010-09-27 | 1 | -8/+0 | |
| | ||||||
* | Use parentheses when using assert_match followed by a regexp to avoid warnings. | Emilio Tagua | 2010-09-27 | 4 | -16/+16 | |
| | ||||||
* | Remove deprecated stuff in ActionController | Carlos Antonio da Silva | 2010-09-26 | 9 | -13/+13 | |
| | | | | | | This removes all deprecated classes in ActionController related to Routing, Abstract Request/Response and Integration/IntegrationTest. All tests and docs were changed to ActionDispatch instead of ActionController. | |||||
* | Add ActionDispatch::TestResponse tests. | John Firebaugh | 2010-09-25 | 1 | -0/+21 | |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Raise ArgumentError instead of normalizing controller name when there is a ↵ | Andrew White | 2010-09-18 | 1 | -6/+30 | |
| | | | | leading slash [#5651 state:resolved] | |||||
* | Remove leading slash from controller [#5651 state:resolved] | Andrew White | 2010-09-18 | 1 | -0/+8 | |
| | ||||||
* | Add RouteSet#append | Carl Lerche | 2010-09-17 | 1 | -0/+33 | |
| | | | Allows specifying blocks to the routeset that will get appended after the RouteSet is drawn. | |||||
* | Only send secure cookies over SSL. | W. Andrew Loe III | 2010-09-13 | 2 | -0/+28 | |
| | ||||||
* | added block arguments to ActionController::Metal#use | Collin Miller | 2010-09-12 | 1 | -1/+16 | |
| | | | | | | | | | | | Useful for cases such as warden, where a block configuration is taken. class SomeController < ApplicationController use RailsWarden::Manager do |manager| manager.default_strategies :facebook_oauth manager.failure_app = SomeController.action(:authorize) end end | |||||
* | Merge remote branch 'drogus/engines' | wycats | 2010-09-11 | 1 | -4/+4 | |
|\ | ||||||
| * | Change app to main_app in mounted_helpers | Piotr Sarnacki | 2010-09-08 | 1 | -3/+3 | |
| | | ||||||
| * | Do not require passing :app to mounted helpers, it's actually useless and ↵ | Piotr Sarnacki | 2010-09-08 | 1 | -1/+1 | |
| | | | | | | | | not DRY | |||||
* | | Add configuration option for tld length | Simon Jefford | 2010-09-09 | 1 | -0/+5 | |
|/ | ||||||
* | Removed deprecated RouteSet API, still many tests fail | Piotr Sarnacki | 2010-09-05 | 7 | -7/+7 | |
| | ||||||
* | Implemented RouteSet#default_scope, which allows to set the scope for the ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -0/+29 | |
| | | | | entire routes object | |||||
* | Modified ActionDispatch::Static to allow passing multiple roots | Piotr Sarnacki | 2010-09-03 | 1 | -11/+50 | |
| | ||||||
* | Ensure that url_helpers included after application's ones have higher priority | Piotr Sarnacki | 2010-09-03 | 1 | -0/+15 | |
| | ||||||
* | Add mounted_helpers to routes | Piotr Sarnacki | 2010-09-03 | 2 | -66/+70 | |
| | | | | | | | | | | | | | | | | mounted_helpers are a bit similar to url_helpers. They're automatically included in controllers for Rails.application and each of mounted Engines. Mounted helper allows to call url_for and named helpers for given application. Given Blog::Engine mounted as blog_engine, there are 2 helpers defined: app and blog_engine. You can call routes for app and engine using those helpers: app.root_url app.url_for(:controller => "foo") blog_engine.posts_path blog_engine.url_for(@post) | |||||
* | Routes refactoring: | Piotr Sarnacki | 2010-09-03 | 1 | -46/+71 | |
| | | | | | | | * added more tests for prefix generation * fixed bug with generating host for both prefix and url * refactored url_for method * organized tests for prefix generation | |||||
* | Use new url_for API instead of including routes.url_helpers | Piotr Sarnacki | 2010-09-03 | 1 | -3/+2 | |
| | ||||||
* | Extended url_for to handle specifying which router should be used. | Piotr Sarnacki | 2010-09-03 | 2 | -0/+53 | |
| | | | | | | | A few examples: url_for Blog::Engine, :posts_path url_for Blog::Engine, @post url_for Blog::Engine, :action => "main", :controller => "index" | |||||
* | Added some more tests for url generation between Engine and Application | Piotr Sarnacki | 2010-09-03 | 1 | -10/+28 | |
| | ||||||
* | Get rid of :skip_prefix options in routes | Piotr Sarnacki | 2010-09-03 | 1 | -1/+1 | |
| | ||||||
* | Refactored tests for prefix generation and added test for url generation in ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -11/+32 | |
| | | | | regular class with default_url_options[:script_name] set | |||||
* | New way of generating urls for Application from Engine. | Piotr Sarnacki | 2010-09-03 | 1 | -0/+1 | |
| | | | | | | | | | It's based specifying application's script_name with: Rails.application.default_url_options = {:script_name => "/foo"} default_url_options method is delegated to routes. If router used to generate url differs from the router passed via env it always overwrites :script_name with this value. | |||||
* | Allow to generate Application routes inside Engine | Piotr Sarnacki | 2010-09-03 | 1 | -3/+22 | |
| | | | | | | This requires knowledge about original SCRIPT_NAME and the parent router. It should be pass through the env as ORIGIAL_SCRIPT_NAME and action_dispatch.parent_routes | |||||
* | Use env['action_dispatch.routes'] to determine if we should generate prefix ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -0/+102 | |
| | | | | | | | | | | or not. This technique is here to allow using routes from Engine in Application and vice versa. When using Engine routes inside Application it should generate prefix based on mount point. When using Engine routes inside Engine it should use env['SCRIPT_NAME']. In any other case it should generate prefix as env should not be even available. | |||||
* | Revert "Setup explicit requires for files with exceptions. Removed them from ↵ | José Valim | 2010-09-02 | 3 | -3/+0 | |
| | | | | | | | | autoloading." Booting a new Rails application does not work after this commit [#5359 state:open] This reverts commit 38a421b34d0b414564e919f67d339fac067a56e6. | |||||
* | Setup explicit requires for files with exceptions. Removed them from ↵ | Łukasz Strzałkowski | 2010-09-02 | 3 | -0/+3 | |
| | | | | | | autoloading. Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Move implicit nested call before options handling so that nested constraints ↵ | Andrew White | 2010-09-01 | 1 | -0/+18 | |
| | | | | | | work [#5513 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Bump rack-mount to 0.6.13 and add test case for named character classes ↵ | Andrew White | 2010-08-31 | 1 | -0/+12 | |
| | | | | | | [#5509 state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
* | Use nested scope for routes defined at the :resources scope level (as in ↵ | Andrew White | 2010-08-24 | 1 | -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é Valim | 2010-08-24 | 1 | -1/+1 | |
| | ||||||
* | Finally fix the bug where symbols and strings were not having the same ↵ | José Valim | 2010-08-24 | 1 | -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é Valim | 2010-08-24 | 1 | -4/+4 | |
| | ||||||
* | Fix how routes inside namespaces are generated. | José Valim | 2010-08-24 | 1 | -3/+3 | |
| | ||||||
* | Allow actions which start with A-Za-z_ only | Santiago Pastorino | 2010-08-24 | 1 | -1/+4 | |
| | ||||||
* | Fix a bug where symbols and strings were not behaving the same in the ↵ | José Valim | 2010-08-24 | 1 | -3/+3 | |
| | | | | router. [#5431 state:resolved] | |||||
* | When custom resource actions are specified using strings add the default ↵ | Andrew White | 2010-08-24 | 1 | -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 White | 2010-08-22 | 1 | -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 White | 2010-08-20 | 2 | -0/+54 | |
| | | | | | | | | 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 White | 2010-08-20 | 1 | -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 White | 2010-08-19 | 1 | -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 White | 2010-08-19 | 1 | -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ín | 2010-08-18 | 1 | -0/+62 | |
| | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
* | Allow member actions (get, etc) to accept strings, with test | Andre Arko | 2010-08-17 | 1 | -1/+1 | |
| | ||||||
* | Allow for any possible TLD when using the :all option with the cookie ↵ | Bryce Thornton | 2010-08-15 | 1 | -1/+29 | |
| | | | | | | session store. This works for subdomain.mysite.local, google.co.uk, google.com.au, etc. [#5147 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵ | Santiago Pastorino | 2010-08-14 | 5 | -8/+8 | |
| | | | | 's/[ \t]*$//' -i {} \;) |