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-core-rename-set_distinct-2003288-11.patch | 1011 bytes | SpartyDan |
#5 | Rename_Views_method-2003288-5.patch | 1001 bytes | NonProfit |
Comments
Comment #1
NonProfit CreditAttribution: NonProfit commentedComment #2
phenaproximaComment #3
phenaproximaOops - sorry, NonProfit! I think we both took this on at the same moment. I'll take another issue; didn't mean to step on your toes.
Comment #4
phenaproximaDurr, forgot to unassign from myself.
Comment #5
NonProfit CreditAttribution: NonProfit commentedThanks phenaproxima; no worries.
Comment #7
NonProfit CreditAttribution: NonProfit commentedComment #8
NonProfit CreditAttribution: NonProfit commented#5: Rename_Views_method-2003288-5.patch queued for re-testing.
Comment #9
NonProfit CreditAttribution: NonProfit commentedComment #10
oenie CreditAttribution: oenie commentedAdd protected access modifier in front of the function to adhere to the new OOP standards.
All occurences have been replaced, so after the correction, the patch should be RTBC...
Comment #11
SpartyDan CreditAttribution: SpartyDan commentedAdded protected access modifier in front of the function to adhere to the new OOP standards.
Comment #13
SpartyDan CreditAttribution: SpartyDan commented#11: views-core-rename-set_distinct-2003288-11.patch queued for re-testing.
Comment #14
oenie CreditAttribution: oenie commentedLooks good to me now !
Comment #15
alexpottCommitted 42af4a5 and pushed to 8.x. Thanks!