Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Changelog about BestStandardsSupport removal | Guillermo Iguaran | 2013-01-29 | 1 | -5/+6 |
| | |||||
* | Add 'X-UA-Compatible' => 'chrome=1' to default headers | Guillermo Iguaran | 2013-01-29 | 2 | -2/+5 |
| | |||||
* | Remove BestStandardsSupport middleware | Guillermo Iguaran | 2013-01-29 | 4 | -67/+0 |
| | |||||
* | Merge pull request #9104 from bemurphy/remove_bad_idea_parser_test | Carlos Antonio da Silva | 2013-01-29 | 1 | -13/+0 |
|\ | | | | | Remove yaml Proc param parser test | ||||
| * | Remove yaml Proc param parser test | Brendon Murphy | 2013-01-28 | 1 | -13/+0 |
| | | | | | | | | | | | | | | | | I don't believe this test is exercising any explicit params_parser behavior that the other two Proc tests aren't already doing. Given that we now know it's a bad idea to load user input via YAML.load, somebody reading this test might get a dangerous idea about building out a YAML params parser. | ||||
* | | s/ERb/ERB/ | Akira Matsuda | 2013-01-29 | 1 | -1/+1 |
|/ | |||||
* | Merge pull request #9096 from tricknotes/fix-code-comment | Rafael Mendonça França | 2013-01-28 | 1 | -4/+4 |
|\ | | | | | Fix code comment | ||||
| * | Fix JavaScript syntax in code comment [ci skip] | Ryunosuke SATO | 2013-01-29 | 1 | -4/+4 |
| | | |||||
* | | Move AS changelog entry to the top, improve AP changelog a bit | Carlos Antonio da Silva | 2013-01-28 | 1 | -4/+4 |
| | | | | | | | | [ci skip] | ||||
* | | Added a Changelog entry for fixing HEAD requests. | Michiel Sikkes | 2013-01-28 | 1 | -0/+5 |
| | | |||||
* | | Make current_url? work with a HEAD method | Michiel Sikkes | 2013-01-28 | 2 | -1/+7 |
| | | | | | | | | | | | | ActionDispatch::Head was removed in favor of Rack::Head. But Rack::Head does not convert GET requests to HEAD requests so we need to do checking for HEAD requests ourselves. | ||||
* | | Remove caching_allowed? from ActionController::Caching | Michiel Sikkes | 2013-01-28 | 1 | -4/+0 |
|/ | | | | | Where is this used? No other code references to this method and it isn't being tested anywhere. No tests fail when commented out. | ||||
* | Merge pull request #9032 from firmhouse/head-breaks-csrf | Santiago Pastorino | 2013-01-28 | 2 | -2/+6 |
|\ | | | | | Make HEAD work / convert to GET once more | ||||
| * | Added request.head? to forgery protection code | Michiel Sikkes | 2013-01-22 | 1 | -2/+2 |
| | | |||||
| * | Added a test that shows that a HEAD request does not normally pass CSRF ↵ | Michiel Sikkes | 2013-01-22 | 1 | -0/+4 |
| | | | | | | | | protection | ||||
* | | Use Encoding::UTF_8 constant :do_not_litter: | Akira Matsuda | 2013-01-28 | 7 | -11/+11 |
| | | |||||
* | | Use already defined Encoding constants rather than creating one-trip Strings | Akira Matsuda | 2013-01-28 | 2 | -2/+2 |
| | | |||||
* | | add fetch to CookieJar | Aaron Patterson | 2013-01-27 | 2 | -0/+34 |
| | | |||||
* | | Merge remote-tracking branch 'docrails/master' | Xavier Noria | 2013-01-26 | 6 | -4/+98 |
|\ \ | | | | | | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/form_options_helper.rb guides/code/getting_started/app/controllers/comments_controller.rb | ||||
| * | | generic pass before merging docrails | Xavier Noria | 2013-01-26 | 4 | -422/+62 |
| | | | |||||
| * | | :scissors: "raise" duplication | Akira Matsuda | 2013-01-26 | 1 | -2/+1 |
| | | | |||||
| * | | Review #translate docs [ci skip] | Carlos Antonio da Silva | 2013-01-20 | 1 | -8/+7 |
| | | | |||||
| * | | Add documentation for abstract controller #translate and #localize method. | Jens Bissinger | 2013-01-20 | 1 | -0/+10 |
| | | | |||||
| * | | Add documentation to ActionDispatch::Response | Matthew Stopa | 2013-01-20 | 1 | -0/+7 |
| | | | |||||
| * | | fix broken examples format in image_tag helper [ci skip] | Francesco Rodriguez | 2013-01-18 | 1 | -0/+2 |
| | | | |||||
| * | | repeating documentation for option form helpers | Thiago Pinto | 2013-01-17 | 1 | -0/+286 |
| | | | |||||
| * | | repeating documentation for date and time form helpers | Thiago Pinto | 2013-01-17 | 1 | -0/+148 |
| | | | |||||
| * | | Fix typo introduced in 0004ca3a | Carlos Antonio da Silva | 2013-01-16 | 1 | -1/+1 |
| | | | | | | | | | | | | [ci skip] | ||||
| * | | More documentation for ActionDispatch::Response | Matthew Stopa | 2013-01-16 | 1 | -1/+3 |
| | | | | | | | | | | | | [ci skip] | ||||
| * | | Document ActionDispatch::Response#body method | Matthew Stopa | 2013-01-16 | 1 | -0/+1 |
| | | | |||||
* | | | Add keys/values methods to TestSession | Carlos Antonio da Silva | 2013-01-25 | 2 | -5/+19 |
| | | | | | | | | | | | | Bring back the same API we have with Request::Session. | ||||
* | | | Integrate Action Pack with Rack 1.5 | Carlos Antonio da Silva | 2013-01-25 | 8 | -19/+37 |
| | | | | | | | | | | | | | | | | | | All ActionPack and Railties tests are passing. Closes #8891. [Carlos Antonio da Silva + Santiago Pastorino] | ||||
* | | | use the helpers list rather than getting the methods from the module | Aaron Patterson | 2013-01-24 | 1 | -1/+1 |
| | | | |||||
* | | | module_eval is not necessary here | Aaron Patterson | 2013-01-24 | 1 | -3/+1 |
| | | | |||||
* | | | don't need to eval everything | Aaron Patterson | 2013-01-24 | 1 | -1/+1 |
| | | | |||||
* | | | Set Thread.abort_on_exception for the whole AS, AP, and AR tests | Akira Matsuda + Koichi Sasada | 2013-01-24 | 1 | -0/+2 |
| | | | | | | | | | | | | this would give us some more clues in case a test silently dies inside Thread | ||||
* | | | ActionDispatch::Http::UploadedFile is a permitted scalar [Closes #9051] | Xavier Noria | 2013-01-23 | 2 | -3/+7 |
| |/ |/| | |||||
* | | Lets kepp using Ruby 1.9 syntax | Rafael Mendonça França | 2013-01-22 | 2 | -37/+37 |
| | | |||||
* | | Only check for unpermmited parameters if | Rafael Mendonça França | 2013-01-22 | 1 | -1/+1 |
| | | | | | | | | action_on_unpermitted_parameters is present | ||||
* | | avoid creating an object in every call | Xavier Noria | 2013-01-22 | 1 | -1/+2 |
| | | | | | | | | This was a suggestion of @carlosantoniodasilva, thanks! | ||||
* | | Remove tabs, use spaces :scissors: | Carlos Antonio da Silva | 2013-01-22 | 1 | -2/+2 |
| | | | | | | | | [ci skip] | ||||
* | | Merge branch 'isolating_tests' | Akira Matsuda | 2013-01-22 | 2 | -2/+12 |
|\ \ | | | | | | | | | | Now we're almost ready to remove this: https://github.com/rails/rails/blob/5294ad8/activesupport/lib/active_support/test_case.rb#L29 | ||||
| * | | Restore I18n.locale after running tests | Akira Matsuda | 2013-01-22 | 1 | -1/+7 |
| | | | |||||
| * | | Make sure to reset default_url_options | Akira Matsuda | 2013-01-22 | 1 | -1/+5 |
| | | | |||||
* | | | Stylistic pass at form_helper_test | Rafael Mendonça França | 2013-01-21 | 1 | -286/+362 |
| | | | |||||
* | | | Capture block so content won't leak. | José Mota | 2013-01-21 | 3 | -4/+61 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The [following pull request](https://github.com/rails/rails/pull/8916) fixed the block being passed to the appropriate helper method. However, the content being passed into the block is generating repeated markup on the page due to some weird ERb evaluation. This commit tries to capture the block's generated output so the page isn't flooded with markup. [Rafael França + José Mota] Closes #8936 | ||||
* | | | Update actionpack's CHANGELOG for 445f14e | Piotr Sarnacki | 2013-01-21 | 1 | -0/+5 |
| | | | |||||
* | | | Fix asset_path in mounted engine | Piotr Sarnacki | 2013-01-21 | 1 | -2/+1 |
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | Historically serving assets from a mountable engine could be achieved by running ActionDispatch::Static as a part of engine middleware stack or to copy assets prefixed with an engine name. After introduction of assets pipeline this is not needed as all of the assets are served or compiled into main application's assets. This commit removes the obsolete line making asset_path always generate paths relative to the root or config.relative_url_root if it's set. (closes #8119) | ||||
* | | Duplicate possible frozen string from route | Andrew White | 2013-01-21 | 2 | -2/+12 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | Ruby 1.9 freezes Hash string keys by default so where a route is defined like this: get 'search' => 'search' then the Mapper will derive the action from the key. This blows up later when the action is added to the parameters hash and the encoding is forced. Closes #3429 | ||||
* | | Merge pull request #9014 from virusman/form_helpers_ar_tests | Rafael Mendonça França | 2013-01-21 | 2 | -0/+92 |
|\ \ | | | | | | | AR integration tests for form helpers |