Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fixing the examples for button_to | Marc Baumbach | 2013-02-06 | 1 | -2/+2 |
| | | | The examples do not generate the output they intend to due to not explicitly declaring the hash separations. This causes it to be treated as one parameter instead of the intended two parameters. | ||||
* | Fix markdown syntax in actionpack CHANGELOG. | Steve Klabnik | 2013-02-01 | 1 | -1/+2 |
| | | | | Starting a line with an octothorpe makes an <h1> | ||||
* | Missing require hash/keys | Akira Matsuda | 2013-02-01 | 2 | -0/+3 |
| | |||||
* | Missing or unneeded require extract_options | Akira Matsuda | 2013-02-01 | 10 | -1/+10 |
| | |||||
* | Use \A in Regexps | Egor Homakov | 2013-02-01 | 1 | -1/+1 |
| | | | | | So, if there is redirect_to params[:q] i can send ?q=javascript:asdf()%0A/localpath Or something more nasty, so please use \A | ||||
* | remove dead code | Aaron Patterson | 2013-01-30 | 1 | -4/+1 |
| | |||||
* | change parameter name for positional args | Aaron Patterson | 2013-01-30 | 1 | -2/+1 |
| | |||||
* | nodoc the helper classes, cache stuff for optimized helper | Aaron Patterson | 2013-01-30 | 1 | -13/+17 |
| | |||||
* | cache path parts in the instance | Aaron Patterson | 2013-01-30 | 1 | -2/+3 |
| | |||||
* | stop evaling a string every time | Aaron Patterson | 2013-01-30 | 1 | -4/+4 |
| | |||||
* | moving helper classes outside the private block | Aaron Patterson | 2013-01-30 | 1 | -90/+88 |
| | |||||
* | pushing specialization down to the optimized class | Aaron Patterson | 2013-01-30 | 1 | -23/+25 |
| | |||||
* | use polymorphism to remove conditional | Aaron Patterson | 2013-01-30 | 1 | -35/+27 |
| | |||||
* | move conditionals to instance | Aaron Patterson | 2013-01-30 | 1 | -6/+16 |
| | |||||
* | pull stuff out of the caller and hide in the instance | Aaron Patterson | 2013-01-30 | 1 | -4/+3 |
| | |||||
* | moving more stuff on to the instance | Aaron Patterson | 2013-01-30 | 1 | -18/+18 |
| | |||||
* | move optimize_helper? to the helper instance | Aaron Patterson | 2013-01-30 | 1 | -9/+12 |
| | |||||
* | 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] |