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 |
---|---|---|---|
#11 | views-rename_build_sort-20088-11.patch | 4.56 KB | JeroenT |
#4 | views-rename_build_sort-2002388-4.patch | 5.38 KB | JeroenT |
#2 | views.core-rename_build_sort-2002388-02.patch | 6.18 KB | toddtomlinson |
Comments
Comment #1
toddtomlinson CreditAttribution: toddtomlinson commentedComment #2
toddtomlinson CreditAttribution: toddtomlinson commentedPatch renames build_sort to buildSort.
Comment #3
oenie CreditAttribution: oenie commentedAdd public access modifier in front of the functions to adher to the new OOP standards.
Comment #4
JeroenTComment #5
oenie CreditAttribution: oenie commentedLooks fine to me now ! Once the patch passed testbot, it's RTBC for me.
Comment #7
JeroenT#4: views-rename_build_sort-2002388-4.patch queued for re-testing.
Comment #8
dawehnerSo we rename vars and functions at the same time?
Comment #9
aspilicious CreditAttribution: aspilicious commentedNo that is out of scope :)
Comment #10
somepal CreditAttribution: somepal commentedexcept variables are renamed too!
Comment #11
JeroenTOnly renamed the build_sort function.
Comment #12
dawehnerThank you. Code looks fine, there is no other call and patch applies.
Comment #14
tim.plunkett#11: views-rename_build_sort-20088-11.patch queued for re-testing.
Comment #15
dcam CreditAttribution: dcam commented#11 still applies and was already RTBC in #12.
Comment #16
alexpottCommitted e41c6e1 and pushed to 8.x. Thanks!