Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | moving more stuff to the instance | Aaron Patterson | 2013-01-30 | 1 | -6/+6 |
| | |||||
* | moving some stuff to the initializer | Aaron Patterson | 2013-01-30 | 1 | -14/+38 |
| | |||||
* | moved more evald code | Aaron Patterson | 2013-01-30 | 1 | -8/+7 |
| | |||||
* | factored out some of the dynamic code | Aaron Patterson | 2013-01-30 | 1 | -19/+31 |
| | |||||
* | Fix `content_tag_for` with array html option. | Semyon Perepelitsa | 2013-01-31 | 3 | -1/+25 |
| | | | | | | | | | | | | | | | | | It would embed array as string instead of joining it like `content_tag` does: content_tag(:td, class: ["foo", "bar"]){} #=> '<td class="foo bar"></td>' Before: content_tag_for(:td, item, class: ["foo", "bar"]){} #=> '<td class="item ["foo", "bar"]" id="item_1"></td>' After: content_tag_for(:td, item, class: ["foo", "bar"]){} #=> '<td class="item foo bar" id="item_1"></td>' | ||||
* | 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 |
| | | |