Fixes #20389: Add FilterSet support to BulkRenameView #986

Closed
opened 2026-04-05 19:29:50 +02:00 by MrUnknownDE · 0 comments
Owner

Originally created by @pheus on 10/24/2025

Fixes: #20389

Summary
Make BulkRenameView honor “Select all N matching query”, aligning behavior with BulkEditView/BulkDeleteView.

What changed

  • BulkRenameView now accepts a filterset.
  • When _all/select_all is posted, the target queryset is derived via filterset(request.GET, self.get_queryset(request)); otherwise it uses the posted pk[].
  • Add a test confirming selection (37 interfaces) is fully renamed.

Impact
No UI or API changes; bulk actions now behave consistently across edit/delete/rename.

*Originally created by @pheus on 10/24/2025* <!-- Thank you for your interest in contributing to NetBox! Please note that our contribution policy requires that a feature request or bug report be approved and assigned prior to opening a pull request. This helps avoid waste time and effort on a proposed change that we might not be able to accept. IF YOUR PULL REQUEST DOES NOT REFERENCE AN ISSUE WHICH HAS BEEN ASSIGNED TO YOU, IT WILL BE CLOSED AUTOMATICALLY. Please specify your assigned issue number on the line below. --> ### Fixes: #20389 **Summary** Make `BulkRenameView` honor **“Select all N matching query”**, aligning behavior with `BulkEditView`/`BulkDeleteView`. **What changed** - `BulkRenameView` now accepts a `filterset`. - When `_all`/`select_all` is posted, the target queryset is derived via `filterset(request.GET, self.get_queryset(request))`; otherwise it uses the posted `pk[]`. - Add a test confirming selection (37 interfaces) is fully renamed. **Impact** No UI or API changes; bulk actions now behave consistently across edit/delete/rename.
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github/netbox#986