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.

Sign up for GitHub

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

Jump to bottom

Interface PoE type reference error #10307

Closed
endreszabo opened this issue Sep 9, 2022 · 2 comments · Fixed by #10309
Closed

Interface PoE type reference error #10307

endreszabo opened this issue Sep 9, 2022 · 2 comments · Fixed by #10309
Assignees
@arthanson
Labels
type: bug A confirmed report of unexpected behavior in the application

Comments

@endreszabo
Copy link

endreszabo commented Sep 9, 2022 •
edited
Loading

NetBox version

v3.3.2

Python version

3.10

Steps to Reproduce

When choosing interface PoE type, select Passive 48V (2-pair), option Passive 48V (4-pair) will be chosen instead.

PoE type of Passive 48V (2-pair) could not be selected as Passive 48V (4-pair) had the same key for that value.

Expected Behavior

Option Passive 48V (2-pair) should be selected.

Observed Behavior

Option Passive 48V (4-pair) will be chosen instead.
@endreszabo endreszabo added the type: bug A confirmed report of unexpected behavior in the application label Sep 9, 2022
@endreszabo
Copy link
Author

The first item in the tuple is wrong here: https://github.com/netbox-community/netbox/blob/develop/netbox/dcim/choices.py#L1099

@arthanson arthanson self-assigned this Sep 9, 2022
arthanson added a commit that referenced this issue Sep 9, 2022
@arthanson
#10307 fix choices for poe type
59a2a43
@arthanson arthanson mentioned this issue Sep 9, 2022
Merged
@arthanson
Copy link
Collaborator

@endreszabo Thanks for tracking this down.

jeremystretch added a commit that referenced this issue Sep 9, 2022
@jeremystretch
Merge pull request #10309 from netbox-community/10307-fix-poe-type
77868a9 
#10307 fix choices for poe type
jeremystretch added a commit that referenced this issue Sep 9, 2022
@jeremystretch
Changelog for #10304, #10307
3aac62c
@arthanson arthanson mentioned this issue Sep 15, 2022
Merged
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@arthanson arthanson

Labels
type: bug A confirmed report of unexpected behavior in the application
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#10307 fix choices for poe type netbox-community/netbox
2 participants
@arthanson @endreszabo