Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Speed up `normalize_keys` by removing dup step. | Kasper Timm Hansen | 2015-01-22 | 1 | -9/+6 |
| | | | | | | | | Previously env was duplicated and then had it's keys mutated. This iterates through the hash twice. Using `transform_keys`, duplication and key mutation is a single iteration. `convert_symbols` was renamed to `http_header_format`. | ||||
* | Merge pull request #18546 from brainopia/action_view_render | Rafael Mendonça França | 2015-01-22 | 5 | -4/+129 |
|\ | | | | | A shortcut to setup controller environment | ||||
| * | Add ActionController::Base.render | brainopia | 2015-01-22 | 1 | -0/+3 |
| | | |||||
| * | Add ActionController::Renderer | brainopia | 2015-01-22 | 3 | -0/+112 |
| | | | | | | | | Render arbitrary templates outside of controller actions | ||||
| * | Add ActionController#build_with_env | brainopia | 2015-01-22 | 1 | -0/+6 |
| | | | | | | | | | | To have an easier way to setup a controller instance with custom environment | ||||
| * | Add `ActionController::Metal#set_request!` | brainopia | 2015-01-21 | 2 | -4/+8 |
| | | | | | | | | | | Add `ActionController::Metal#set_request!` to set a request on controller instance without calling dispatch. | ||||
* | | Merge pull request #18483 from rono23/fix-name_for_action-in-routing | Santiago Pastorino | 2015-01-19 | 1 | -2/+3 |
|\ \ | |/ |/| | Fix name_for_action in routing | ||||
| * | Fix name_for_action in routing | rono23 | 2015-01-19 | 1 | -2/+3 |
| | | |||||
* | | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2015-01-15 | 1 | -2/+2 |
|\ \ | |||||
| * | | Better examples for fresh_when and stale? | claudiob | 2015-01-06 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is clearer and closer to reality to use `@article.updated_at` as the `:last_modified` parameter of `fresh_when` and `stale?`. Using `@article.created_at` would result in the cache never expiring, since the creation timestamp never changes. [ci skip] | ||||
* | | | Extract `Date` header to string constant in Http Response, similar to other ↵ | Vipul A M | 2015-01-14 | 1 | -3/+4 |
| |/ |/| | | | | | headers | ||||
* | | remove autoload HideActions, no longer exist | yuuji.yaginuma | 2015-01-11 | 1 | -1/+0 |
| | | |||||
* | | Default headers, removed in controller actions, will not be reapplied to the ↵ | Jonas Baumann | 2015-01-09 | 1 | -0/+7 |
| | | | | | | | | test response. | ||||
* | | Deprecate all *_filter callbacks in favor of *_action callbacks | Abdelkader Boudih | 2015-01-08 | 1 | -5/+27 |
| | | |||||
* | | Merge pull request #18404 from claudiob/rebase-14549 | Rafael Mendonça França | 2015-01-08 | 1 | -0/+10 |
|\ \ | | | | | | | Add test case and documentation for skip_before_filter. | ||||
| * | | Add test/doc for :if/:except in skip_before_action | claudiob | 2015-01-08 | 1 | -0/+5 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new test/docs further explain the conflicts that can happen when mixing `:if`/`:unless` options with `:only`/`:except` options in `skip_before_action`. The gist is that "positive" filters always have priority over negative ones. The previous commit already showed that `:only` has priority over `:if`. This commit shows that `:if` has priority over `:except`. For instance, the following snippets are equivalent: ```ruby skip_before_action :some_callback, if: -> { condition }, except: action ``` ```ruby skip_before_action :some_callback, if: -> { condition } ``` | ||||
| * | | Add test case and documentation for skip_before_filter. | Lauro Caetano | 2015-01-08 | 1 | -0/+5 |
| | | | | | | | | | | | | | | | | | | | | | | | | Test case for using skip_before_filter with the options :only and :if both present. In this case, the :if option will be ignored and :only will be executed. Closes #14549 (the commit was cherry-picked from there). | ||||
* | | | Add prepend option to protect_from_forgery. | Josef Šimánek | 2015-01-08 | 1 | -1/+8 |
| | | | |||||
* | | | Remove ActionController::HideActions (closes #18336) | brainopia | 2015-01-06 | 3 | -55/+2 |
|/ / | |||||
* | | Changing sets -> set | Lindsey Bieda | 2015-01-06 | 1 | -1/+1 |
| | | | | | | Updating some minor grammar issue. | ||||
* | | Improve protect_from_forgery documentation. [ci skip]. | Josef Šimánek | 2015-01-06 | 1 | -3/+3 |
| | | |||||
* | | Merge pull request #18341 from aditya-kapoor/remove-unneeded-checks | Rafael Mendonça França | 2015-01-05 | 1 | -2/+1 |
|\ \ | | | | | | | remove unneeded check since /_one_time_conditions/ is not present anymore. | ||||
| * | | remove unneeded check since /_one_time_conditions/ is removed | Aditya Kapoor | 2015-01-05 | 1 | -2/+1 |
| | | | |||||
* | | | Reset template assertions without warnings | brainopia | 2015-01-05 | 1 | -2/+4 |
|/ / | |||||
* | | Document all options for protect_from_forgery. | Josef Šimánek | 2015-01-04 | 1 | -8/+2 |
| | | | | | | | | [ci skip] | ||||
* | | Remove unused require | Carlos Antonio da Silva | 2015-01-04 | 1 | -1/+0 |
| | | | | | | | | | | This was used by the respond_to/respond_with implementation on this file, which is now extracted to the responders gem. | ||||
* | | Remove respond_to/respond_with placeholder methods | Carlos Antonio da Silva | 2015-01-04 | 1 | -20/+0 |
| | | | | | | | | This functionality has been extracted to the responders gem. | ||||
* | | Remove Struct#to_h backport | Rafael Mendonça França | 2015-01-04 | 1 | -1/+0 |
| | | |||||
* | | Remove some comments about Ruby 1.9 behaviors | Rafael Mendonça França | 2015-01-04 | 1 | -1/+1 |
| | | |||||
* | | Remove unneeded requires | Rafael Mendonça França | 2015-01-04 | 3 | -3/+0 |
| | | | | | | | | These requires were added only to change deprecation message | ||||
* | | Remove deprecated assertion files | Rafael Mendonça França | 2015-01-04 | 3 | -9/+0 |
| | | |||||
* | | Remove deprecated usage of string keys in URL helpers | Rafael Mendonça França | 2015-01-04 | 1 | -17/+1 |
| | | |||||
* | | Remove deprecated `only_path` option on `*_path` helpers | Rafael Mendonça França | 2015-01-04 | 1 | -28/+1 |
| | | |||||
* | | Remove deprecate `*_path` helpers in email views | Rafael Mendonça França | 2015-01-04 | 1 | -25/+4 |
| | | |||||
* | | Remove deprecated `NamedRouteCollection#helpers` | Rafael Mendonça França | 2015-01-04 | 1 | -8/+0 |
| | | |||||
* | | Remove deprecated support to define routes with `:to` option that | Rafael Mendonça França | 2015-01-04 | 1 | -17/+2 |
| | | | | | | | | doesn't contain `#` | ||||
* | | Remove deprecated `ActionDispatch::Response#to_ary` | Rafael Mendonça França | 2015-01-04 | 1 | -15/+0 |
| | | |||||
* | | Remove deprecated `ActionDispatch::Request#deep_munge` | Rafael Mendonça França | 2015-01-04 | 1 | -9/+0 |
| | | |||||
* | | Remove deprecated `ActionDispatch::Http::Parameters#symbolized_path_parameters` | Rafael Mendonça França | 2015-01-04 | 1 | -8/+0 |
| | | |||||
* | | Remove deprecated option `use_route` in controller tests | Rafael Mendonça França | 2015-01-04 | 2 | -38/+1 |
| | | |||||
* | | Merge pull request #17227 from claudiob/explicitly-abort-callbacks | Rafael Mendonça França | 2015-01-03 | 1 | -1/+1 |
|\ \ | | | | | | | | | | | | | | | | | | | Introduce explicit way of halting callback chains by throwing :abort. Deprecate current implicit behavior of halting callback chains by returning `false` in apps ported to Rails 5.0. Completely remove that behavior in brand new Rails 5.0 apps. Conflicts: railties/CHANGELOG.md | ||||
| * | | Throw :abort halts default CallbackChains | claudiob | 2015-01-02 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit changes arguments and default value of CallbackChain's :terminator option. After this commit, Chains of callbacks defined **without** an explicit `:terminator` option will be halted as soon as a `before_` callback throws `:abort`. Chains of callbacks defined **with** a `:terminator` option will maintain their existing behavior of halting as soon as a `before_` callback matches the terminator's expectation. For instance, ActiveModel's callbacks will still halt the chain when a `before_` callback returns `false`. | ||||
* | | | Fix a few typos [ci skip] | Robin Dupret | 2015-01-03 | 2 | -3/+3 |
| |/ |/| | |||||
* | | Merge branch 'master' of github.com:rails/docrails | Vijay Dev | 2015-01-03 | 3 | -18/+24 |
|\ \ | |/ |/| | |||||
| * | Better docs for AbstractController | claudiob | 2014-12-22 | 3 | -18/+24 |
| | | | | | | | | | | | | Fixes internal links, adds examples and set fixed-width fonts. [ci skip] | ||||
* | | Merge pull request #8740 from amatsuda/missing_source_file | Rafael Mendonça França | 2015-01-02 | 1 | -1/+1 |
|\ \ | | | | | | | | | | | | | | | | | | | replace use of MissingSourceFile with LoadError Conflicts: activesupport/test/core_ext/load_error_test.rb | ||||
| * | | replace use of MissingSourceFile with LoadError | Akira Matsuda | 2013-01-04 | 1 | -1/+1 |
| | | | |||||
* | | | Merge pull request #17978 from kommen/fixed-pr-14903 | Rafael Mendonça França | 2015-01-02 | 1 | -4/+7 |
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | Ensure append_info_to_payload is called even if an exception is raised. Conflicts: actionpack/CHANGELOG.md | ||||
| * | | | Ensure append_info_to_payload is called even if an exception is raised. | Dieter Komendera | 2014-12-10 | 1 | -4/+7 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See: * https://github.com/rails/rails/pull/14903 * https://github.com/roidrage/lograge/issues/37 Some code by mxrguspxrt from #14903. | ||||
* | | | | Integration requests should work in contexts without setup and teardown | brainopia | 2015-01-02 | 1 | -1/+2 |
| | | | |