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
Comments
Comment #1
x.algorithm CreditAttribution: x.algorithm commentedI'll work on this
Comment #2
x.algorithm CreditAttribution: x.algorithm commentedI made changes requested.
I edited 4 files.
I created a view utilising articles.
My format was table. I made a sort on title.
It rendered successfully so I believe it should be ready
Please review
Comment #3
oenie CreditAttribution: oenie commentedAdd public access modifier in front of the functions to adher to the new OOP standards.
Comment #4
nathangervais CreditAttribution: nathangervais commentedRerolled Patch to only include buildSortPost with public modifier.
Comment #6
aspilicious CreditAttribution: aspilicious commentedIndentation is incorrect :s
Comment #7
chrisguindon CreditAttribution: chrisguindon commentedReroll of #4 + with 2 spaces before public function buildSortPost() { }.
Comment #9
aspilicious CreditAttribution: aspilicious commentedLook at the first oatches, you're missing some changes
Comment #10
suhel.rangnekar CreditAttribution: suhel.rangnekar commentedCovered all build_sort_post().
Comment #11
oenie CreditAttribution: oenie commentedLooks good to me now.
Comment #12
alexpottCommitted c3ac610 and pushed to 8.x. Thanks!