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 |
---|---|---|---|
#21 | 2002920-rename-get_items_per_page.patch | 7.73 KB | sillygwailo |
#16 | 2002920-rename-get_items_per_page.patch | 7.72 KB | sillygwailo |
#14 | 2002920-rename-get_items_per_page.patch | 7.7 KB | sillygwailo |
#12 | views.module-rename_get_items_per_page-2002920-12.patch | 7.86 KB | Maxis |
#9 | 2002920-9.patch | 7.71 KB | jibran |
Comments
Comment #1
Psikik CreditAttribution: Psikik commentedComment #2
Psikik CreditAttribution: Psikik commentedConvert get_items_per_page function to getItemsPerPage and add public decorator.
Comment #4
nielsonm CreditAttribution: nielsonm commented#2: 2002920-get_items_per_page-conversion.patch queued for re-testing.
Comment #6
oenie CreditAttribution: oenie commented#2: 2002920-get_items_per_page-conversion.patch queued for re-testing.
Comment #7
oenie CreditAttribution: oenie commentedLooks fine to me now !
Comment #8
alexpottNeeds a re-roll
Comment #9
jibranreroll. Had conflict with #1998330: Minipager is broken on page size == 1 due to ceil(PHP_MAX_INT / 1)
Comment #11
Maxis CreditAttribution: Maxis commentedstart working
Comment #12
Maxis CreditAttribution: Maxis commentedbot?
Comment #14
sillygwailoThe patch applied with
-p0
but not the standard-p1
. Re-rolling. This patch should at least apply.Comment #15
oenie CreditAttribution: oenie commentedOops ... seems we've missed an access modifier here, my bad, should have noticed this before:
Add public access modifier in front of the functions to adhere to the new OOP standards.
Comment #16
sillygwailoRe-rolled with public.
Comment #17
sillygwailoComment #18
aspilicious CreditAttribution: aspilicious commentedComment #19
aspilicious CreditAttribution: aspilicious commented#16: 2002920-rename-get_items_per_page.patch queued for re-testing.
Comment #21
sillygwailoNow that renaming set_offset() has been committed, this one should apply.
Comment #22
sillygwailoA note that #2003282: Rename Views method set_items_per_page() to setItemsPerPage() is related to this. In that issue, the lines patched refer to
get_items_per_page()
and will have to be updated to getItemsPerPage. That issue has a simpler patch than this one, but it may not matter, as long as one goes before the other.Comment #23
dcam CreditAttribution: dcam commented#21 looks good to me. All instances of get_items_per_page() are rewritten. The public access modifier has been added to all getItemsPerPage() methods.
#2003282: Rename Views method set_items_per_page() to setItemsPerPage() will need a reroll if this is committed.
Comment #24
alexpottCommitted 52befbe and pushed to 8.x. Thanks!