aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Deleted test with extended regexp - it fails with no reason, probably ↵Piotr Sarnacki2010-09-051-21/+0
| | | | rack-mount is doing something weird with that regexp
* Fixed 2 broken tests for router.Łukasz Strzałkowski2010-09-051-3/+3
| | | | They were broken after rebase/merge
* This test is invalid for new routerŁukasz Strzałkowski2010-09-051-20/+0
|
* Removed deprecated_mapper - we don't need it anymoreŁukasz Strzałkowski2010-09-051-525/+0
|
* Removed tests for setting default value of *path in routeŁukasz Strzałkowski2010-09-051-25/+0
| | | | If we want to have this - we have to change Rack::Mount source
* Made test_generate passŁukasz Strzałkowski2010-09-051-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łkowski2010-09-051-11/+0
| | | | in git history, so removing it.
* Fix action mailer tests after old mapper removalPiotr Sarnacki2010-09-051-5/+3
|
* Ported missing functionality from Rails 2.3.x, raise error on wrong regexps ↵Piotr Sarnacki2010-09-051-0/+9
| | | | in :constraints in routes
* Fixed routes to use new API in a few more actionpack testsPiotr Sarnacki2010-09-055-18/+56
|
* Removed rest of errors and failures in routing_test.rb. Almost all squashed, ↵Łukasz Strzałkowski2010-09-051-92/+79
| | | | only few left
* raise error on invalid HTTP methods or :head passed with :via in routesPiotr Sarnacki2010-09-052-1/+10
|
* Implemented resources :foos, :except => :all optionPiotr Sarnacki2010-09-052-5/+20
|
* Removed deprecated router API from railtiesPiotr Sarnacki2010-09-056-25/+14
|
* Test for recognizing routes with http method setŁukasz Strzałkowski2010-09-051-1/+40
|
* Fixed about half of broken tests in routing_testŁukasz Strzałkowski2010-09-051-125/+82
|
* Fixed almost all resources testsPiotr Sarnacki2010-09-051-159/+213
|
* Fixed url_for testŁukasz Strzałkowski2010-09-051-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łkowski2010-09-051-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łkowski2010-09-051-2/+2
| | | | | | action, id and format if given. This fixes url_for tests
* Removed deprecated RouteSet API, still many tests failPiotr Sarnacki2010-09-0534-606/+567
|
* order should always be concatenated.Neeraj Singh2010-09-056-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é Valim2010-09-042-46/+3
|
* Small tidy up on Rails::Engine.José Valim2010-09-041-48/+49
|
* This commit merges most of the work done by Piotr Sarnacki in his Ruby ↵José Valim2010-09-0475-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 Sarnacki2010-09-041-4/+8
| | | | | | | | load more ActiveSupport than we need
| * Optimize ActionDispatch::StaticPiotr Sarnacki2010-09-041-51/+35
| |
| * Forgot to move that line to railtie on rebasePiotr Sarnacki2010-09-031-1/+1
| |
| * Fixed tests after rebasePiotr Sarnacki2010-09-031-4/+3
| |
| * I've changed that test along the way, it should actually stay without changesPiotr Sarnacki2010-09-032-8/+15
| |
| * Use default_scope in isolated Engines to not force user to scope his routesPiotr Sarnacki2010-09-033-16/+12
| |
| * Implemented RouteSet#default_scope, which allows to set the scope for the ↵Piotr Sarnacki2010-09-033-2/+41
| | | | | | | | entire routes object
| * engines_blank_point should always be the last initializer in EnginePiotr Sarnacki2010-09-031-5/+5
| |
| * We should avoid creating additional initializers when we can, adding them ↵Piotr Sarnacki2010-09-031-3/+1
| | | | | | | | makes boot process slower
| * Add some documantation on new route_key and param_key in ActiveModel::NamingPiotr Sarnacki2010-09-031-0/+16
| |
| * Remove namespace for isolated namespaced models in formsPiotr Sarnacki2010-09-034-6/+101
| |
| * Use new ActiveModel::Naming.route_key in polymorphic_routesPiotr Sarnacki2010-09-032-12/+3
| |
| * Prepared ActiveModel::Naming to handle cases for namespaced isolated enginesPiotr Sarnacki2010-09-033-4/+134
| |
| * Add some more docs on polymorphic_url with routes proxyPiotr Sarnacki2010-09-031-0/+10
| |
| * Add short note on using url_for instead of directly calling named route in ↵Piotr Sarnacki2010-09-031-0/+3
| | | | | | | | polymorphic_url
| * Add possibility to explicitly call engine's routes through ↵Piotr Sarnacki2010-09-033-1/+58
| | | | | | | | polymorphic_routes, for example: polymorphic_url([blog, @post])
| * Updated tests to use scope(:module => :engine_name) instead of namespace and ↵Piotr Sarnacki2010-09-032-23/+28
| | | | | | | | updated mounted engine tests to actually use the namespacing
| * Move RoutesProxy to separate filePiotr Sarnacki2010-09-033-32/+36
| |
| * Include all helpers from non-namespaced enginesPiotr Sarnacki2010-09-034-12/+38
| |
| * Set only helpers_path on inherited hook in action_controller/railtie.rb and ↵Piotr Sarnacki2010-09-035-38/+17
| | | | | | | | use helper(:all) just after that
| * Added more tests for polymorphic_url with namespaced models and implemented ↵Piotr Sarnacki2010-09-032-5/+66
| | | | | | | | missing use cases
| * This was used only to clear warning in ActionMailer tests, it shouldn't be ↵Piotr Sarnacki2010-09-031-5/+0
| | | | | | | | done like that
| * Moved ActionMailer and ActionController railties options to inherited hookPiotr Sarnacki2010-09-035-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 namespacePiotr Sarnacki2010-09-032-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 Sarnacki2010-09-036-17/+29
| | | | | | | | ActionDispatch::Static in initializers