Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Remove deprecated stuff in ActionController | Carlos Antonio da Silva | 2010-09-26 | 1 | -2/+2 |
| | | | | | | 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. | ||||
* | Don't duplicate Rack::Response functionality. [#5320 state:resolved] | John Firebaugh | 2010-09-25 | 1 | -18/+4 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Improve performance of applications using file uploads by not busting the ↵ | Carl Lerche | 2010-09-22 | 1 | -29/+20 |
| | | | | method cache on every request containing a file upload. | ||||
* | fixing bug with rails use of rack-test | Aaron Patterson | 2010-09-22 | 1 | -2/+4 |
| | |||||
* | Refactor routing methods. | Emilio Tagua | 2010-09-22 | 2 | -11/+2 |
| | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | Raise ArgumentError instead of normalizing controller name when there is a ↵ | Andrew White | 2010-09-18 | 1 | -1/+5 |
| | | | | leading slash [#5651 state:resolved] | ||||
* | Remove leading slash from controller [#5651 state:resolved] | Andrew White | 2010-09-18 | 1 | -1/+1 |
| | |||||
* | Add RouteSet#append | Carl Lerche | 2010-09-17 | 1 | -4/+12 |
| | | | Allows specifying blocks to the routeset that will get appended after the RouteSet is drawn. | ||||
* | First pass at Rack::Cache | wycats | 2010-09-13 | 2 | -0/+68 |
| | |||||
* | Only send secure cookies over SSL. | W. Andrew Loe III | 2010-09-13 | 1 | -1/+4 |
| | |||||
* | removes /i from the TRUSTED_PROXIES regexp, adds /x and comments for ↵ | Xavier Noria | 2010-09-12 | 1 | -2/+10 |
| | | | | readability, adds a pointer to a Wikipedia section that documents the matched IPs | ||||
* | gets rid of a double negation, no need to force exactly true/false in a ↵ | Xavier Noria | 2010-09-12 | 1 | -4/+4 |
| | | | | predicate | ||||
* | Partial revert of commit #7fae0aa4ac116 removing ↵ | Mikel Lindsaar | 2010-09-11 | 1 | -5/+0 |
| | | | | | | | action_dispatch.prepare_dispatcher per commit #48bf667a8b1 See line note: http://github.com/rails/rails/commit/7fae0aa4ac116bcbf522a9a4f19972c653d366fa#commitcomment-145148 | ||||
* | Merge remote branch 'drogus/engines' | wycats | 2010-09-11 | 1 | -1/+1 |
|\ | |||||
| * | Change app to main_app in mounted_helpers | Piotr Sarnacki | 2010-09-08 | 1 | -1/+1 |
| | | |||||
| * | 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 | 2 | -4/+16 |
|/ | |||||
* | Cleanup deprecations in Action Dispatch | Carlos Antonio da Silva | 2010-09-06 | 3 | -106/+1 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Remove a few tests from old router that do not make sense with the new one. | José Valim | 2010-09-05 | 1 | -21/+3 |
| | |||||
* | Removed deprecated_mapper - we don't need it anymore | Łukasz Strzałkowski | 2010-09-05 | 1 | -525/+0 |
| | |||||
* | Ported missing functionality from Rails 2.3.x, raise error on wrong regexps ↵ | Piotr Sarnacki | 2010-09-05 | 1 | -0/+9 |
| | | | | in :constraints in routes | ||||
* | raise error on invalid HTTP methods or :head passed with :via in routes | Piotr Sarnacki | 2010-09-05 | 1 | -0/+9 |
| | |||||
* | Implemented resources :foos, :except => :all option | Piotr Sarnacki | 2010-09-05 | 1 | -2/+11 |
| | |||||
* | Removed deprecated RouteSet API, still many tests fail | Piotr Sarnacki | 2010-09-05 | 3 | -16/+4 |
| | |||||
* | Do not use ActionController::Base.page_cache_extension in initialize to not ↵ | Piotr Sarnacki | 2010-09-04 | 1 | -4/+8 |
| | | | | load more ActiveSupport than we need | ||||
* | Optimize ActionDispatch::Static | Piotr Sarnacki | 2010-09-04 | 1 | -51/+35 |
| | |||||
* | Implemented RouteSet#default_scope, which allows to set the scope for the ↵ | Piotr Sarnacki | 2010-09-03 | 2 | -2/+12 |
| | | | | entire routes object | ||||
* | Use new ActiveModel::Naming.route_key in polymorphic_routes | Piotr Sarnacki | 2010-09-03 | 1 | -11/+2 |
| | |||||
* | Add short note on using url_for instead of directly calling named route in ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -0/+3 |
| | | | | polymorphic_url | ||||
* | Add possibility to explicitly call engine's routes through ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -1/+20 |
| | | | | polymorphic_routes, for example: polymorphic_url([blog, @post]) | ||||
* | Move RoutesProxy to separate file | Piotr Sarnacki | 2010-09-03 | 3 | -32/+36 |
| | |||||
* | Added more tests for polymorphic_url with namespaced models and implemented ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -5/+10 |
| | | | | missing use cases | ||||
* | This was used only to clear warning in ActionMailer tests, it shouldn't be ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -5/+0 |
| | | | | done like that | ||||
* | Modified polymorphic_url to check for model's namespace | Piotr Sarnacki | 2010-09-03 | 1 | -0/+4 |
| | | | | | | | | | | | This change allows using namespaced models with polymorphic_url, in the way that you would use them without namespace. Let's say that you have Blog::Post model in namespaced Engine. When you use polymorphic_path with Blog::Post instances, like in form_for(@post), it will look for blog_posts_path named url helper. As we are inside Blog::Engine, it's annoying to always use the prefix. With this commit, blog_ prefix will be removed and posts_path will be called. | ||||
* | Refactor RoutesProxy to avoid using _with_routes in helpers | Piotr Sarnacki | 2010-09-03 | 2 | -1/+5 |
| | |||||
* | Refactor ActionMailer to not use hide_actions | Piotr Sarnacki | 2010-09-03 | 1 | -1/+6 |
| | |||||
* | Fix generating urls with mounted helpers in view context | Piotr Sarnacki | 2010-09-03 | 1 | -1/+1 |
| | | | | | | | | | | There were actually 2 problems with this one: * script_name was added to options as a string and then it was used in RouteSet#url_for with usage of <<, which was changing the original script_name * the second issue was with _with_routes method. It was called in RoutesProxy to modify _routes in view_context, but url_helpers in views is just delegating it to controller, so another _with_routes call is needed there | ||||
* | For view_context we need to initialize RoutesProxy in context of controller, ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -2/+4 |
| | | | | not view, quick fix, I need to dig into it later | ||||
* | Modified ActionDispatch::Static to allow passing multiple roots | Piotr Sarnacki | 2010-09-03 | 1 | -10/+50 |
| | |||||
* | Add Rails::Railtie.railtie_name method to allow setting custom name for railtie | Piotr Sarnacki | 2010-09-03 | 1 | -2/+7 |
| | |||||
* | We don't need delegating polymorphic_url and polymorphic_path anymore | Piotr Sarnacki | 2010-09-03 | 1 | -10/+6 |
| | |||||
* | Added ability to set asset_path for engines | Piotr Sarnacki | 2010-09-03 | 1 | -1/+1 |
| | |||||
* | Add mounted_helpers to routes | Piotr Sarnacki | 2010-09-03 | 3 | -23/+69 |
| | | | | | | | | | | | | | | | | 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 | 3 | -9/+6 |
| | | | | | | | * 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 | ||||
* | Extended url_for to handle specifying which router should be used. | Piotr Sarnacki | 2010-09-03 | 3 | -29/+42 |
| | | | | | | | A few examples: url_for Blog::Engine, :posts_path url_for Blog::Engine, @post url_for Blog::Engine, :action => "main", :controller => "index" | ||||
* | Get rid of :skip_prefix options in routes | Piotr Sarnacki | 2010-09-03 | 1 | -6/+3 |
| | |||||
* | New way of generating urls for Application from Engine. | Piotr Sarnacki | 2010-09-03 | 2 | -7/+12 |
| | | | | | | | | | 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 | 2 | -4/+4 |
| | | | | | | 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 | 3 | -7/+48 |
| | | | | | | | | | | 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. | ||||
* | require 'active_support/dependencies' in action_dispatch/middleware/stack | Piotr Sarnacki | 2010-09-03 | 1 | -0/+1 |
| |