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
We need to update the views functions with names that match function naming convention.
I'm starting this off assigned to myself.
Comment | File | Size | Author |
---|---|---|---|
#11 | drupal-viewsRename-get_where_args-2003410-11.patch | 659 bytes | pcambra |
#7 | drupal-viewsRename-get_where_args-2003410-7.patch | 1.06 KB | heddn |
#5 | core-rename-get_where_args-function-2003410-5.patch | 652 bytes | ricky.middaugh |
#1 | core-rename-get_where_args-function-2003410-1.patch | 652 bytes | ricky.middaugh |
Comments
Comment #1
ricky.middaugh CreditAttribution: ricky.middaugh commentedIssue ready for review. The function name has been updated.
Comment #3
ricky.middaugh CreditAttribution: ricky.middaugh commented#1: core-rename-get_where_args-function-2003410-1.patch queued for re-testing.
Comment #5
ricky.middaugh CreditAttribution: ricky.middaugh commentedUpdating with new patch.
Comment #7
heddnThere's no calls to get_where_args() so I'm guess that it needs to be public. Otherwise how could contrib module use it? Or maybe it isn't used any more, but that's cruft for another issue. Let's see if the testbot is happy today.
Comment #8
aspilicious CreditAttribution: aspilicious commentedComment #9
SpartyDan CreditAttribution: SpartyDan commentedTested patch to make sure it still applies. Queued patch for testing.
Comment #11
pcambraRerroll, it should pass?
Comment #12
aspilicious CreditAttribution: aspilicious commentedComment #13
alexpottCommitted 9791575 and pushed to 8.x. Thanks!
Comment #14.0
(not verified) CreditAttribution: commentedAdding parent issue