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 |
---|---|---|---|
#7 | views-core-rename-element_classes-2002468-07.patch | 1.72 KB | SpartyDan |
#4 | rename_method_elementClasses-2002468-04.patch | 1.71 KB | FredericoT |
#2 | 0001-changing-element_classes-to-camelcase.patch | 2.1 KB | mikedotexe |
Comments
Comment #1
mikedotexe CreditAttribution: mikedotexe commentedWorking on this in Portland as a first patch with the mentors provided here
Comment #2
mikedotexe CreditAttribution: mikedotexe commentedThere were only 3 changes to make, so I'm marking it as fixed
Comment #3
oenie CreditAttribution: oenie commentedAdd public access modifier in front of the function to adher to the new OOP standards.
Comment #4
FredericoT CreditAttribution: FredericoT commentedComment #5
oenie CreditAttribution: oenie commentedLooks fine to me now ! Once the patch passed testbot, it's RTBC for me.
Comment #6
oenie CreditAttribution: oenie commentedHate to do this ... Unfortunately we need a reroll, the patch doesn't apply anymore.
Comment #7
SpartyDan CreditAttribution: SpartyDan commentedre-roll
Comment #8
dawehnerPerfect, there are non other instances available.
Comment #9
alexpottCommitted cd486a6 and pushed to 8.x. Thanks!