Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #3479 from arvida/ensure-date-header-on-expires-in | José Valim | 2012-02-18 | 1 | -0/+7 |
|\ | | | | | Ensure Date header on expires_in | ||||
| * | Added test for setting of HTTP Date header when calling #expires_in | arvida | 2011-10-31 | 1 | -0/+7 |
| | | |||||
* | | let expires_in accept a must_revalidate flag | Xavier Noria | 2012-02-17 | 1 | -0/+20 |
| | | |||||
* | | 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 | ||||
* | | Merge pull request #3305 from cjolly/request_filtered_env | Jeremy Kemper | 2012-02-13 | 1 | -0/+33 |
|\ \ | | | | | | | Exclude rack.request.form_vars from request.filtered_env | ||||
| * | | Exclude rack.request.form_vars from request.filtered_env | Chad Jolly | 2012-01-18 | 1 | -0/+33 |
| | | | |||||
* | | | Removing old Controller test | Thiago Pradi | 2012-02-12 | 1 | -37/+0 |
| | | | |||||
* | | | Cleaning Route generation tests | Thiago Pradi | 2012-02-12 | 1 | -30/+0 |
| | | | |||||
* | | | replacing the orderhash with hash for ruby-1.9 | ganesh | 2012-02-09 | 1 | -3/+2 |
| | | | |||||
* | | | 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 Conflicts: actionpack/test/controller/base_test.rb | ||||
* | | | Fixed force_ssl redirects to include original query params | Ryan McGeary | 2012-02-07 | 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. Conflicts: actionpack/lib/action_controller/metal/force_ssl.rb | ||||
* | | | 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 | ||||
* | | | Merge pull request #4529 from ask4prasath/refactor | Vijay Dev | 2012-01-25 | 1 | -8/+10 |
|\ \ \ | | | | | | | | | Refactored class methods on address render test | ||||
| * | | | Refactored class methods on address render test | prasath | 2012-01-19 | 1 | -8/+10 |
| |/ / | |||||
* | | | Merge pull request #3775 from karevn/master | Aaron Patterson | 2012-01-24 | 1 | -1/+14 |
|\ \ \ | | | | | | | | | Please pull my changes - they fix a rare problem with tests framework | ||||
| * | | | Fix: when using subdomains and constraints, request params were not passed ↵ | karevn | 2011-11-28 | 1 | -1/+14 |
| | | | | | | | | | | | | | | | | to constraints callback | ||||
* | | | | 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 | ||||
* | | | | moved the `get` testing method to a module | Aaron Patterson | 2012-01-23 | 1 | -11/+2 |
| | | | | |||||
* | | | | Remove not used requires from csrf helper file and test | Carlos Antonio da Silva | 2012-01-21 | 1 | -7/+0 |
| | | | | | | | | | | | | | | | | | | | | These requires were added in a87b92d and the implementation changed in 2cdc1f0, removing the need for them. | ||||
* | | | | Removed unused assigns from ActionView::Template::Error | brainopia | 2012-01-20 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | | | They existed since initial rails commit by DHH but lost use a long time ago | ||||
* | | | | Use performed? instead of checking for response_body | Carlos Antonio da Silva | 2012-01-19 | 1 | -3/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | * Check for performed? instead of response_body * Change performed? to return a boolean * Refactor AC::Metal#response_body= to reuse variable | ||||
* | | | | Do not deprecate performed. | José Valim | 2012-01-19 | 1 | -0/+6 |
| |/ / |/| | | |||||
* | | | ActionView now has its own logger | Rafael Mendonça França | 2012-01-18 | 1 | -1/+2 |
| | | | |||||
* | | | Refactor FormBuilder arguments and default config | Carlos Antonio da Silva | 2012-01-18 | 1 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Do not reopen AV::Base to define default form builder Inside the load hook we are already in AV::Base context. * Do not pass the given block to the form builder The block is evaluated in fields_for context using capture, with the builder as argument. This means we do not need to give the block to the FormBuilder itself. | ||||
* | | | Bring back rendering templates that start with / in nested structures | Carlos Antonio da Silva | 2012-01-17 | 1 | -0/+22 |
| | | | |||||
* | | | Remove deprecated default_charset= from AC::Base | Carlos Antonio da Silva | 2012-01-17 | 1 | -4/+4 |
| | | | | | | | | | | | | | | | This should be set globally as a configuration, using `config.action_dispatch.default_charset` instead | ||||
* | | | Move render_to_body logic to return a spaced string to AC::Rendering | Carlos Antonio da Silva | 2012-01-17 | 1 | -2/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This seems to be required only when calling render :partial with an empty collection from a controller. This call happens to return no content, letting the response body empty, which means to Rails that it should go on and try to find a template to render based on the current action name, thus failing hard. Although tests keep all green, we need to check a better way to fix this. | ||||
* | | | Remove method missing handling when action is not found, use action missing ↵ | Carlos Antonio da Silva | 2012-01-17 | 1 | -45/+0 |
| | | | | | | | | | | | | | | | | | | | | | instead Do not create a method_missing method to handle not found actions, use the action_missing method provided by Rails instead. | ||||
* | | | Rename test class and fix tests to keep consistency | Carlos Antonio da Silva | 2012-01-17 | 1 | -10/+10 |
| | | | | | | | | | | | | Based on 50d23bc2bd3653b3c66e480c22ae97c5f7fd7f62. | ||||
* | | | Remove deprecated logic to render templates starting with / | Carlos Antonio da Silva | 2012-01-17 | 1 | -22/+0 |
| | | | | | | | | | | | | render :template => "/foo/bar" | ||||
* | | | Remove rescue_action from compatibility module and tests | Carlos Antonio da Silva | 2012-01-17 | 16 | -56/+1 |
| | | | |||||
* | | | Remove other old compatibility constants | Carlos Antonio da Silva | 2012-01-17 | 2 | -8/+8 |
| | | | |||||
* | | | Remove old compatibility methods not being used | Carlos Antonio da Silva | 2012-01-17 | 2 | -9/+0 |
| | | | |||||
* | | | When force redirecting to SSL, make sure that the session is kept. As we're ↵ | Marcin Bunsch and Przemek Dąbek | 2012-01-12 | 1 | -1/+37 |
| | | | | | | | | | | | | moving from a non-secure to secure environment, it's safe | ||||
* | | | AC:TestCase::Behavior#head only accepts parameters as second argument | Rafael Mendonça França | 2012-01-10 | 1 | -0/+4 |
| | | | |||||
* | | | Whitespaces :scissors: | Rafael Mendonça França | 2012-01-10 | 1 | -4/+4 |
| | | | |||||
* | | | Fix ActionController::TestCase::Behavior.head | Rafael Mendonça França | 2012-01-10 | 1 | -0/+5 |
| | | | |||||
* | | | Merge pull request #4371 from rafaelfranca/test_unit_cleanup | Aaron Patterson | 2012-01-09 | 1 | -3/+5 |
|\ \ \ | | | | | | | | | Test unit cleanup | ||||
| * | | | Test constant is not defined anymore | Rafael Mendonça França | 2012-01-06 | 1 | -3/+5 |
| | | | | |||||
* | | | | get rid of using instance_variable_names method from AS | Sergey Nartimov | 2012-01-07 | 1 | -1/+1 |
|/ / / | | | | | | | | | | | | | - instance_variables return symbols in 1.9 - there is instance_variable_defined? method | ||||
* | | | Remove deprecation warning from test related to old process api | Carlos Antonio da Silva | 2012-01-06 | 1 | -1/+1 |
| | | | |||||
* | | | AP tests should inherit from AS::TestCase | Aaron Patterson | 2012-01-05 | 6 | -8/+8 |
| | | | |||||
* | | | Revert "remove deprecated API" | Aaron Patterson | 2012-01-05 | 2 | -1/+8 |
| | | | | | | | | | | | | This reverts commit f53c247d10acbaacb0d61824cfce888c4b0520d2. | ||||
* | | | remove deprecated API | Aaron Patterson | 2012-01-05 | 2 | -8/+1 |
| | | | |||||
* | | | Moved all the logger methods to active support logger | Karunakar (Ruby) | 2012-01-06 | 4 | -5/+5 |
| | | | | | | | | | | | | minor | ||||
* | | | Clean up routes inclusion and add some comments for the next soul that ↵ | José Valim | 2012-01-03 | 1 | -14/+0 |
| | | | | | | | | | | | | decides to adventure on this code. | ||||
* | | | moving test_test.rb to test_case_test.rb for consistency. | Nick Sutterer | 2012-01-03 | 1 | -0/+0 |
| | | | |||||
* | | | allow sending documents in AC::TestCase#post and friends. | Nick Sutterer | 2012-01-03 | 1 | -8/+30 |
| | | | |||||
* | | | remove ActiveSupport::Base64 in favor of ::Base64 | Sergey Nartimov | 2012-01-02 | 1 | -1/+1 |
| | | |