Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #20329 from EduardoBautista/json-api-support | Rafael Mendonça França | 2015-05-27 | 1 | -1/+1 |
|\ | | | | | Add application/vnd.api+json alias to the JSON MIME Type. | ||||
| * | Add application/vnd.api+json alias to the JSON MIME Type. | Eduardo Bautista | 2015-05-27 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #19808 from byroot/action-parameter | Rafael Mendonça França | 2015-05-27 | 2 | -15/+32 |
|\ \ | | | | | | | [PoC] Stop shadowing parameters named `action` | ||||
| * | | Allow to parameters named `action` or `controller` from AC::TestCase helpers | Jean Boussier | 2015-04-18 | 2 | -15/+32 |
| | | | |||||
* | | | Merge pull request #20316 from ↵ | Rafael Mendonça França | 2015-05-27 | 1 | -0/+1 |
|\ \ \ | | | | | | | | | | | | | | | | | vngrs/add_assertion_for_get_method_into_request_test Add assertion for get? method into test cases | ||||
| * | | | Add assertion for get? method into test cases | Mehmet Emin İNAÇ | 2015-05-27 | 1 | -0/+1 |
| | | | | |||||
* | | | | Documentation for ActionDispatch::Request form_data? method [ci skip] | Mehmet Emin İNAÇ | 2015-05-27 | 1 | -0/+2 |
|/ / / | |||||
* | | | Merge pull request #20262 from arunagw/aa-remove-broken-unused-release-task | Yves Senn | 2015-05-27 | 1 | -7/+0 |
|\ \ \ | | | | | | | | | Remove broken and unused release task | ||||
| * | | | Remove broken and unused release task | Arun Agrawal | 2015-05-22 | 1 | -7/+0 |
| | | | | | | | | | | | | | | | | | | | | - We do release with release.rb - There is no `rake/gemcutter` | ||||
* | | | | Add missing nodocs and docs for ActionDispatch::Request [ci skip] | Mehmet Emin İNAÇ | 2015-05-26 | 1 | -3/+5 |
| | | | | | | | | | | | | | | | | add missing dot to end of the doc | ||||
* | | | | Use memoization while accessing request headers for minimizing memory usage | Mehmet Emin İNAÇ | 2015-05-25 | 1 | -1/+1 |
| |_|/ |/| | | |||||
* | | | remove useless ivar | Aaron Patterson | 2015-05-23 | 1 | -1/+0 |
| | | | | | | | | | | | | I should have deleted this earlier with 42e66fac38b54dd53d062fb5d3376218ed2ffdae | ||||
* | | | add a branch to eliminate multiple nil checks | Aaron Patterson | 2015-05-23 | 1 | -3/+5 |
| | | | | | | | | | | | | | | | if we add an else conditional to the `presence` check, we can eliminate the second `||` branch in the caller | ||||
* | | | move request id manipulation to the request object | Aaron Patterson | 2015-05-22 | 2 | -7/+18 |
| | | | | | | | | | | | | this way we can keep the knowledge of `env` hash keys in one place. | ||||
* | | | Spelling/typo/grammatical fixes [ci skip] | karanarora | 2015-05-23 | 6 | -7/+7 |
|/ / | | | | | | | | | | | | | | | | | | | spelling fix [ci skip] example to be consistent [ci skip] grammatical fix typo fixes [ci skip] | ||||
* | | Merge pull request #17788 from sivagollapalli/master | Arthur Nogueira Neves | 2015-05-20 | 1 | -0/+5 |
|\ \ | | | | | | | Issue#17703 Test case for tempfile attribute | ||||
| * | | Issue#17703 Test case for tempfile attribute | Siva Gollapalli | 2015-04-16 | 1 | -0/+5 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | + To avoid regression I am adding this test case on action pack Changed to assert from assert_equal + Added msg to assertion Modified msg | ||||
* | | | Prefer assert_not over refute | Rafael Mendonça França | 2015-05-18 | 2 | -2/+2 |
| | | | |||||
* | | | ActionDispatch::Journey::Routes#empty? test cases | Valentine Valyaeff | 2015-05-19 | 3 | -1/+17 |
| | | | |||||
* | | | Added ActionDispatch::Journey::Routes#empty? | juggernaut- | 2015-05-18 | 1 | -0/+4 |
| | | | |||||
* | | | [ci skip] remove `assigns` from the integration test example in API docs | Roque Pinel | 2015-05-17 | 1 | -2/+2 |
| | | | | | | | | | | | | Based on #19976 and #18305. | ||||
* | | | Pass over CHANGELOGS [ci skip] | Prathamesh Sonpatki | 2015-05-16 | 1 | -1/+1 |
| | | | |||||
* | | | Merge pull request #20113 from claudiob/remove-rails31-refs | Yves Senn | 2015-05-14 | 2 | -5/+4 |
|\ \ \ | | | | | | | | | [ci skip] Remove comments about Rails 3.1 | ||||
| * | | | [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. | ||||
* | | | | [ci skip] Don’t encourage `sudo gem install` | claudiob | 2015-05-12 | 1 | -1/+1 |
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I think we are better off leaving `sudo` outside of the documented way of installing gems (`activerecord`, `actionpack`, …). We don’t want newbies to think that `sudo` is required or, even worse, than they actually have to type `[sudo] gem install`. In most scenarios, `sudo` is not needed to install gems, and people who do need it, probably already know about it. What do you think? :grin: | ||||
* | | | 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] | ||||
* | | | | [skip ci] Fix typos in actionpack changelog and security guide | Anton Davydov | 2015-05-07 | 1 | -1/+1 |
| | | | | |||||
* | | | | Silence ambiguous first argument warning | Prem Sichanugrist | 2015-05-04 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This silences: actionpack/test/journey/route_test.rb:33: warning: ambiguous first argument; put parentheses or a space even after `/' operator | ||||
* | | | | Tiny documentation edits [ci skip] | Robin Dupret | 2015-05-04 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | * Fix a few typos * Wrap lines to 80 chars * Use `+` instead of `<tt>` | ||||
* | | | | Give authentication methods the ability to customize response message. | Keenan Brock | 2015-05-03 | 3 | -20/+22 |
| | | | | | | | | | | | | | | | | Digest allowed the messages. Add the same feature to basic and token | ||||
* | | | | Use ruby 1.9 lambda syntax in documentations [ci skip] | Mehmet Emin İNAÇ | 2015-05-03 | 3 | -6/+6 |
| | | | | |||||
* | | | | Use `args` instead of `*args` in `kwargs_request?` method | eileencodes | 2015-05-02 | 2 | -6/+6 |
| | | | | | | | | | | | | | | | | | | | | | | | | `*args` is not required here and should be avoided when not necessary because `*args` are slower than `args` and create unnecessary array allocations. | ||||
* | | | | Document :tld_length option for cookies. | Ryan Wallace | 2015-04-29 | 1 | -0/+3 |
| | | | | |||||
* | | | | 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. | ||||
* | | | | Add changelog for rake routes default fix | Arthur Neves | 2015-04-27 | 1 | -0/+7 |
| | | | | | | | | | | | | | | | | [see #18392] | ||||
* | | | | Merge pull request #18392 from brainopia/fix_route_requirements | Arthur Nogueira Neves | 2015-04-27 | 2 | -1/+9 |
|\ \ \ \ | | | | | | | | | | | Correct route requirements by overriding defaultls | ||||
| * | | | | Correct route requirements by overriding defaultls (fixes #18373) | brainopia | 2015-01-08 | 2 | -1/+9 |
| | | | | | |||||
* | | | | | Merge pull request #19904 from zzak/rm_route_wrapper_internal_hacks | Rafael Mendonça França | 2015-04-26 | 1 | -0/+16 |
|\ \ \ \ \ | | | | | | | | | | | | | Rm route wrapper internal hacks | ||||
| * | | | | | Remove internal hacks dependent on Sprockets from RouteWrapper | Zachary Scott | 2015-04-25 | 1 | -5/+1 |
| | | | | | | |||||
| * | | | | | Fix rake routes for api apps | Jorge Bejar | 2015-04-25 | 2 | -1/+21 |
| | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | | | | | | Don't reference sprockets assets on action pack | Arthur Neves | 2015-04-26 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We need to ignore the `assets_prefix` when running a command like `rake routes`. However we cannot reference asserts_prefix from action_pack as that is a sprockets-rails concern. See this is now implemented on sprockets-rails https://github.com/rails/sprockets-rails/commit/85b89c44ad40af3056899808475e6e4bf65c1f5a | ||||
* | | | | | | Removed unused code from request_forgery_protection tests | Prathamesh Sonpatki | 2015-04-26 | 1 | -17/+0 |
|/ / / / / | |||||
* | | | | | No need to capitalize verbs. [CI SKIP] | Guo Xiang Tan | 2015-04-26 | 1 | -2/+2 |
| | | | | | |||||
* | | | | | Why do we add a top-level constant here? | Akira Matsuda | 2015-04-25 | 1 | -5/+3 |
| | | | | | |||||
* | | | | | Merge pull request #19879 from bboe/RedirectTestCleanup | Rafael Mendonça França | 2015-04-23 | 1 | -3/+0 |
|\ \ \ \ \ | | | | | | | | | | | | | Remove unused WorkshopsController class in redirect_test. | ||||
| * | | | | | Remove unused WorkshopsController class in redirect_test. | Bryce Boe | 2015-04-23 | 1 | -3/+0 |
| | | | | | | |||||
* | | | | | | Add nodoc to some private constants [ci skip] | Rafael Mendonça França | 2015-04-23 | 1 | -3/+3 |
| | | | | | | |||||
* | | | | | | Merge pull request #19823 from sbhatore/doc_fix_1 | Rafael Mendonça França | 2015-04-23 | 1 | -0/+8 |
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | [ci skip] Description inside Signed and Encrypted CookieJars added |