Select all with bulk edit only changes the currently visible objects #10454

Closed
opened 2025-12-29 21:31:41 +01:00 by adam · 1 comment
Owner

Originally created by @julianstolp on GitHub (Nov 8, 2024).

Originally assigned to: @jeremystretch on GitHub.

Deployment Type

Self-hosted

Triage priority

N/A

NetBox Version

v4.1.6

Python Version

3.12

Steps to Reproduce

  1. select all objects using the Toggle all checkbox
  2. tick the box Select all xxx object matching query
  3. go to Edit Selected
  4. change something and go to Apply

Expected Behavior

All selected objects will be modified.

Observed Behavior

Only the current visible objects get changed, if all objects are visible all get changed. The bulk delete is unaffected.
image
image

Originally created by @julianstolp on GitHub (Nov 8, 2024). Originally assigned to: @jeremystretch on GitHub. ### Deployment Type Self-hosted ### Triage priority N/A ### NetBox Version v4.1.6 ### Python Version 3.12 ### Steps to Reproduce 1. select all objects using the `Toggle all` checkbox 2. tick the box `Select all xxx object matching query` 3. go to `Edit Selected` 4. change something and go to `Apply` ### Expected Behavior All selected objects will be modified. ### Observed Behavior Only the current visible objects get changed, if all objects are visible all get changed. The bulk delete is unaffected. ![image](https://github.com/user-attachments/assets/7b1d54f6-ccf5-4564-8765-ec09fe05cb00) ![image](https://github.com/user-attachments/assets/934cd88a-4625-448f-832f-ebace4eddd0a)
adam added the type: bugstatus: acceptedseverity: medium labels 2025-12-29 21:31:41 +01:00
adam closed this issue 2025-12-29 21:31:41 +01:00
Author
Owner

@jeremystretch commented on GitHub (Nov 12, 2024):

I've traced this to a regression in 6a316df787, where the bulk edit form is initialized using POST data. This specific change doesn't appear to be needed for the functionality implemented under #17789, so I'm going to remove it.

@jeremystretch commented on GitHub (Nov 12, 2024): I've traced this to a regression in 6a316df787915141f8038507c531cdcfaffc2b39, where the bulk edit form is initialized using POST data. This specific change doesn't _appear_ to be needed for the functionality implemented under #17789, so I'm going to remove it.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#10454