Fixes #20475: Collapse singleton VLAN IDs in VLANGroup display #1149

Closed
opened 2026-04-05 21:09:41 +02:00 by MrUnknownDE · 0 comments
Owner

Originally created by @pheus on 10/2/2025

Fixes: #20475

Summary
Collapse singleton VLAN ID ranges when rendering/storing display strings.
Input 1-5,10,20-30 remains 1-5,10,20-30 instead of 1-5,10-10,20-30.

Implementation
Update utilities.ranges_to_string() to emit a bare integer when start == end.

Before → After
1-5,10-10,20-301-5,10,20-30

*Originally created by @pheus on 10/2/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: #20475 <!-- Please include a summary of the proposed changes below. --> **Summary** Collapse singleton VLAN ID ranges when rendering/storing display strings. Input `1-5,10,20-30` remains `1-5,10,20-30` instead of `1-5,10-10,20-30`. **Implementation** Update `utilities.ranges_to_string()` to emit a bare integer when `start == end`. **Before → After** `1-5,10-10,20-30` → `1-5,10,20-30`
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github/netbox#1149