Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | allocate new responses rather than recycling them | Aaron Patterson | 2015-07-08 | 1 | -12/+5 | |
| | | | | | | | | | | There is no reason to "recycle" response objects when we can just allocate a new one. | |||||
* | | pass cookies from the jar in the HTTP_COOKIE header | eileencodes | 2015-07-07 | 1 | -8/+7 | |
| | | | | | | | | | | we should be pushing the cookies in via headers rather than maintaining some object and "recycling" it. | |||||
* | | Send cookies with request | eileencodes | 2015-07-07 | 1 | -1/+4 | |
|/ | ||||||
* | Merge pull request #20276 from davetron5000/revert-head-on-no-template | Kasper Timm Hansen | 2015-06-27 | 1 | -2/+17 | |
|\ | | | | | Allow default_render to take a block to customize behavior when there's no template | |||||
| * | Override default_render's behavior with a block | Dave Copeland | 2015-06-20 | 1 | -2/+17 | |
| | | | | | | | | | | | | | | | | | | | | In 0de4a23 the behavior when there is a missing template was changed to not raise an error, but instead head :no_content. This is a breaking change and some gems rely on this happening. To allow gems and other code to work around this, allow `default_render` to take a block which, if provided, will execute the contents of that block instead of doing the `head :no_content`. | |||||
* | | Merge pull request #20659 from ↵ | Yves Senn | 2015-06-22 | 1 | -2/+4 | |
|\ \ | | | | | | | | | | | | | | | | vngrs/strong_parameters_unpermitted_parameters_wrong_doc_fix Fix the documentation about ActionController::UnpermittedParameters [ci skip] | |||||
| * | | Fix the documentation about ActionController::UnpermittedParameters [ci skip] | Mehmet Emin İNAÇ | 2015-06-22 | 1 | -2/+4 | |
|/ / | ||||||
* / | Fix the documentation about ActionController::ParameterMissing [ci skip] | Mehmet Emin İNAÇ | 2015-06-22 | 1 | -5/+5 | |
|/ | ||||||
* | Deprecate passing hash as first parameter into ActionController::Head | Mehmet Emin İNAÇ | 2015-06-15 | 1 | -2/+12 | |
| | ||||||
* | A few documentation edits [ci skip] | Robin Dupret | 2015-06-15 | 2 | -10/+12 | |
| | ||||||
* | Add missing "header" word in documentation of Token#authentication_request ↵ | Prathamesh Sonpatki | 2015-06-14 | 1 | -1/+1 | |
| | | | | [ci skip] | |||||
* | Update RDoc for ActionController::TestCase for kwargs. | Victor Costan | 2015-06-12 | 1 | -1/+1 | |
| | ||||||
* | Merge pull request #20522 from colby-swandale/doc-update | Sean Griffin | 2015-06-11 | 1 | -1/+1 | |
|\ | | | | | fixed sring to be string in ActiveRecord::Base params documentation [ci skip] | |||||
| * | fixed sring to be string in ActiveRecord::Base params documentation | Colby Swandale | 2015-06-12 | 1 | -1/+1 | |
| | | ||||||
* | | Mention that doing nothing in Rails API controllers returns 204 | Santiago Pastorino | 2015-06-11 | 1 | -1/+1 | |
| | | ||||||
* | | Return 204 if render is not called in API controllers | Jorge Bejar | 2015-06-11 | 3 | -6/+15 | |
| | | ||||||
* | | Include ParamsWrapper in AC::API | Jorge Bejar | 2015-06-11 | 1 | -1/+5 | |
| | | | | | | | | | | | | | | | | | | ParamsWrapper was initially removed from API controllers according to the following discusision: https://github.com/rails-api/rails-api/issues/33 However, we're including it again so Rails API devs can decide whether to enable or disable it. | |||||
* | | Fix MimeResponds example in AC::API documentation | Santiago Pastorino | 2015-06-11 | 1 | -3/+5 | |
| | | ||||||
* | | Do not say that Api Controllers are faster than regular ones in docs | Jorge Bejar | 2015-06-11 | 1 | -4/+4 | |
| | | ||||||
* | | Remove Compatibility module since we don't remember why it was added :smile: | Jorge Bejar | 2015-06-11 | 1 | -19/+0 | |
| | | ||||||
* | | Remove api_rendering is not needed | Santiago Pastorino | 2015-06-11 | 2 | -15/+1 | |
| | | ||||||
* | | Add ActionController API functionality | Santiago Pastorino | 2015-06-11 | 2 | -0/+171 | |
|/ | ||||||
* | better clarity of params source in ActionController::Base documentation [ci ↵ | Colby Swandale | 2015-06-11 | 1 | -3/+3 | |
| | | | | skip] | |||||
* | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2015-06-05 | 1 | -1/+1 | |
|\ | ||||||
| * | [ci skip] Upcase `is` | yui-knk | 2015-05-25 | 1 | -1/+1 | |
| | | ||||||
* | | Merge pull request #19094 from phoet/have_bearer_be_valid_as_well | Rafael Mendonça França | 2015-06-01 | 1 | -1/+1 | |
|\ \ | | | | | | | Have Bearer be valid as well | |||||
| * | | allow `Bearer` as well as `Token` | phoet | 2015-06-01 | 1 | -1/+1 | |
| | | | ||||||
* | | | Merge pull request #20138 from tgxworld/deprecated_assert_template | Rafael Mendonça França | 2015-06-01 | 2 | -188/+3 | |
|\ \ \ | |/ / |/| | | Deprecate `assert_template` and `assigns()`. | |||||
| * | | Remove `assigns` and `assert_template`. | Guo Xiang Tan | 2015-05-30 | 2 | -188/+3 | |
| | | | ||||||
* | | | Use `any?` rather than `present?` to check args | eileencodes | 2015-05-31 | 1 | -1/+1 | |
|/ / | | | | | | | | | | | It's better to use Ruby methods when possible over methods defined by Active Support because then it does not need to rely on any dependencies. | |||||
* | | Remove unused code. | Guo Xiang Tan | 2015-05-29 | 1 | -2/+0 | |
| | | | | | | | | Code was moved into the `assign` method. | |||||
* | | Deprecate `:nothing` option for render method | Mehmet Emin İNAÇ | 2015-05-28 | 1 | -0/+1 | |
| | | | | | | | | `head` method works similar to `render` method with `:nothing` option | |||||
* | | Merge pull request #19808 from byroot/action-parameter | Rafael Mendonça França | 2015-05-27 | 1 | -15/+12 | |
|\ \ | | | | | | | [PoC] Stop shadowing parameters named `action` | |||||
| * | | Allow to parameters named `action` or `controller` from AC::TestCase helpers | Jean Boussier | 2015-04-18 | 1 | -15/+12 | |
| | | | ||||||
* | | | Spelling/typo/grammatical fixes [ci skip] | karanarora | 2015-05-23 | 4 | -5/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | spelling fix [ci skip] example to be consistent [ci skip] grammatical fix typo fixes [ci skip] | |||||
* | | | [ci skip] Remove comments about Rails 3.1 | claudiob | 2015-05-11 | 2 | -5/+4 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Stems from https://github.com/rails/rails/pull/20105#issuecomment-100900939 where @senny said: > From my point of view, all the docs (guides, API) are version bound. > They should describe that version and continue to be available when newer versions are released. > The cross referencing can be done by the interested user. | |||||
* | | | Move `TemplateAssertions` to their own file | eileencodes | 2015-05-09 | 2 | -188/+189 | |
| |/ |/| | | | | | | | | | | | | | | | | | | | This moves `TemplateAssertions` out of the `test_case.rb` file and into it's own `template_assertions` file. It still inherits from `ActionController`.` This is in preparation for combining the code for Integration tests and Controller tests. This will need to be it's own file to be added to the `requires` for Integration tests. This does not currently change ANY behavior, just moving it for access later on. | |||||
* | | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2015-05-08 | 1 | -1/+1 | |
|\ \ | ||||||
| * | | Add missing "of" to RequestForgeryProtection doc. | Hendy Tanata | 2015-04-27 | 1 | -1/+1 | |
| | | | | | | | | | | | | [ci skip] | |||||
* | | | Give authentication methods the ability to customize response message. | Keenan Brock | 2015-05-03 | 1 | -14/+16 | |
| | | | | | | | | | | | | Digest allowed the messages. Add the same feature to basic and token | |||||
* | | | Use `args` instead of `*args` in `kwargs_request?` method | eileencodes | 2015-05-02 | 1 | -3/+3 | |
| | | | | | | | | | | | | | | | | | | `*args` is not required here and should be avoided when not necessary because `*args` are slower than `args` and create unnecessary array allocations. | |||||
* | | | Updated request_forgery_protection docs [ci skip] | Prathamesh Sonpatki | 2015-04-28 | 1 | -5/+6 | |
| |/ |/| | | | | | | | | | - Changed Javascript to JavaScript. - Added full-stop which was missing, also wrapped the sentence to 80 chars. - Changed proc to Proc and oauth to OAuth. | |||||
* | | remove extra `=` in form builder example [ci skip] | yuuji.yaginuma | 2015-04-15 | 1 | -1/+1 | |
| | | ||||||
* | | Merge pull request #19757 from Strech/http-auth-realm-unquoting | Arthur Nogueira Neves | 2015-04-14 | 1 | -2/+2 | |
|\ \ | | | | | | | Tiny optimization of http auth Realm unquoting | |||||
| * | | Tiny optimization of http auth Realm unquoting | Strech (Sergey Fedorov) | 2015-04-14 | 1 | -2/+2 | |
| |/ | ||||||
* | | Fix typos in the documentation [ci skip] | Rafael Mendonça França | 2015-04-14 | 1 | -1/+1 | |
| | | ||||||
* | | Merge pull request #19736 from kmcphillips/master | Rafael Mendonça França | 2015-04-14 | 2 | -0/+49 | |
|\ \ | |/ |/| | Set default form builder for a controller | |||||
| * | Override default form builder for a controller | Kevin McPhillips | 2015-04-13 | 2 | -0/+49 | |
| | | ||||||
* | | Add note regarding CSRF for APIs, as a use-case for skipping it [ci skip] | Zachary Scott | 2015-04-12 | 1 | -0/+4 | |
| | | ||||||
* | | Apply comments from @jeremy regarding why HTML and Javascript requests | Zachary Scott | 2015-04-12 | 1 | -0/+5 | |
| | | | | | | | | | | | | specifically are checked for CSRF, when dealing with the browser. [ci skip] |