starred/netbox
1
0
Fork 0
mirror of https://github.com/netbox-community/netbox.git synced 2026-01-11 21:10:29 +01:00
Code Issues 273 Packages Projects Releases 10 Wiki Activity

Front/reat port connection always shows B side of connected cable #2339

New Issue
Closed
opened 2025-12-29 17:24:58 +01:00 by adam · 1 comment
adam commented 2025-12-29 17:24:58 +01:00
Owner
Copy Link

Originally created by @jeremystretch on GitHub (Feb 1, 2019).

Environment

  • Python version: 3.5.2
  • NetBox version: 2.5.5

Steps to Reproduce

  1. Create or find a front or rear pass-through port with a connected cable.

Expected Behavior

The "front ports" and "rear ports" tables on the device view should list the far end of each attached cable.

Observed Behavior

The B side is always listed. This is due to a bug in the get_cable_peer() method:

def get_cable_peer(self):
    if self.cable is None:
        return None
    if self._cabled_as_a:
        return self.cable.termination_b
    if self._cabled_as_b:
        return self.cable.termination_a

The _cabled_as_x methods each return a generic relation manager, whereas what we need is the queryset.

Originally created by @jeremystretch on GitHub (Feb 1, 2019). ### Environment * Python version: 3.5.2 * NetBox version: 2.5.5 ### Steps to Reproduce 1. Create or find a front or rear pass-through port with a connected cable. ### Expected Behavior The "front ports" and "rear ports" tables on the device view should list the far end of each attached cable. ### Observed Behavior The B side is always listed. This is due to a bug in the `get_cable_peer()` method: ``` def get_cable_peer(self): if self.cable is None: return None if self._cabled_as_a: return self.cable.termination_b if self._cabled_as_b: return self.cable.termination_a ``` The `_cabled_as_x` methods each return a generic relation manager, whereas what we need is the queryset.
adam added the type: bugstatus: accepted labels 2025-12-29 17:24:58 +01:00
adam closed this issue 2025-12-29 17:24:59 +01:00
adam commented 2025-12-29 17:24:59 +01:00
Author
Owner
Copy Link

@tvberlin commented on GitHub (Feb 13, 2019):

@jeremystretch thanks for implementing!

@tvberlin commented on GitHub (Feb 13, 2019): @jeremystretch thanks for implementing!
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
beta
breaking change
complexity: high
complexity: low
complexity: medium
needs milestone
netbox
pending closure
plugin candidate
pull-request
Mirrored from GitHub Pull Request
 severity: high
severity: low
severity: medium
status: accepted
status: backlog
status: blocked
status: duplicate
status: needs owner
status: needs triage
status: revisions needed
status: under review
topic: GraphQL
topic: Internationalization
topic: OpenAPI
topic: UI/UX
topic: cabling
topic: event rules
topic: htmx navigation
topic: industrialization
topic: migrations
topic: plugins
topic: scripts
topic: templating
topic: testing
type: bug
type: deprecation
type: documentation
type: feature
type: housekeeping
type: translation
No Label status: accepted type: bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
adam
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#2339
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?