Remove querystring template tag #10989

Open
opened 2025-12-29 21:38:50 +01:00 by adam · 0 comments
Owner

Originally created by @alehaa on GitHub (Apr 6, 2025).

NetBox version

v4.2.6

Feature type

Change to existing functionality

Proposed functionality

With Django 5.1 a built-in querystring template tag is available. I suggest removing the custom implementation in favor of the built-in one.

Use case

The built-in template tag doesn't need the request parameter. This can cause confusion when importing the helpers module into a template (e.g. for a custom plugin) which overrides the built-in template tag.

Database changes

None

External dependencies

None

Originally created by @alehaa on GitHub (Apr 6, 2025). ### NetBox version v4.2.6 ### Feature type Change to existing functionality ### Proposed functionality With [Django 5.1](https://docs.djangoproject.com/en/5.1/ref/templates/builtins/#querystring) a built-in `querystring` template tag is available. I suggest removing the custom implementation in favor of the built-in one. ### Use case The built-in template tag doesn't need the `request` parameter. This can cause confusion when importing the `helpers` module into a template (e.g. for a custom plugin) which overrides the built-in template tag. ### Database changes None ### External dependencies None
adam added the type: featurenetboxstatus: backlogcomplexity: low labels 2025-12-29 21:38:50 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#10989