Bug #33276
closedUpdating per_page on table sends the page into infinite loops
Description
To reproduce:
On any page of CV UI with a table, update pagination per_page to a value of 5.
The table keeps refreshing between the new perPage value and the default value.
Updated by Jeremy Lenz about 3 years ago
<jeremylenz> We need to be sure that if the user has changed the perPage value in the dropdown, we no longer care about what's in the Setting.
<adewar> indeed
<jeremylenz> so we could maybe useRef and set a flag
<jeremylenz> and check that flag inside onPaginationUpdate
<jeremylenz> sorry, check that flag right after we set our consts
<jeremylenz> and not use Foreman settings if the user has already set a perPage
<jeremylenz> this may not be fully thought out yet, but i do think we need some sort of decision to use or not use foremanPerPage
<jeremylenz> and don't just use it blindly as a fallback like we're doing
<jeremylenz> we should be checking it once and only once
<jeremylenz> and thereafter we only care about what the user has selected in the dropdown
Updated by The Foreman Bot about 3 years ago
- Status changed from New to Ready For Testing
- Assignee set to Jeremy Lenz
- Pull request https://github.com/Katello/katello/pull/9727 added
Updated by The Foreman Bot about 3 years ago
- Fixed in Releases Katello 4.3.0 added
Updated by Jeremy Lenz about 3 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset katello|fbdd242accf4c62d6712912c576790c79c8b3ddf.