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
ricky.middaugh CreditAttribution: ricky.middaugh commentedI'll take this one.
Comment #2
ricky.middaugh CreditAttribution: ricky.middaugh commentedIssue is ready for review.
All instances of set_current_page() have been updated to the proper convention.
Comment #3
ricky.middaugh CreditAttribution: ricky.middaugh commentedOops... included the wrong patch. Here's the new one.
Comment #5
heddnFailing tests. Please test and re-roll.
Comment #6
elvis2 CreditAttribution: elvis2 commentedFailing is due to an error on the test server side... Re-testing...
Comment #7
elvis2 CreditAttribution: elvis2 commented#3: core-rename-set-current-page-2003292-3.patch queued for re-testing.
Comment #8
oenie CreditAttribution: oenie commentedAdd public access modifier in front of the function(s) to adhere to the new OOP standards.
All occurences seem to have been replaced.
Comment #9
elvis2 CreditAttribution: elvis2 commentedRe-rolled.
Comment #10
heddnLooks good now.
Comment #11
alexpottNeeds a re-roll
Comment #12
elvis2 CreditAttribution: elvis2 commentedRe-rolled based on most recent pull or 8.x branch.
Comment #13
dawehnerComment #14
alexpottNeeds a re-roll
Comment #15
jibranReroll
Conflict
Resolved
Comment #16
aspilicious CreditAttribution: aspilicious commentedComment #17
alexpottCommitted cf23ee6 and pushed to 8.x. Thanks!