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.
Problem/Motivation
Pager is shown twice in admin/people if views module is disabled. This is caused by UserListBuilder::render()
which adds a pager although the pager is already added by the parentEntityListBuilder::render()
Steps to reproduce on a clean installation:
- create 50+ users (devel generate speed up the things)
- disable views, views_ui
- go to admin/people
- pager is shown twice
Remaining tasks
create patch.
User interface changes
none
API changes
none
Comment | File | Size | Author |
---|---|---|---|
#4 | d8-users-duplicated-pager-2478091-4.patch | 640 bytes | willzyx |
users-pager.jpg | 43.48 KB | willzyx |
Comments
Comment #1
willzyx CreditAttribution: willzyx commentedComment #2
willzyx CreditAttribution: willzyx commentedComment #3
dawehnerIsn't the actual problem that we do $build['accounts'] = parent::render() and not
$build = parent::render(); $build['table']['#empty'];
? I bet the #empty text doesn't' work as expected, maybe we should fix it.Comment #4
willzyx CreditAttribution: willzyx commented@dawehner you are completely right
Comment #5
willzyx CreditAttribution: willzyx commentedComment #7
jibranIs it worth adding tests?
Comment #8
willzyx CreditAttribution: willzyx commentedI think add a test for this is unnecessary and does not add any value, but let me know
Comment #9
jibranFair enough
Comment #10
alexpottAgreed that a test for this feels unnecessary.
This issue is a normal bug fix, and doesn't include any disruptive changes, so it is allowed per https://www.drupal.org/core/beta-changes. Committed d361bd6 and pushed to 8.0.x. Thanks!