aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
...
* Cleanup Action Dispatch assertions testsCarlos Antonio da Silva2010-09-061-108/+37
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Cleanup deprecations in Action DispatchCarlos Antonio da Silva2010-09-066-135/+8
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Cleanup deprecation warnings in Action ControllerCarlos Antonio da Silva2010-09-067-119/+2
| | | | 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é Valim2010-09-053-88/+6
|
* Merge remote branch 'drogus/remove_deprecated_routes'José Valim2010-09-0539-1402/+861
|\ | | | | | | This merge removes the deprecated routes mapper from Rails and update its tests.
| * 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.
| * 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
| |
| * 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
| |
* | Make number_to_human and number_with_precision work with negativesSantiago Pastorino2010-09-052-2/+4
| |
* | Fix indentation errorsSantiago Pastorino2010-09-051-47/+47
| |
* | Fix code style a bitSantiago Pastorino2010-09-051-2/+1
| |
* | Fix number_to_human(0) exception [#5532 state:resolved]Ben Sharpe2010-09-052-1/+3
|/ | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* 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
|
* Implemented RouteSet#default_scope, which allows to set the scope for the ↵Piotr Sarnacki2010-09-033-2/+41
| | | | entire routes object
* Remove namespace for isolated namespaced models in formsPiotr Sarnacki2010-09-033-5/+37
|
* Use new ActiveModel::Naming.route_key in polymorphic_routesPiotr Sarnacki2010-09-032-12/+3
|
* 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-032-1/+28
| | | | polymorphic_routes, for example: polymorphic_url([blog, @post])
* Move RoutesProxy to separate filePiotr Sarnacki2010-09-033-32/+36
|
* Include all helpers from non-namespaced enginesPiotr Sarnacki2010-09-031-9/+3
|
* Set only helpers_path on inherited hook in action_controller/railtie.rb and ↵Piotr Sarnacki2010-09-033-9/+11
| | | | 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-033-16/+32
| | | | | | | | | | | 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-031-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 helpersPiotr Sarnacki2010-09-033-4/+10
|
* Refactor ActionMailer to not use hide_actionsPiotr Sarnacki2010-09-034-15/+36
|
* Move ActionController::Railties::RoutesHelpers and ↵Piotr Sarnacki2010-09-032-5/+6
| | | | ActionMailer::Railties::RoutesHelper to AbstractController::Railties::RoutesHelpers
* Fix generating urls with mounted helpers in view contextPiotr Sarnacki2010-09-033-3/+5
| | | | | | | | | | 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
* Include application's helpers and router helpers by default, but include ↵Piotr Sarnacki2010-09-034-4/+30
| | | | engine's ones for controllers inside isolated namespace
* For view_context we need to initialize RoutesProxy in context of controller, ↵Piotr Sarnacki2010-09-031-2/+4
| | | | not view, quick fix, I need to dig into it later
* This is not neededPiotr Sarnacki2010-09-031-26/+0
|