Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add failing test re #3436 which demonstrates content_type is not respected ↵ | Kunal Shah | 2012-05-07 | 1 | -0/+21 |
| | | | | when using the :head method/shortcut | ||||
* | Add a role option to wrap_parameters. | Nick Ragaz | 2012-05-04 | 1 | -2/+15 |
| | | | | The role option identifies which parameters are accessible and should be wrapped. The default role is :default. | ||||
* | Don't convert params if the request isn't HTML - fixes #5341 | Andrew White | 2012-04-29 | 1 | -2/+40 |
| | | | | | | | | (cherry picked from commit 7a80b69e00f68e673c6ceb5cc684aa9196ed3d9f) Conflicts: actionpack/test/controller/test_test.rb | ||||
* | default_url_options does not receive one argument anymore | Rafael Mendonça França | 2012-04-08 | 1 | -1/+1 |
| | |||||
* | Remove the leading \n added by textarea on assert_select | Santiago Pastorino | 2012-03-30 | 1 | -0/+7 |
| | |||||
* | Set proper rendered_format when doing render :inline | Santiago Pastorino | 2012-03-28 | 1 | -0/+1 |
| | | | | Closes #5632 | ||||
* | Cover one more case in auth_token and remote forms | Piotr Sarnacki | 2012-03-28 | 1 | -16/+22 |
| | | | | | If embedding auth_token in remote forms is off and we pass a value for auth_token it should respect it. | ||||
* | Added config.action_view.embed_authenticity_token_in_remote_forms | Piotr Sarnacki | 2012-03-28 | 1 | -6/+48 |
| | | | | | | | | There is a regression introduced in 16ee611fa, which breaks remote forms that should also work without javascript. This commit introduces config option that allows to configure this behavior defaulting to the old behavior (ie. include authenticity token in remote forms by default) | ||||
* | If partial is rendered in controller, grab format from template | Piotr Sarnacki | 2012-03-27 | 1 | -0/+13 |
| | | | | | | | | | Previously `rendered_format` was set only based on mime types passed in Accept header, which was wrong if first type from Accept was different than rendered partial. The fix is to simply move setting rendered_format to the place where template is available and grab format from the template. If it fails we can fallback to formats passed by Accept header. | ||||
* | Build fix for ruby1.8.7-358 | Arun Agrawal | 2012-03-19 | 1 | -1/+1 |
| | |||||
* | Add missing test for #5308 | Piotr Sarnacki | 2012-03-17 | 1 | -0/+13 |
| | |||||
* | Fix #5440 - multiple render_to_string breaks partials formats | Piotr Sarnacki | 2012-03-17 | 1 | -0/+26 |
| | | | | | | | | | This fixes situation where rendering template to string sets `rendered_format` to the format rendered there. This is ok to have consistent formats rendered in partials, but it breaks on next renders if format is explicitly set or on last render where default format does not necessarily need to be the format of first rendered template. | ||||
* | Merge pull request #5457 from brianmario/typo-fix | Aaron Patterson | 2012-03-15 | 1 | -1/+1 |
| | | | | Fix typo in redirect test | ||||
* | Merge pull request #5456 from brianmario/redirect-sanitization | Aaron Patterson | 2012-03-15 | 1 | -0/+20 |
| | | | | Strip null bytes from Location header | ||||
* | Remove ActionController::TestCase#rescue_action_in_public! | Piotr Sarnacki | 2012-03-15 | 1 | -5/+0 |
| | | | | | | This method has no effect since exception handling was moved to middlewares and ActionController tests do not use any middlewares. | ||||
* | Allow you to force the authenticity_token to be rendered even on remote ↵ | David Heinemeier Hansson | 2012-03-14 | 1 | -0/+11 |
| | | | | forms if you pass true | ||||
* | Do not include the authenticity token in forms where remote: true as ajax ↵ | David Heinemeier Hansson | 2012-03-14 | 1 | -0/+11 |
| | | | | forms use the meta-tag value | ||||
* | Add test case for #5307 | Carlos Antonio da Silva | 2012-03-07 | 1 | -2/+6 |
| | |||||
* | Failing test for mime responder respond_with using a block. | Mario Visic | 2012-03-05 | 1 | -0/+36 |
| | |||||
* | Fix #5238, rendered_format is not set when template is not rendered | Piotr Sarnacki | 2012-03-02 | 1 | -2/+6 |
| | |||||
* | Adding tests for non-optional glob parameters | Andrew White | 2012-02-26 | 1 | -2/+27 |
| | |||||
* | assert => assert_equal | Arun Agrawal | 2012-02-25 | 1 | -3/+3 |
| | |||||
* | Checking headers in a better way. as doing here | Arun Agrawal | 2012-02-25 | 1 | -4/+3 |
| | | | | 406ece4729e8fd81f156bcb3fce16e938522466e | ||||
* | Build fix for ruby187-p358 | Arun Agrawal | 2012-02-25 | 1 | -2/+7 |
| | |||||
* | Fix some warnings on 3-2-stable | kennyj | 2012-02-21 | 2 | -5/+5 |
| | |||||
* | adding tests to document behavior for #4817 | Aaron Patterson | 2012-02-14 | 1 | -0/+26 |
| | |||||
* | Rack body respond to each and not to join | Santiago Pastorino | 2012-02-14 | 1 | -0/+12 |
| | | | | | | | This fixes undef `to_str' for Rack::Chunked::Body when using caches_action + streaming on an action Closes #5027 | ||||
* | Added unit test to cover changes to RouteSet.url_for | Ryan McGeary | 2012-02-07 | 1 | -0/+16 |
| | | | | | ActionDispatch::Routing::RouteSet.url_for now handles passing params through to ActionDispatch::Http::Url.url_for | ||||
* | Fixed force_ssl redirects to include original query params | Ryan McGeary | 2012-02-06 | 1 | -0/+6 |
| | | | | | | `ActionController.force_ssl` redirects http URLs to their https equivalent; however, when a URL contains a query string, the resulting redirect lacked the original query string. | ||||
* | Fix override API response bug in respond_with | Prem Sichanugrist | 2012-02-03 | 1 | -0/+25 |
| | | | | | | | | | | Default responder was only using the given respond block when user requested for HTML format, or JSON/XML format with valid resource. This fix the responder so that it will use the given block regardless of the validity of the resource. Note that in this case you'll have to check for object's validity by yourself in the controller. Fixes #4796 | ||||
* | Added custom regexps to ASTs that have literal nodes on either side of | Aaron Patterson | 2012-01-23 | 1 | -0/+62 |
| | | | | symbol nodes. Fixes #4585 | ||||
* | Do not deprecate performed? | José Valim | 2012-01-19 | 1 | -0/+6 |
| | |||||
* | Deprecate AC::UnknownError and AC::DoubleRenderError | Carlos Antonio da Silva | 2012-01-17 | 2 | -7/+7 |
| | | | | | Use the constants AbstractController::ActionNotFound and AbstractController::DoubleRenderError respectively instead. | ||||
* | Deprecate default_charset= at controller level | Carlos Antonio da Silva | 2012-01-17 | 1 | -12/+20 |
| | |||||
* | Add some deprecations for logic being removed in 4.0 | Carlos Antonio da Silva | 2012-01-17 | 4 | -13/+10 |
| | |||||
* | Fix http digest authentication with trailing '/' or '?' (fixes #4038 and #3228) | Piotr Sarnacki | 2012-01-10 | 1 | -2/+43 |
| | |||||
* | When generating routes, the last defined named route wins. This is in | Aaron Patterson | 2012-01-03 | 1 | -15/+23 |
| | | | | | | | | | | contrast to route recognition where the first recognized route wins. This behavior will not exist in Rails 4.0. See: https://github.com/rails/rails/issues/4245 https://github.com/rails/rails/issues/4164 | ||||
* | Revert "Named Routes shouldn't override existing ones (currently route ↵ | Aaron Patterson | 2012-01-03 | 1 | -2/+2 |
| | | | | | | recognition goes with the earliest match, named routes use the latest match)" This reverts commit 71d769e3b58cb56b4b1d5143936c65be8b27c490. | ||||
* | deprecate ActiveSupport::Base64 | Sergey Nartimov | 2012-01-03 | 1 | -1/+1 |
| | | | | extend and define ::Base64 if needed | ||||
* | last named route wins. fixes #4164 | Aaron Patterson | 2011-12-28 | 1 | -0/+40 |
| | | | | | | This differs from route recognition where first recognized route wins. This will not be supported in Rails 4.0 so that route recognition and generation rules are consistent. | ||||
* | Provide a class optin for page_cache_compression. | José Valim | 2011-12-24 | 1 | -3/+4 |
| | |||||
* | Gzip files on page caching | Andrey A.I. Sitnik | 2011-12-24 | 1 | -1/+40 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | adding an integration test for splat regexp substitutions. fixes #4138 | Aaron Patterson | 2011-12-22 | 1 | -0/+13 |
| | |||||
* | Adding an integration test. Fixes #4136 | Aaron Patterson | 2011-12-22 | 1 | -0/+9 |
| | |||||
* | rack bodies should be a list | Aaron Patterson | 2011-12-22 | 1 | -7/+7 |
| | |||||
* | refactoring routing tests | Aaron Patterson | 2011-12-22 | 1 | -34/+19 |
| | |||||
* | adding integration test for journey #7 | Aaron Patterson | 2011-12-21 | 1 | -0/+22 |
| | |||||
* | caches should not be global, so no need to clear in the teardown | Aaron Patterson | 2011-12-21 | 1 | -4/+0 |
| | |||||
* | remove warnings about @variable_for_layout | lest | 2011-12-20 | 1 | -5/+6 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | adding tests for #4029 | Aaron Patterson | 2011-12-19 | 1 | -0/+51 |
| |