Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Move the direct_upload: true convenience option from the activestorage ↵ | David Heinemeier Hansson | 2017-07-31 | 2 | -2/+10 |
| | | | | helper into actionview | ||||
* | Fix testing helpers that use Action View's capturing helpers (e.g. content_for) | George Claghorn | 2017-07-25 | 1 | -0/+2 |
| | |||||
* | Suppress warning from actionview controller helper. | Christina Thompson | 2017-07-24 | 1 | -1/+1 |
| | | | | | | Explicitly pass a destructed array to the delegate method. Signed-off-by: Yuki Nishijima <yk.nishijima@gmail.com> | ||||
* | Merge pull request #29897 from kirs/frozen-actionview | Rafael França | 2017-07-24 | 99 | -1/+196 |
|\ | | | | | Use frozen string literal in actionview/ | ||||
| * | Use frozen string literal in actionview/ | Kir Shatrov | 2017-07-24 | 99 | -1/+196 |
| | | |||||
* | | Merge pull request #29792 from lugray/delegate_respond_to | Rafael França | 2017-07-24 | 1 | -2/+10 |
|\ \ | |/ |/| | Delegate respond_to? in ActionView::Helpers::ControllerHelper | ||||
| * | Delegate respond_to? in ActionView::Helpers::ControllerHelper | Lisa Ugray | 2017-07-19 | 1 | -2/+10 |
| | | | | | | | | | | | | Since methods defined in the controller helper are mostly delegated to the controller, delegate respond_to? as well, so that for example `respond_to?(:params)` behaves as expected. | ||||
* | | Make actionview ready for frozen strings | Kir Shatrov | 2017-07-23 | 1 | -1/+2 |
| | | |||||
* | | Merge pull request #29884 from padi/update_rails_5_upgrade_guide | Rafael França | 2017-07-21 | 1 | -2/+2 |
|\ \ | |/ |/| | Updates Rails upgrade guide on `ActionView::Helpers::RecordTagHelper` | ||||
| * | Suppress documentation for content_tag_for and div_for since they were ↵ | Marc Rendl Ignacio | 2017-07-21 | 1 | -2/+2 |
| | | | | | | | | removed already | ||||
* | | [ci skip]Replace jquery-ujs with rails-ujs | willnet | 2017-07-19 | 1 | -1/+1 |
|/ | |||||
* | [Action View] `rubocop -a --only Layout/EmptyLineAfterMagicComment` | Koichi ITO | 2017-07-11 | 2 | -0/+2 |
| | |||||
* | Prepare AP and AR to be frozen string friendly | Kir Shatrov | 2017-07-06 | 2 | -1/+3 |
| | |||||
* | Merge branch 'master' into require_relative_2017 | Xavier Noria | 2017-07-02 | 9 | -15/+29 |
|\ | |||||
| * | Avoid shadowed variable | Matthew Draper | 2017-07-02 | 1 | -2/+2 |
| | | |||||
| * | Revert "Merge pull request #29540 from kirs/rubocop-frozen-string" | Matthew Draper | 2017-07-02 | 101 | -101/+1 |
| | | | | | | | | | | This reverts commit 3420a14590c0e6915d8b6c242887f74adb4120f9, reversing changes made to afb66a5a598ce4ac74ad84b125a5abf046dcf5aa. | ||||
| * | Merge pull request #29540 from kirs/rubocop-frozen-string | Matthew Draper | 2017-07-02 | 101 | -1/+101 |
| |\ | | | | | | | | | | Enforce frozen string in Rubocop | ||||
| | * | Enforce frozen string in Rubocop | Kir Shatrov | 2017-07-01 | 101 | -1/+101 |
| | | | |||||
| * | | Merge pull request #29506 from pat/frozen-string-literals | Matthew Draper | 2017-07-02 | 8 | -13/+13 |
| |\ \ | | | | | | | | | | | | | Make ActiveSupport frozen-string-literal friendly. | ||||
| | * | | Make ActionView frozen string literal friendly. | Pat Allan | 2017-06-20 | 6 | -8/+8 |
| | | | | | | | | | | | | | | | | Plus a couple of related ActionPack patches. | ||||
| | * | | Make ActionMailer frozen string literal friendly. | Pat Allan | 2017-06-20 | 2 | -3/+3 |
| | | | | |||||
| | * | | Make ActiveModel frozen string literal friendly. | Pat Allan | 2017-06-20 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | | Includes two external changes because they're referenced within the ActiveModel test suite. | ||||
| * | | | Merge pull request #29349 from robertomiranda/responsive-images | Matthew Draper | 2017-07-02 | 1 | -2/+16 |
| |\ \ \ | | |_|/ | |/| | | | | | | Add srcset option to image_tag helper | ||||
| | * | | Add `srcset` option to `image_tag` helper | Roberto Miranda | 2017-06-29 | 1 | -1/+15 |
| |/ / | |||||
* / / | [Action View] require => require_relative | Akira Matsuda | 2017-07-01 | 24 | -34/+34 |
|/ / | |||||
* / | Fix current_page? helper issue with engine root path | savroff | 2017-06-19 | 1 | -1/+4 |
|/ | |||||
* | Generate field ids in `collection_check_boxes` and `collection_radio_buttons` | yuuji.yaginuma | 2017-06-11 | 2 | -0/+2 |
| | | | | | This makes sure that the labels are linked up with the fields. Fixes #29014 | ||||
* | Don't rely on the @view_renderer being defined. | Kasper Timm Hansen | 2017-06-08 | 1 | -2/+2 |
| | | | | That won't be true for Action Pack and Action Mailer. | ||||
* | Merge pull request https://github.com/rails/rails/pull/28637 from ↵ | Kasper Timm Hansen | 2017-06-08 | 4 | -10/+11 |
| | | | | st0012/fix-partial-cache-logging | ||||
* | Use a hash to record every partial's cache hit status instead of sharing a ↵ | Stan Lo | 2017-06-08 | 3 | -3/+6 |
| | | | | boolean. | ||||
* | Move slicing to initializer. | Kasper Timm Hansen | 2017-06-07 | 2 | -2/+2 |
| | | | | | | Forgot all about https://github.com/rails/rails/pull/28844/files#r113780934 cc @rafaelfranca | ||||
* | Use mattr_accessor default: option throughout the project | Genadi Samokovarov | 2017-06-03 | 6 | -23/+12 |
| | |||||
* | Merge pull request #29191 from ↵ | Kasper Timm Hansen | 2017-05-29 | 1 | -1/+1 |
|\ | | | | | | | | | bogdanvlviv/pass_params_filename_lineno_to_class_eval Pass params __FILE__ and __LINE__ + 1 if class_eval with << | ||||
| * | Pass params __FILE__ and __LINE__ + 1 if class_eval with << | bogdanvlviv | 2017-05-29 | 1 | -1/+1 |
| | | |||||
* | | Add option for class_attribute default (#29270) | David Heinemeier Hansson | 2017-05-29 | 5 | -24/+17 |
| | | | | | | | | | | | | | | | | | | | | | | | | * Allow a default value to be declared for class_attribute * Convert to using class_attribute default rather than explicit setter * Removed instance_accessor option by mistake * False is a valid default value * Documentation | ||||
* | | Merge pull request #29119 from spohlenz/fix-select-with-enumerable | Matthew Draper | 2017-05-24 | 1 | -1/+1 |
|\ \ | |/ |/| | Fix select tag helper used with Enumerable choices | ||||
| * | Fix select tag helper used with Enumerable choices | Sam Pohlenz | 2017-05-17 | 1 | -1/+1 |
| | | | | | | | | | | | | Allows a custom object implementing Enumerable to be used as the choices parameter for a select tag, which previously wasn't possible due to the call to `empty?` on the choices (which isn't implemented on Enumerable). | ||||
* | | Add :json type to auto_discovery_link_tag | Mike Gunderloy | 2017-05-20 | 1 | -5/+7 |
| | | | | | | | | | | This allows auto_discovery_link_tag to support the JSON Feed standard. See https://jsonfeed.org/version/1 for more information. | ||||
* | | Use recyclable cache keys (#29092) | David Heinemeier Hansson | 2017-05-18 | 2 | -10/+18 |
|/ | |||||
* | Merge pull request #20701 from iamvery/date-helper-argument-error | Andrew White | 2017-04-27 | 1 | -18/+26 |
|\ | | | | | Ensure input to distance_of_time_in_words is not nil | ||||
| * | Ensure input to distance_of_time_in_words is not nil | Jay Hayes | 2017-04-27 | 1 | -18/+26 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Internally all input is converted to time so that it can be treated uniformly. Remove now-unneeded condition * Now that all input is treated is converted to time, we no longer need to type check it. Rename variables to clarify their purpose Extract private method to normalize distance_of_time args to time Update actionview changelog | ||||
* | | Merge pull request #28844 from ↵ | Kasper Timm Hansen | 2017-04-27 | 1 | -1/+1 |
|\ \ | | | | | | | | | | | | | kyuden/remove_unnecessary_attributes_of_select_in_form_with Remove unnecessary `skip_default_ids` and `allow_method_names_outside_object` attributes of select tag in `form_with` | ||||
| * | | Remove unnecessary `skip_default_ids` and ↵ | kyuden | 2017-04-24 | 1 | -1/+1 |
| |/ | | | | | | | `allow_method_names_outside_object` attributes of select tag in `form_with` | ||||
* / | Fix `current_page?` regression: | Edouard CHIN | 2017-04-26 | 1 | -1/+1 |
|/ | | | | | | - `check_parameters` kwargs was added to the `current_page?` method, the implementation was assuming only hashes responds to `delete`. This was causing issues when `current_page?` was called with a Active Model object - ref https://github.com/rails/rails/pull/27549 - Fixes #28846 | ||||
* | Set to `form_with_generates_remote_forms` only when config is explicitly ↵ | yuuji.yaginuma | 2017-04-22 | 1 | -2/+4 |
| | | | | | | | | | specified Without this check, even if config is not specified, `ActionView::Helpers::FormHelper.form_with_generates_remote_forms` always be set to nil and remote form not be generated. Follow up to 128b804c6ce40fcbde744f294f8cb98654f6efec | ||||
* | Configure form_with_generates_remote_forms in its own initializer | Rafael Mendonça França | 2017-04-21 | 1 | -0/+7 |
| | | | | | | | | | This configuration is not present in ActionView::Base so we can't let the action_view.set_configs initializer set it. Also add tests to make sure this config works. Fixes #28824 | ||||
* | Reuse the Parameters#to_h check in the routing helpers | Rafael Mendonça França | 2017-04-18 | 1 | -5/+0 |
| | | | | | Since this protection is now in Parameters we can use it instead of reimplementing again. | ||||
* | Add `form_with_generates_remote_forms` config. | Kasper Timm Hansen | 2017-04-16 | 1 | -1/+3 |
| | | | | | | | Allows users to not have remote forms by default, since there's more JS harness, e.g. bundling rails-ujs, otherwise. Also don't skip creating defaults file anymore. Sprockets isn't the only new config. | ||||
* | Default embed_authenticity_token_in_remote_forms to nil. | Kasper Timm Hansen | 2017-04-16 | 3 | -8/+10 |
| | | | | | | | Effectively treat nil values as "auto", e.g. whatever a form helper chooses to interpret it as. But treat an explicitly assigned false value as disabling. | ||||
* | Fix module name [ci skip] | yuuji.yaginuma | 2017-04-15 | 1 | -10/+10 |
| | | | | Replace `FormOptionHelper` to `FormOptionsHelper`. |