Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The function of selecting all devices fails. #18964

Closed
Wythetells opened this issue Mar 20, 2025 · 5 comments · Fixed by #18997
Closed

The function of selecting all devices fails. #18964

Wythetells opened this issue Mar 20, 2025 · 5 comments · Fixed by #18997
Assignees
Labels
severity: medium Results in substantial degraded or broken functionality for specfic workflows status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application
Milestone

Comments

@Wythetells
Copy link

Wythetells commented Mar 20, 2025

Deployment Type

Self-hosted

NetBox Version

v4.2.5

Python Version

3.11

Steps to Reproduce

1、Select all the devices (25 in total) on the current page in the device list.

Image

2、Select the option "Select all 43 devices matching query". and edit

3、Edit the comments and click "Apply".

4、Only 25 devices are updated.

Image

Expected Behavior

all 43 devices are updated.

Observed Behavior

Only 25 devices are updated.

@Wythetells Wythetells added status: needs triage This issue is awaiting triage by a maintainer type: bug A confirmed report of unexpected behavior in the application labels Mar 20, 2025
@bctiemann bctiemann added status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation severity: low Does not significantly disrupt application functionality, or a workaround is available and removed status: needs triage This issue is awaiting triage by a maintainer labels Mar 20, 2025
@bctiemann
Copy link
Contributor

Verified this. Note for reproduction: set page size to 25 in user preferences, and select filters that result in > 25 results.

@antoinekh
Copy link
Contributor

I’ll be happy to take this issue

@antoinekh
Copy link
Contributor

antoinekh commented Mar 23, 2025

This is not related to devices only but global (IPAdress, Cable, Prefix...)

Same behavior with IPaddress for example :

Edit show all devices but at the end only the devices limit from the page size are updated

@jeremystretch jeremystretch added status: accepted This issue has been accepted for implementation and removed status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation labels Mar 24, 2025
@jeremystretch
Copy link
Member

@antoinekh assigned to you, thanks

@antoinekh
Copy link
Contributor

This issue is on all models (not only devices)

Same issue on the past:
#17963

Fix by @jeremystretch on #17994

Issue seems to have been reinsert here: #18197 to handle the HTMXSelect

Where the else condition disappears

I have readded the last else condition and this fix back the issue

        else:
            form = self.form(initial=initial_data)
            restrict_form_fields(form, request.user)

I'm not sure if this is the way to fix it cleanly, but I've tested it, it works and I don't see any problems.

antoinekh added a commit to antoinekh/netbox that referenced this issue Mar 25, 2025
@jeremystretch jeremystretch marked this as a duplicate of #18494 Mar 27, 2025
@jeremystretch jeremystretch added severity: medium Results in substantial degraded or broken functionality for specfic workflows and removed severity: low Does not significantly disrupt application functionality, or a workaround is available labels Mar 28, 2025
bctiemann added a commit that referenced this issue Mar 31, 2025
Fixes #18964 Select all with bulk edit only changes the currently visible objects
@jeremystretch jeremystretch added this to the v4.2.7 milestone Apr 1, 2025
@jnovinger jnovinger marked this as a duplicate of #19082 Apr 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity: medium Results in substantial degraded or broken functionality for specfic workflows status: accepted This issue has been accepted for implementation type: bug A confirmed report of unexpected behavior in the application
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants