Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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] | ||||
* | | update request_forgery_protection docs [ci skip] | Vladimir Lyzo | 2015-04-12 | 1 | -7/+8 |
| | | |||||
* | | Revert "Merge pull request #19682 from ↵ | Santiago Pastorino | 2015-04-12 | 2 | -6/+3 |
|/ | | | | | | | supercaracal/fix_force_ssl_redirection_flash_error" This reverts commit d215620340be7cb29e2aa87aab22da5ec9e6e6a7, reversing changes made to bbbbfe1ac02162ecb5e9a7b560134a3221f129f3. | ||||
* | fix fails to force_ssl_redirection if session_store is disabled | Taishi Kasuga | 2015-04-09 | 2 | -3/+6 |
| | |||||
* | Merge pull request #19666 from mikej/master | Santiago Pastorino | 2015-04-06 | 1 | -1/+1 |
|\ | | | | | fix missing "if" in API docs for ActionController::Parameters#permit | ||||
| * | fix missing "if" in API docs for ActionController::Parameters#permit | Michael Josephson | 2015-04-06 | 1 | -1/+1 |
| | | |||||
* | | Fix ActionPack tests after changes to missing template logger | eileencodes | 2015-04-06 | 1 | -1/+1 |
|/ | | | | | | | | | After merging #19377 ActionPack tests were missing a require for `ActiveSupport::LogSubscriber::TestHelper` and change didn't take into account that logger could be nil. Added the require and only log to info if logger exists. This wasn't caught earlier because these tests only run after a merge. | ||||
* | head no_content when there is no template or action performed | Stephen Bussey | 2015-04-05 | 1 | -1/+6 |
| | |||||
* | Fix a few typos [ci skip] | Robin Dupret | 2015-04-05 | 1 | -3/+3 |
| | |||||
* | Freeze static arguments for gsub | brainopia | 2015-04-02 | 2 | -3/+3 |
| | |||||
* | Prefer string patterns for gsub | brainopia | 2015-04-02 | 2 | -3/+3 |
| | | | | | | | | | | | | | | | | | https://github.com/ruby/ruby/pull/579 - there is a new optimization since ruby 2.2 Previously regexp patterns were faster (since a string was converted to regexp underneath anyway). But now string patterns are faster and better reflect the purpose. Benchmark.ips do |bm| bm.report('regexp') { 'this is ::a random string'.gsub(/::/, '/') } bm.report('string') { 'this is ::a random string'.gsub('::', '/') } bm.compare! end # string: 753724.4 i/s # regexp: 501443.1 i/s - 1.50x slower | ||||
* | Merge pull request #19544 from shuhei/fix-parameters-const-missing | Xavier Noria | 2015-03-28 | 1 | -1/+1 |
|\ | | | | | Return super in ActionController::Parameters.const_missing |