Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Removed unused file! Moved to actionview | Arun Agrawal | 2013-07-13 | 1 | -91/+0 |
| | | | | Removed unused file as this has been moved to actionview | ||||
* | add test-case to verify `error.path` when a helper was not found. | Yves Senn | 2013-07-12 | 1 | -0/+5 |
| | | | | | This verifies the code added with: https://github.com/rails/rails/commit/99c9d18601539c7e7e87f26bb047add1f93072af | ||||
* | move `MissingHelperError` out of the `ClassMethods` module. | Yves Senn | 2013-07-12 | 3 | -16/+23 |
| | |||||
* | Add documentation for add_flash_types [ci skip] | Robin Dupret | 2013-07-10 | 1 | -0/+17 |
| | | | | | Document a little bit the ActionController::Flash::add_flash_types class method. | ||||
* | Remove accidentally committed vi .swp file. | Marc Schütz | 2013-07-10 | 1 | -0/+0 |
| | |||||
* | Show real LoadError on helpers require | Piotr Niełacny | 2013-07-10 | 5 | -1/+38 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | When helper try to require missing file rails will throw exception about missing helper. # app/helpers/my_helper.rb require 'missing' module MyHelper end And when we try do load helper class ApplicationController helper :my end Rails will throw exception. This is wrong because there is a helper file. Missing helper file helpers/my_helper.rb Now when helper try to require non-existed file rails will throw proper exception. No such file to load -- missing | ||||
* | Merge pull request #11351 from jetthoughts/cleanup_request | Rafael Mendonça França | 2013-07-09 | 1 | -1/+1 |
|\ | | | | | Use content_length method instead of ENV['CONTENT_LENGTH'].to_i | ||||
| * | Use helper method to get ENV['CONTENT_LENGTH'] | Paul Nikitochkin | 2013-07-08 | 1 | -1/+1 |
| | | |||||
* | | Use Request#raw_post instead Request#body | Paul Nikitochkin | 2013-07-08 | 3 | -1/+15 |
|/ | | | | | | | | In order to get raw_post to be not empty after ParamsParser#parse_formatted_parameters, added rewinding of body stream input on parsing json params. Closes #11345 | ||||
* | Merge pull request #11313 from gsamokovarov/url-encoded-params-test | Rafael Mendonça França | 2013-07-06 | 1 | -25/+27 |
|\ | | | | | Cleanup ul_encoded_params_parsing_test | ||||
| * | Cleanup ul_encoded_params_parsing_test | Genadi Samokovarov | 2013-07-05 | 1 | -25/+27 |
| | | |||||
* | | Running isolated test for assertions and journey | Arun Agrawal | 2013-07-06 | 1 | -1/+1 |
| | | | | | | | | in actionpack | ||||
* | | actionpack not having activerecord tests [ci skip] | Arun Agrawal | 2013-07-06 | 1 | -10/+0 |
| | | |||||
* | | Space is not required for Set-Cookie header | Yamagishi Kazutoshi | 2013-07-05 | 3 | -1/+34 |
|/ | |||||
* | remove useless `env.merge!(env)` from ActionDispatch::IntegrationTest` | Yves Senn | 2013-07-04 | 1 | -2/+0 |
| | |||||
* | Missing closing + in documentation [ci skip] | Edho Arief | 2013-07-04 | 1 | -1/+1 |
| | |||||
* | Remove deprecated Rails application fallback for integration testing | Carlos Antonio da Silva | 2013-07-02 | 2 | -4/+5 |
| | | | | Set ActionDispatch.test_app instead. | ||||
* | Remove deprecated constants autoload | Carlos Antonio da Silva | 2013-07-02 | 1 | -3/+0 |
| | | | | | These constants were removed in 4b97ce5eb16cc20207516387fba98bf577e2e281, but I forgot to remove the autoload calls :sparkles:. | ||||
* | use extract_options! | Neeraj Singh | 2013-07-02 | 1 | -1/+1 |
| | |||||
* | Remove `page_cache_extension` deprecated method | Francesco Rodriguez | 2013-07-01 | 3 | -25/+4 |
| | |||||
* | Remove deprecated constants from Action Controller | Carlos Antonio da Silva | 2013-07-01 | 3 | -12/+12 |
| | | | | | | | | | | ActionController::AbstractRequest => ActionDispatch::Request ActionController::Request => ActionDispatch::Request ActionController::AbstractResponse => ActionDispatch::Response ActionController::Response => ActionDispatch::Response ActionController::Routing => ActionDispatch::Routing ActionController::Integration => ActionDispatch::Integration ActionController::IntegrationTest => ActionDispatch::IntegrationTest | ||||
* | store a symbol directly inside DISPATCH_CACHE for extra roflscale | Charlie Somerville | 2013-06-30 | 1 | -1/+1 |
| | |||||
* | use bytesize rather than force encoding | Aaron Patterson | 2013-06-28 | 1 | -2/+1 |
| | |||||
* | writing the new body can cause the response to be committed and the | Aaron Patterson | 2013-06-28 | 1 | -1/+3 |
| | | | | | | | | request thread to return up the stack before the instance variable is assigned. Synchronize so that the ivar is assigned before the other thread can activate. fixes #10984 | ||||
* | Re-remove deprecated API. | kennyj | 2013-06-28 | 2 | -19/+0 |
| | |||||
* | Minor changes to actionpack/actionview Rakefile | Arun Agrawal | 2013-06-26 | 1 | -5/+1 |
| | |||||
* | Merge pull request #11000 from sbeckeriv/accept_header | Carlos Antonio da Silva | 2013-06-25 | 3 | -1/+15 |
|\ | | | | | | | Fix undefined method `ref' for nil:NilClass for bad accept headers | ||||
| * | Fix undefined method `ref' for nil:NilClass for bad accept headers | Stephen Becker IV | 2013-06-19 | 3 | -1/+11 |
| | | |||||
* | | Merge pull request #11046 from arunagw/verbose_mode_on | Carlos Antonio da Silva | 2013-06-25 | 1 | -1/+1 |
|\ \ | | | | | | | Verbose mode on when running tests | ||||
| * | | Adding verbose in isolated test. | Arun Agrawal | 2013-06-21 | 1 | -1/+1 |
| | | | |||||
* | | | Fixing Issue #11083 | Akshay Khole | 2013-06-25 | 1 | -1/+1 |
| | | | | | | | | | | | | Removing other occurrences of `the` appearing twice | ||||
* | | | Fix shorthand routes where controller and action are in the scope | Andrew White | 2013-06-25 | 2 | -2/+19 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Merge `:action` from routing scope and assign endpoint if both `:controller` and `:action` are present. The endpoint assignment only occurs if there is no `:to` present in the options hash so should only affect routes using the shorthand syntax (i.e. endpoint is inferred from the the path). Fixes #9856 | ||||
* | | | test-case => failing | Yves Senn | 2013-06-25 | 1 | -0/+13 |
| | | | |||||
* | | | Merge pull request #11069 from ykzts/actiondispatch-ssl-secure-flag-igonore-case | Guillermo Iguaran | 2013-06-24 | 2 | -1/+15 |
|\ \ \ | | | | | | | | | Flag cookies as secure with ignore case in ActionDispatch::SSL | ||||
| * | | | Flag cookies as secure with ignore case in ActionDispatch::SSL | Yamagishi Kazutoshi | 2013-06-24 | 2 | -1/+15 |
| | | | | |||||
* | | | | ActionDispatch:SSL: don't include STS header in non-https responses | Geoff Buesing | 2013-06-23 | 2 | -2/+6 |
|/ / / | |||||
* / / | Saying gemcutter => rubygems [ci skip] | Arun Agrawal | 2013-06-21 | 1 | -1/+1 |
|/ / | |||||
* | | Don't remove trailing slash from PATH_INFO for mounted apps | Piotr Sarnacki | 2013-06-21 | 3 | -1/+16 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously when app was mounted as following: class Foo def call(env) [200, {}, [env['PATH_INFO']]] end end RackMountRailsBug::Application.routes.draw do mount RackTest.new => "/foo" end trailing slash was removed from PATH_INFO. For example requesting GET /foo/bar/ on routes defined above would result in a response containing "/foo/bar" instead of "/foo/bar/". This commit fixes the issue. (closes #3215) | ||||
* | | Adjust changelog for AV & AP | Łukasz Strzałkowski | 2013-06-20 | 1 | -35/+2 |
| | | |||||
* | | Template test were moved to AV | Łukasz Strzałkowski | 2013-06-20 | 1 | -1/+1 |
| | | |||||
* | | Remove digestor fixtures from AP | Łukasz Strzałkowski | 2013-06-20 | 12 | -25/+0 |
| | | | | | | | | They were moved to actionview/ and are not used in actionpack | ||||
* | | Remove unneeded test fixtures in AV | Piotr Sarnacki | 2013-06-20 | 108 | -281/+0 |
| | | |||||
* | | Remove unneeded files | Piotr Sarnacki | 2013-06-20 | 2 | -50/+0 |
| | | |||||
* | | Move template tests from actionpack to actionview | Piotr Sarnacki | 2013-06-20 | 53 | -16223/+2 |
| | | |||||
* | | Move actionpack/lib/action_view* into actionview/lib | Piotr Sarnacki | 2013-06-20 | 96 | -14646/+0 |
| | | |||||
* | | Add bare actionview gem to the root directory | Piotr Sarnacki | 2013-06-20 | 1 | -2/+2 |
| | | | | | | | | | | This commit creates structure for Action View gem and is first of a series of commits extracting Action View from Action Pack. | ||||
* | | Fix name of nested attributes option include_id | Eric Hankins | 2013-06-19 | 1 | -2/+2 |
| | | | | | | The option to disable including a hidden ID field on a `fields_for` nested association was incorrectly documented as `hidden_field_id` instead of `include_id` | ||||
* | | Remove duplicate letter 'a'. [ci skip] | Uģis Ozols | 2013-06-19 | 1 | -1/+1 |
| | | |||||
* | | Add `respond_with` `location` option to the docs | Tute Costa | 2013-06-18 | 1 | -2/+6 |
| | | |||||
* | | Added test for link_to_unless to make sure the result consistency. | dtaniwaki | 2013-06-16 | 1 | -0/+5 |
| | |