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 |
---|---|---|---|
#4 | 2003386-01-page_feed_display_options.patch | 1.17 KB | SpartyDan |
#2 | rename-pagefeedisplayoptions-2003386-2.patch | 1.17 KB | illeace |
Comments
Comment #1
illeace CreditAttribution: illeace commentedWorking at this at Drupalcon code sprints.
Comment #2
illeace CreditAttribution: illeace commentedComment #3
illeace CreditAttribution: illeace commentedComment #4
SpartyDan CreditAttribution: SpartyDan commentedPatch to change page_feed_display_options() to pageFeedDisplayOptions()
Comment #5
illeace CreditAttribution: illeace commentedSorry, Dan. I didn't mean to steal your bug.
Comment #6
SpartyDan CreditAttribution: SpartyDan commentedNo worries. I posted my comment and patch without first refreshing my browser. Our patches are identical so that means we are both right. :-)
Comment #7
illeace CreditAttribution: illeace commentedThis was my first core patch, so I was very glad to see they were the same. :)
Comment #9
SpartyDan CreditAttribution: SpartyDan commented#4: 2003386-01-page_feed_display_options.patch queued for re-testing.
Comment #10
oenie CreditAttribution: oenie commentedLooks good to me !
Comment #11
alexpottCommitted fe3d859 and pushed to 8.x. Thanks!