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 |
---|---|---|---|
#8 | 2002446-8.patch | 1.05 KB | jibran |
#8 | interdiff.txt | 650 bytes | jibran |
#6 | 2002446-rename-default-default-6.patch | 1.04 KB | fmizzell |
#3 | 2002446-rename-default-default-3.patch | 1.03 KB | fmizzell |
#2 | 2002446-rename-default-default.patch | 1.02 KB | fmizzell |
Comments
Comment #1
fmizzell CreditAttribution: fmizzell commentedComment #2
fmizzell CreditAttribution: fmizzell commentedComment #3
fmizzell CreditAttribution: fmizzell commentedsomeone made me aware of the need for 'public' in front of the method as part of our coding standards.
Comment #5
heddnFailing tests. Please test and re-roll.
Comment #6
fmizzell CreditAttribution: fmizzell commentedComment #7
aspilicious CreditAttribution: aspilicious commentedShould be protected in stead of public
Comment #8
jibranConverted to protected.
Comment #9
dawehnerNice!
Comment #10
alexpottCommitted cad23a5 and pushed to 8.x. Thanks!