Clearing page length user preference breaks pagination #9692

New Issue

adam · 2025-12-29T21:20:59+01:00

adam commented

2025-12-29 21:20:59 +01:00

Originally created by @devon-mar on GitHub (May 17, 2024).

Originally assigned to: @devon-mar on GitHub.

Deployment Type

Self-hosted

NetBox Version

v4.0.2

Python Version

3.10

Steps to Reproduce

Create a new user account
Visit a page that uses pagination. It should load properly.
Open /user/preferences/ and click the "X" on Page length.
Click Save.
Visit the page in step 2. An error should occur.

Expected Behavior

The page should load without an error.

Observed Behavior

<class 'TypeError'>

'<' not supported between instances of 'int' and 'str'

Python version: 3.10.14
NetBox version: 4.0.2
Plugins: 
  netbox_demo: 0.5.0

I believe this is because the paginate.per_page preference is an empty string:

GET /api/users/config/

{
    "ui": {
        "htmx_navigation": ""
    },
    "locale": {
        "language": ""
    },
    "pagination": {
        "per_page": "",
        "placement": "bottom"
    },
    "data_format": "json"
}

Therefore, per_page is set to "" instead of config.PAGINATE_COUNT:
4b2f26a800/netbox/utilities/paginator.py (L90)

Originally created by @devon-mar on GitHub (May 17, 2024). Originally assigned to: @devon-mar on GitHub. ### Deployment Type Self-hosted ### NetBox Version v4.0.2 ### Python Version 3.10 ### Steps to Reproduce 1. Create a new user account 2. Visit a page that uses pagination. It should load properly. 3. Open `/user/preferences/` and click the "X" on `Page length`. 4. Click `Save`. 5. Visit the page in step 2. An error should occur. ### Expected Behavior The page should load without an error. ### Observed Behavior ``` <class 'TypeError'> '<' not supported between instances of 'int' and 'str' Python version: 3.10.14 NetBox version: 4.0.2 Plugins: netbox_demo: 0.5.0 ``` I believe this is because the `paginate.per_page` preference is an empty string: ``` GET /api/users/config/ { "ui": { "htmx_navigation": "" }, "locale": { "language": "" }, "pagination": { "per_page": "", "placement": "bottom" }, "data_format": "json" } ``` Therefore, `per_page` is set to `""` instead of `config.PAGINATE_COUNT`: https://github.com/netbox-community/netbox/blob/4b2f26a8002a8f01e186211c648768892cc199c8/netbox/utilities/paginator.py#L90

adam added the type: bug status: accepted severity: medium labels 2025-12-29 21:20:59 +01:00

adam closed this issue

2025-12-29 21:21:00 +01:00

adam commented

2025-12-29 21:21:00 +01:00

@devon-mar commented on GitHub (May 17, 2024):

Can I be assigned to this? I have a PR ready to go.

@devon-mar commented on GitHub (May 17, 2024): Can I be assigned to this? I have a PR ready to go.

Sign in to join this conversation.

Branches Tags

main

update-changelog-comments-docs

feature-removal-issue-type

20911-dropdown

20239-plugin-menu-classes-mutable-state

21097-graphql-id-lookups

feature

fix_module_substitution

20923-dcim-templates

20044-elevation-stuck-lightmode

feature-ip-prefix-link

v4.5-beta1-release

20068-import-moduletype-attrs

20766-fix-german-translation-code-literals

20378-del-script

7604-filter-modifiers-v3

circuit-swap

12318-case-insensitive-uniqueness

20637-improve-device-q-filter

20660-script-load

19724-graphql

20614-update-ruff

14884-script

02496-max-page

19720-macaddress-interface-generic-relation

19408-circuit-terminations-export-templates

20203-openapi-check

fix-19669-api-image-download

7604-filter-modifiers

19275-fixes-interface-bulk-edit

fix-17794-get_field_value_return_list

11507-show-aggregate-and-rir-on-api

9583-add_column_specific_search_field_to_tables

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/netbox#9692