Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Part of meta-issue #1856630: [Change notice] [META] Rename Views methods to core standards
Comment | File | Size | Author |
---|---|---|---|
#13 | drupal-views_method_rename-2002462-13.patch | 1.08 KB | danylevskyi |
#9 | views-method_rename-2002462-9.patch | 16.61 KB | danylevskyi |
#5 | Rename_Views_method-2002462-5.patch | 16.79 KB | NonProfit |
#2 | Rename_Views_method-2002462-2.patch | 13.35 KB | NonProfit |
Comments
Comment #1
NonProfit CreditAttribution: NonProfit commentedComment #2
NonProfit CreditAttribution: NonProfit commentedPatch created.
Comment #3
NonProfit CreditAttribution: NonProfit commentedNeed to reroll. default_summary_form() improperly changed.
Comment #4
oenie CreditAttribution: oenie commentedAdd public access modifier in front of the function to adher to the new OOP standards.
Comment #5
NonProfit CreditAttribution: NonProfit commentedPatch created.
Comment #6
heddnI think need to be protected.
Also needs a re-roll:
$ git apply --index Rename_Views_method-2002462-5.patch
error: patch failed: core/modules/views/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php:462
error: core/modules/views/lib/Drupal/views/Plugin/views/argument/ArgumentPluginBase.php: patch does not apply
error: patch failed: core/modules/views/lib/Drupal/views/ViewExecutable.php:353
error: core/modules/views/lib/Drupal/views/ViewExecutable.php: patch does not apply
error: patch failed: core/modules/views/tests/views_test_config/test_views/views.view.test_aggregate_count.yml:14
error: core/modules/views/tests/views_test_config/test_views/views.view.test_aggregate_count.yml: patch does not apply
error: patch failed: core/modules/views/tests/views_test_config/test_views/views.view.test_destroy.yml:34
error: core/modules/views/tests/views_test_config/test_views/views.view.test_destroy.yml: patch does not apply
Comment #7
danylevskyiI'd like to take it. #2009672: [META] CodeSprint UA 2013
Comment #8
danylevskyiComment #9
danylevskyiComment #10
danylevskyiComment #11
dawehnerThis change is fine.
Please skip this chage.
the method should be protected.
This ID of the plugin should be kept, so please skip all the other changes in all the files.
Comment #12
danylevskyiComment #13
danylevskyiComment #14
podarok#13 looks good
RTBC
Comment #15
webchickCommitted and pushed to 8.x. Thanks!
Comment #16
webchickOops.
Comment #17
xjmWe'll create a single change notice in #1856630: [Change notice] [META] Rename Views methods to core standards.