Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | This test is invalid for new router | Łukasz Strzałkowski | 2010-09-05 | 1 | -20/+0 |
| | |||||
* | Removed deprecated_mapper - we don't need it anymore | Łukasz Strzałkowski | 2010-09-05 | 1 | -525/+0 |
| | |||||
* | Removed tests for setting default value of *path in route | Łukasz Strzałkowski | 2010-09-05 | 1 | -25/+0 |
| | | | | If we want to have this - we have to change Rack::Mount source | ||||
* | Made test_generate pass | Łukasz Strzałkowski | 2010-09-05 | 1 | -9/+1 |
| | | | | I've removed assertions with @routes.generate(:use_route => ...). I'm not 100% sure if not supporting :use_route in new router is intentional or it should rather be supported and backported from 2.3.x. | ||||
* | If it's unused there's no reason to keep it commented. It will always remain ↵ | Łukasz Strzałkowski | 2010-09-05 | 1 | -11/+0 |
| | | | | in git history, so removing it. | ||||
* | Fix action mailer tests after old mapper removal | Piotr Sarnacki | 2010-09-05 | 1 | -5/+3 |
| | |||||
* | 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 | ||||
* | Fixed routes to use new API in a few more actionpack tests | Piotr Sarnacki | 2010-09-05 | 5 | -18/+56 |
| | |||||
* | Removed rest of errors and failures in routing_test.rb. Almost all squashed, ↵ | Łukasz Strzałkowski | 2010-09-05 | 1 | -92/+79 |
| | | | | only few left | ||||
* | raise error on invalid HTTP methods or :head passed with :via in routes | Piotr Sarnacki | 2010-09-05 | 2 | -1/+10 |
| | |||||
* | Implemented resources :foos, :except => :all option | Piotr Sarnacki | 2010-09-05 | 2 | -5/+20 |
| | |||||
* | Removed deprecated router API from railties | Piotr Sarnacki | 2010-09-05 | 6 | -25/+14 |
| | |||||
* | Test for recognizing routes with http method set | Łukasz Strzałkowski | 2010-09-05 | 1 | -1/+40 |
| | |||||
* | Fixed about half of broken tests in routing_test | Łukasz Strzałkowski | 2010-09-05 | 1 | -125/+82 |
| | |||||
* | Fixed almost all resources tests | Piotr Sarnacki | 2010-09-05 | 1 | -159/+213 |
| | |||||
* | Fixed url_for test | Łukasz Strzałkowski | 2010-09-05 | 1 | -1/+1 |
| | | | | Added route which matches all: controller, action, id and format | ||||
* | Revert "Extended default route for match in tests - now it matches ↵ | Łukasz Strzałkowski | 2010-09-05 | 1 | -2/+2 |
| | | | | | | controller, action, id and format if given." This reverts commit 05e9d0df0dea68ca4655aa8723c3ee69049fac78. | ||||
* | Extended default route for match in tests - now it matches controller, ↵ | Łukasz Strzałkowski | 2010-09-05 | 1 | -2/+2 |
| | | | | | | action, id and format if given. This fixes url_for tests | ||||
* | Removed deprecated RouteSet API, still many tests fail | Piotr Sarnacki | 2010-09-05 | 34 | -606/+567 |
| | |||||
* | order should always be concatenated. | Neeraj Singh | 2010-09-05 | 6 | -8/+35 |
| | | | | | | | | | | | | | | | | | | | | | | order that is declared first has highest priority in all cases. Here are some examples. Car.order('name desc').find(:first, :order => 'id').name Car.named_scope_with_order.named_scope_with_another_order Car.order('id DESC').scoping do Car.find(:first, :order => 'id asc') end No special treatment to with_scope or scoping. Also note that if default_scope declares an order then the order declared in default_scope has the highest priority unless with_exclusive_scope is used. Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | Remove more deprecated behavior from plugins. | José Valim | 2010-09-04 | 2 | -46/+3 |
| | |||||
* | Small tidy up on Rails::Engine. | José Valim | 2010-09-04 | 1 | -48/+49 |
| | |||||
* | This commit merges most of the work done by Piotr Sarnacki in his Ruby ↵ | José Valim | 2010-09-04 | 75 | -468/+2479 |
|\ | | | | | | | | | | | | | | | Summer of Code project. His work brings several capabilities from app to engines, as routes, middleware stack, asset handling and much more. Please check Rails::Engine documentation for more refenrences. Merge remote branch 'drogus/engines' | ||||
| * | 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 |
| | | |||||
| * | Forgot to move that line to railtie on rebase | Piotr Sarnacki | 2010-09-03 | 1 | -1/+1 |
| | | |||||
| * | Fixed tests after rebase | Piotr Sarnacki | 2010-09-03 | 1 | -4/+3 |
| | | |||||
| * | I've changed that test along the way, it should actually stay without changes | Piotr Sarnacki | 2010-09-03 | 2 | -8/+15 |
| | | |||||
| * | Use default_scope in isolated Engines to not force user to scope his routes | Piotr Sarnacki | 2010-09-03 | 3 | -16/+12 |
| | | |||||
| * | Implemented RouteSet#default_scope, which allows to set the scope for the ↵ | Piotr Sarnacki | 2010-09-03 | 3 | -2/+41 |
| | | | | | | | | entire routes object | ||||
| * | engines_blank_point should always be the last initializer in Engine | Piotr Sarnacki | 2010-09-03 | 1 | -5/+5 |
| | | |||||
| * | We should avoid creating additional initializers when we can, adding them ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -3/+1 |
| | | | | | | | | makes boot process slower | ||||
| * | Add some documantation on new route_key and param_key in ActiveModel::Naming | Piotr Sarnacki | 2010-09-03 | 1 | -0/+16 |
| | | |||||
| * | Remove namespace for isolated namespaced models in forms | Piotr Sarnacki | 2010-09-03 | 4 | -6/+101 |
| | | |||||
| * | Use new ActiveModel::Naming.route_key in polymorphic_routes | Piotr Sarnacki | 2010-09-03 | 2 | -12/+3 |
| | | |||||
| * | Prepared ActiveModel::Naming to handle cases for namespaced isolated engines | Piotr Sarnacki | 2010-09-03 | 3 | -4/+134 |
| | | |||||
| * | Add some more docs on polymorphic_url with routes proxy | Piotr Sarnacki | 2010-09-03 | 1 | -0/+10 |
| | | |||||
| * | 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 | 3 | -1/+58 |
| | | | | | | | | polymorphic_routes, for example: polymorphic_url([blog, @post]) | ||||
| * | Updated tests to use scope(:module => :engine_name) instead of namespace and ↵ | Piotr Sarnacki | 2010-09-03 | 2 | -23/+28 |
| | | | | | | | | updated mounted engine tests to actually use the namespacing | ||||
| * | Move RoutesProxy to separate file | Piotr Sarnacki | 2010-09-03 | 3 | -32/+36 |
| | | |||||
| * | Include all helpers from non-namespaced engines | Piotr Sarnacki | 2010-09-03 | 4 | -12/+38 |
| | | |||||
| * | Set only helpers_path on inherited hook in action_controller/railtie.rb and ↵ | Piotr Sarnacki | 2010-09-03 | 5 | -38/+17 |
| | | | | | | | | use helper(:all) just after that | ||||
| * | Added more tests for polymorphic_url with namespaced models and implemented ↵ | Piotr Sarnacki | 2010-09-03 | 2 | -5/+66 |
| | | | | | | | | 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 | ||||
| * | Moved ActionMailer and ActionController railties options to inherited hook | Piotr Sarnacki | 2010-09-03 | 5 | -22/+61 |
| | | | | | | | | | | | | | | | | | | | | | | This change is needed, because we must take namespace into account and if controller's/mailer's class is namespaced, engine's paths should be set instead of application's ones. The nice side effect of this is removing unneeded logic in ActionController::Base.inherited - now the helpers_path should be set correctly even for engine's controllers, so helper(:all) will always include correct helpers. | ||||
| * | Modified polymorphic_url to check for model's namespace | Piotr Sarnacki | 2010-09-03 | 2 | -4/+34 |
| | | | | | | | | | | | | | | | | | | | | | | 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. | ||||
| * | Get rid of static_paths method and instead configure paths for ↵ | Piotr Sarnacki | 2010-09-03 | 6 | -17/+29 |
| | | | | | | | | ActionDispatch::Static in initializers | ||||
| * | We don't need to clear action_methods on inherited hook as they are cleaned ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -5/+0 |
| | | | | | | | | on method_added hook | ||||
| * | ActionMailer should not depend on ActionDispatch | Piotr Sarnacki | 2010-09-03 | 3 | -3/+2 |
| | |