Require registered filterset for filter modifier enhancements

Updates FilterModifierMixin to only enhance form fields when the
associated model has a registered filterset. This provides plugin
safety by ensuring unregistered plugin filtersets fall back to
simple filters without lookup modifiers.

Test changes:
- Create TestModel and TestFilterSet using BaseFilterSet for
automatic lookup generation
- Import dcim.filtersets to ensure Device filterset registration
- Adjust tag field expectations to match actual Device filterset
(has exact/n but not empty lookups)

.github/ISSUE_TEMPLATE/01-feature_request.yaml

@@ -15,7 +15,7 @@ body:
     attributes:
       label: NetBox version
       description: What version of NetBox are you currently running?
-      placeholder: v4.5.1
+      placeholder: v4.4.7
     validations:
       required: true
   - type: dropdown

.github/ISSUE_TEMPLATE/02-bug_report.yaml

@@ -27,7 +27,7 @@ body:
     attributes:
       label: NetBox Version
       description: What version of NetBox are you currently running?
-      placeholder: v4.5.1
+      placeholder: v4.4.7
     validations:
       required: true
   - type: dropdown

.github/ISSUE_TEMPLATE/03-performance.yaml

@@ -1,43 +0,0 @@
----
-name: 🏁 Performance
-type: Performance
-description: An opportunity to improve application performance
-labels: ["netbox", "type: performance", "status: needs triage"]
-body:
-  - type: input
-    attributes:
-      label: NetBox Version
-      description: What version of NetBox are you currently running?
-      placeholder: v4.5.1
-    validations:
-      required: true
-  - type: dropdown
-    attributes:
-      label: Python Version
-      description: What version of Python are you currently running?
-      options:
-        - "3.12"
-        - "3.13"
-        - "3.14"
-    validations:
-      required: true
-  - type: checkboxes
-    attributes:
-      label: Area(s) of Concern
-      description: Which application interface(s) are affected?
-      options:
-        - label: User Interface
-        - label: REST API
-        - label: GraphQL API
-        - label: Python ORM
-        - label: Other
-    validations:
-      required: true
-  - type: textarea
-    attributes:
-      label: Details
-      description: >
-        Describe in detail the operations being performed and the indications of a performance issue.
-        Include any relevant testing parameters, benchmarks, and expected results.
-    validations:
-      required: true

.github/ISSUE_TEMPLATE/06-deprecation.yaml

@@ -0,0 +1,25 @@
+---
+name: 🗑️ Deprecation
+type: Deprecation
+description: The removal of an existing feature or resource
+labels: ["netbox", "type: deprecation"]
+body:
+  - type: textarea
+    attributes:
+      label: Proposed Changes
+      description: >
+        Describe in detail the proposed changes. What is being removed?
+    validations:
+      required: true
+  - type: textarea
+    attributes:
+      label: Justification
+      description: Please provide justification for the proposed change(s).
+    validations:
+      required: true
+  - type: textarea
+    attributes:
+      label: Impact
+      description: List all areas of the application that will be affected by this change.
+    validations:
+      required: true

.github/ISSUE_TEMPLATE/07-deprecation.yaml

@@ -1,31 +0,0 @@
----
-name: ⚠️ Deprecation
-type: Deprecation
-description: Designation of a feature or behavior that will be removed in a future release
-labels: ["netbox", "type: deprecation"]
-body:
-  - type: textarea
-    attributes:
-      label: Deprecated Functionality
-      description: >
-        Describe the feature(s) and/or behavior that is being flagged for deprecation.
-    validations:
-      required: true
-  - type: input
-    attributes:
-      label: Scheduled removal
-      description: In what future release will the deprecated functionality be removed?
-    validations:
-      required: true
-  - type: textarea
-    attributes:
-      label: Justification
-      description: Please provide justification for the deprecation.
-    validations:
-      required: true
-  - type: textarea
-    attributes:
-      label: Impact
-      description: List all areas of the application that will be affected by this change.
-    validations:
-      required: true

.github/ISSUE_TEMPLATE/08-feature_removal.yaml

@@ -1,20 +0,0 @@
----
-name: 🗑️ Feature Removal
-type: Removal
-description: The removal of a deprecated feature or resource
-labels: ["netbox", "type: removal"]
-body:
-  - type: input
-    attributes:
-      label: Deprecation Issue
-      description: Specify the issue in which this deprecation was announced.
-      placeholder: "#1234"
-    validations:
-      required: true
-  - type: textarea
-    attributes:
-      label: Summary of Changes
-      description: >
-        List all changes necessary to remove the deprecated feature or resource.
-    validations:
-      required: true

.github/workflows/ci.yml

@@ -31,7 +31,7 @@ jobs:
       NETBOX_CONFIGURATION: netbox.configuration_testing
     strategy:
       matrix:
-        python-version: ['3.12', '3.13', '3.14']
+        python-version: ['3.12', '3.13']
         node-version: ['20.x']
     services:
       redis:

.github/workflows/codeql.yml

@@ -30,13 +30,13 @@ jobs:
       uses: actions/checkout@v4
 
     - name: Initialize CodeQL
-      uses: github/codeql-action/init@v4
+      uses: github/codeql-action/init@v3
       with:
         languages: ${{ matrix.language }}
         build-mode: ${{ matrix.build-mode }}
         config-file: .github/codeql/codeql-config.yml
 
     - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@v4
+      uses: github/codeql-action/analyze@v3
       with:
         category: "/language:${{matrix.language}}"

.github/workflows/update-translation-strings.yml

@@ -34,7 +34,7 @@ jobs:
     - name: Set up Python
       uses: actions/setup-python@v5
       with:
-        python-version: 3.12
+        python-version: 3.11
 
     - name: Install system dependencies
       run: sudo apt install -y gettext

.gitignore

@@ -9,8 +9,7 @@ yarn-error.log*
 /netbox/netbox/configuration.py
 /netbox/netbox/ldap_config.py
 /netbox/local/*
-/netbox/media/*
-!/netbox/media/.gitkeep
+/netbox/media
 /netbox/reports/*
 !/netbox/reports/__init__.py
 /netbox/scripts/*

README.md

@@ -5,7 +5,7 @@
   <a href="https://github.com/netbox-community/netbox/blob/main/LICENSE.txt"><img src="https://img.shields.io/badge/license-Apache_2.0-blue.svg" alt="License" /></a>
   <a href="https://github.com/netbox-community/netbox/graphs/contributors"><img src="https://img.shields.io/github/contributors/netbox-community/netbox?color=blue" alt="Contributors" /></a>
   <a href="https://github.com/netbox-community/netbox/stargazers"><img src="https://img.shields.io/github/stars/netbox-community/netbox?style=flat" alt="GitHub stars" /></a>
-  <a href="https://explore.transifex.com/netbox-community/netbox/"><img src="https://img.shields.io/badge/languages-16-blue" alt="Languages supported" /></a>
+  <a href="https://explore.transifex.com/netbox-community/netbox/"><img src="https://img.shields.io/badge/languages-15-blue" alt="Languages supported" /></a>
   <a href="https://github.com/netbox-community/netbox/actions/workflows/ci.yml"><img src="https://github.com/netbox-community/netbox/actions/workflows/ci.yml/badge.svg" alt="CI status" /></a>
   <p>
     <strong><a href="https://netboxlabs.com/community/">NetBox Community</a></strong> |

docs/administration/permissions.md

@@ -88,7 +88,7 @@ While permissions are typically assigned to specific groups and/or users, it is
 
 ### Viewing Objects
 
-Object-based permissions work by filtering the database query generated by a user's request to restrict the set of objects returned. When a request is received, NetBox first determines whether the user is authenticated and has been granted permission to perform the requested action. For example, if the requested URL is `/dcim/devices/`, NetBox will check for the `dcim.view_device` permission. If the user has not been assigned this permission (either directly or via a group assignment), NetBox will return a 403 (forbidden) HTTP response.
+Object-based permissions work by filtering the database query generated by a user's request to restrict the set of objects returned. When a request is received, NetBox first determines whether the user is authenticated and has been granted to perform the requested action. For example, if the requested URL is `/dcim/devices/`, NetBox will check for the `dcim.view_device` permission. If the user has not been assigned this permission (either directly or via a group assignment), NetBox will return a 403 (forbidden) HTTP response.
 
 If the permission _has_ been granted, NetBox will compile any specified constraints for the model and action. For example, suppose two permissions have been assigned to the user granting view access to the device model, with the following constraints:
 
@@ -102,9 +102,9 @@ If the permission _has_ been granted, NetBox will compile any specified constrai
 This grants the user access to view any device that is assigned to a site named NYC1 or NYC2, **or** which has a status of "offline" and has no tenant assigned. These constraints are equivalent to the following ORM query:
 
 ```no-highlight
-Device.objects.filter(
+Site.objects.filter(
     Q(site__name__in=['NYC1', 'NYC2']),
-    Q(status='offline', tenant__isnull=True)
+    Q(status='active', tenant__isnull=True)
 )
 ```
 

docs/development/models.md

@@ -12,7 +12,7 @@ Depending on its classification, each NetBox model may support various features
 
 | Feature                                                    | Feature Mixin           | Registry Key        | Description                                                                             |
 |------------------------------------------------------------|-------------------------|---------------------|-----------------------------------------------------------------------------------------|
-| [Bookmarks](../features/user-preferences.md#bookmarks)     | `BookmarksMixin`        | `bookmarks`         | These models can be bookmarked natively in the user interface                           |
+| [Bookmarks](../features/customization.md#bookmarks)        | `BookmarksMixin`        | `bookmarks`         | These models can be bookmarked natively in the user interface                           |
 | [Change logging](../features/change-logging.md)            | `ChangeLoggingMixin`    | `change_logging`    | Changes to these objects are automatically recorded in the change log                   |
 | Cloning                                                    | `CloningMixin`          | `cloning`           | Provides the `clone()` method to prepare a copy                                         |
 | [Contacts](../features/contacts.md)                        | `ContactsMixin`         | `contacts`          | Contacts can be associated with these models                                            |

docs/features/change-logging.md

@@ -10,11 +10,9 @@ Change records are exposed in the API via the read-only endpoint `/api/extras/ob
 
 ## User Messages
 
-When creating, modifying, or deleting an object in NetBox, a user has the option of recording an arbitrary message (up to 200 characters) that will appear in the change record. This can be helpful to capture additional context, such as the reason for a change or a reference to an external ticket.
+!!! info "This feature was introduced in NetBox v4.4."
 
-When editing an object via the web UI, the "Changelog message" field appears at the bottom of the form. This field is optional. The changelog message field is available in object create forms, object edit forms, delete confirmation dialogs, and bulk operations.
-
-For information on including changelog messages when making changes via the REST API, see [Changelog Messages](../integrations/rest-api.md#changelog-messages).
+When creating, modifying, or deleting an object in NetBox, a user has the option of recording an arbitrary message that will appear in the change record. This can be helpful to capture additional context, such as the reason for the change.
 
 ## Correlating Changes by Request
 

docs/integrations/rest-api.md

@@ -610,7 +610,9 @@ http://netbox/api/dcim/sites/ \
 
 ## Changelog Messages
 
-Most objects in NetBox support [change logging](../features/change-logging.md), which generates a detailed record each time an object is created, modified, or deleted. Additionally, users can attach a message to the change record as well. This is accomplished via the REST API by including a `changelog_message` field in the object representation.
+!!! info "This feature was introduced in NetBox v4.4."
+
+Most objects in NetBox support [change logging](../features/change-logging.md), which generates a detailed record each time an object is created, modified, or deleted. Beginning in NetBox v4.4, users can attach a message to the change record as well. This is accomplished via the REST API by including a `changelog_message` field in the object representation.
 
 For example, the following API request will create a new site and record a message in the resulting changelog entry:
 
@@ -626,7 +628,7 @@ http://netbox/api/dcim/sites/ \
 }'
 ```
 
-This approach works when creating, modifying, or deleting objects, either individually or in bulk. For more information about change logging, see [Change Logging](../features/change-logging.md).
+This approach works when creating, modifying, or deleting objects, either individually or in bulk.
 
 ## Uploading Files
 

docs/plugins/development/filtersets.md

@@ -6,17 +6,12 @@ Filter sets define the mechanisms available for filtering or searching through a
 
 To support additional functionality standard to NetBox models, such as tag assignment and custom field support, the `NetBoxModelFilterSet` class is available for use by plugins. This should be used as the base filter set class for plugin models which inherit from `NetBoxModel`. Within this class, individual filters can be declared as directed by the `django-filters` documentation. An example is provided below.
 
-!!! info "New in NetBox v4.5: FilterSet Registration"
-    NetBox v4.5 introduced the `register_filterset()` utility function. This enables plugins to register their filtersets to receive advanced functionality, such as the automatic attachment of field-specific lookup modifiers on the filter form. Registration is optional: Unregistered filtersets will continue to work as before, but will not receive the enhanced functionality.
-
 ```python
 # filtersets.py
 import django_filters
 from netbox.filtersets import NetBoxModelFilterSet
-from utilities.filtersets import register_filterset
 from .models import MyModel
 
-@register_filterset
 class MyFilterSet(NetBoxModelFilterSet):
     status = django_filters.MultipleChoiceFilter(
         choices=(
@@ -32,14 +27,6 @@ class MyFilterSet(NetBoxModelFilterSet):
         fields = ('some', 'other', 'fields')
 ```
 
-In addition to the base NetBoxModelFilterSet class, the following filterset classes are also available for subclasses of standard base models.
-
-| Model Class           | FilterSet Class                                  |
-|-----------------------|--------------------------------------------------|
-| `PrimaryModel`        | `netbox.filtersets.PrimaryModelFilterSet`        |
-| `OrganizationalModel` | `netbox.filtersets.OrganizationalModelFilterSet` |
-| `NestedGroupModel`    | `netbox.filtersets.NestedGroupModelFilterSet`    |
-
 ### Declaring Filter Sets
 
 To utilize a filter set in a subclass of one of NetBox's generic views (such as `ObjectListView` or `BulkEditView`), define the `filterset` attribute on the view class:
@@ -55,7 +42,7 @@ class MyModelListView(ObjectListView):
     filterset = MyModelFilterSet
 ```
 
-To enable a filter set on a REST API endpoint, set the `filterset_class` attribute on the API view:
+To enable a filter set on a  REST API endpoint, set the `filterset_class` attribute on the API view:
 
 ```python
 # api/views.py
@@ -75,9 +62,7 @@ The `ObjectListView` has a field called Quick Search. For Quick Search to work t
 ```python
 from django.db.models import Q
 from netbox.filtersets import NetBoxModelFilterSet
-from utilities.filtersets import register_filterset
 
-@register_filterset
 class MyFilterSet(NetBoxModelFilterSet):
     ...
     def search(self, queryset, name, value):
@@ -105,9 +90,7 @@ This class filters `tags` using the `slug` field. For example:
 ```python
 from django_filters import FilterSet
 from extras.filters import TagFilter
-from utilities.filtersets import register_filterset
 
-@register_filterset
 class MyModelFilterSet(FilterSet):
     tag = TagFilter()
 ```
@@ -123,9 +106,7 @@ This class filters `tags` using the `id` field. For example:
 ```python
 from django_filters import FilterSet
 from extras.filters import TagIDFilter
-from utilities.filtersets import register_filterset
 
-@register_filterset
 class MyModelFilterSet(FilterSet):
     tag_id = TagIDFilter()
 ```

docs/plugins/development/forms.md

@@ -2,7 +2,7 @@
 
 ## Form Classes
 
-NetBox provides several base form classes for use by plugins. Additional form classes are also available for other standard base model classes (PrimaryModel, OrganizationalModel, and NestedGroupModel).
+NetBox provides several base form classes for use by plugins.
 
 | Form Class                 | Purpose                              |
 |----------------------------|--------------------------------------|
@@ -19,17 +19,7 @@ This is the base form for creating and editing NetBox models. It extends Django'
 |-------------|---------------------------------------------------------------------------------------|
 | `fieldsets` | A tuple of `FieldSet` instances which control how form fields are rendered (optional) |
 
-#### Subclasses
-
-The corresponding model-specific subclasses of `NetBoxModelForm` are documented below.
-
-| Model Class           | Form Class                |
-|-----------------------|---------------------------|
-| `PrimaryModel`        | `PrimaryModelForm`        |
-| `OrganizationalModel` | `OrganizationalModelForm` |
-| `NestedGroupModel`    | `NestedGroupModelForm`    |
-
-#### Example
+**Example**
 
 ```python
 from django.utils.translation import gettext_lazy as _
@@ -59,19 +49,9 @@ class MyModelForm(NetBoxModelForm):
 
 ### `NetBoxModelImportForm`
 
-This form facilitates the bulk import of new objects from CSV, JSON, or YAML data. As with model forms, you'll need to declare a `Meta` subclass specifying the associated `model` and `fields`. NetBox also provides several form fields suitable for importing various types of CSV data, listed [below](#csv-import-fields).
+This form facilitates the bulk import of new objects from CSV, JSON, or YAML data. As with model forms, you'll need to declare a `Meta` subclass specifying the associated `model` and `fields`. NetBox also provides several form fields suitable for import various types of CSV data, listed below.
 
-#### Subclasses
-
-The corresponding model-specific subclasses of `NetBoxModelImportForm` are documented below.
-
-| Model Class           | Form Class                      |
-|-----------------------|---------------------------------|
-| `PrimaryModel`        | `PrimaryModelImportForm`        |
-| `OrganizationalModel` | `OrganizationalModelImportForm` |
-| `NestedGroupModel`    | `NestedGroupModelImportForm`    |
-
-#### Example
+**Example**
 
 ```python
 from django.utils.translation import gettext_lazy as _
@@ -103,17 +83,7 @@ This form facilitates editing multiple objects in bulk. Unlike a model form, thi
 | `fieldsets`       | A tuple of `FieldSet` instances which control how form fields are rendered (optional)       |
 | `nullable_fields` | A tuple of fields which can be nullified (set to empty) using the bulk edit form (optional) |
 
-#### Subclasses
-
-The corresponding model-specific subclasses of `NetBoxModelBulkEditForm` are documented below.
-
-| Model Class           | Form Class                        |
-|-----------------------|-----------------------------------|
-| `PrimaryModel`        | `PrimaryModelBulkEditForm`        |
-| `OrganizationalModel` | `OrganizationalModelBulkEditForm` |
-| `NestedGroupModel`    | `NestedGroupModelBulkEditForm`    |
-
-#### Example
+**Example**
 
 ```python
 from django import forms
@@ -155,17 +125,7 @@ This form class is used to render a form expressly for filtering a list of objec
 | `model`     | The model of object being edited                                                      |
 | `fieldsets` | A tuple of `FieldSet` instances which control how form fields are rendered (optional) |
 
-#### Subclasses
-
-The corresponding model-specific subclasses of `NetBoxModelFilterSetForm` are documented below.
-
-| Model Class           | Form Class                         |
-|-----------------------|------------------------------------|
-| `PrimaryModel`        | `PrimaryModelFilterSetForm`        |
-| `OrganizationalModel` | `OrganizationalModelFilterSetForm` |
-| `NestedGroupModel`    | `NestedGroupModelFilterSetForm`    |
-
-#### Example
+**Example**
 
 ```python
 from dcim.models import Site

docs/plugins/development/graphql-api.md

@@ -46,19 +46,3 @@ NetBox provides two object type classes for use by plugins.
 ::: netbox.graphql.types.NetBoxObjectType
     options:
       members: false
-
-## GraphQL Filters
-
-NetBox provides a base filter class for use by plugins which employ subclasseses of `NetBoxModel`.
-
-::: netbox.graphql.filters.NetBoxModelFilter
-    options:
-      members: false
-
-Additionally, the following filter classes are available for subclasses of standard base models.
-
-| Model Class           | FilterSet Class                                    |
-|-----------------------|----------------------------------------------------|
-| `PrimaryModel`        | `netbox.graphql.filters.PrimaryModelFilter`        |
-| `OrganizationalModel` | `netbox.graphql.filters.OrganizationalModelFilter` |
-| `NestedGroupModel`    | `netbox.graphql.filters.NestedGroupModelFilter`    |

docs/plugins/development/index.md

@@ -74,7 +74,7 @@ The plugin source directory contains all the actual Python code and other resour
 
 The `PluginConfig` class is a NetBox-specific wrapper around Django's built-in [`AppConfig`](https://docs.djangoproject.com/en/stable/ref/applications/) class. It is used to declare NetBox plugin functionality within a Python package. Each plugin should provide its own subclass, defining its name, metadata, and default and required configuration parameters. An example is below:
 
-```python title="__init__.py"
+```python
 from netbox.plugins import PluginConfig
 
 class FooBarConfig(PluginConfig):
@@ -151,7 +151,7 @@ Any additional apps must be installed within the same Python environment as NetB
 
 An example `pyproject.toml` is below:
 
-```toml title="pyproject.toml"
+```
 # See PEP 518 for the spec of this file
 # https://www.python.org/dev/peps/pep-0518/
 
@@ -179,24 +179,11 @@ classifiers=[
 ]
 
 requires-python = ">=3.12.0"
+
 ```
 
 Many of these are self-explanatory, but for more information, see the [pyproject.toml documentation](https://packaging.python.org/en/latest/specifications/pyproject-toml/).
 
-## Compatibility Matrix
-
-Consider adding a file named `COMPATIBILITY.md` to your plugin project root (alongside `pyproject.toml`). This file should contain a table listing the minimum and maximum supported versions of NetBox (`min_version` and `max_version`) for each release. This serves as a handy reference for users who are upgrading from a previous version of your plugin. An example is shown below:
-
-```markdown title="COMPATIBILITY.md"
-# Compatibility Matrix
-
-| Release | Minimum NetBox Version | Maximum NetBox Version |
-|---------|------------------------|------------------------|
-| 0.2.0   | 4.4.0                  | 4.5.x                  |
-| 0.1.1   | 4.3.0                  | 4.4.x                  |
-| 0.1.0   | 4.3.0                  | 4.4.x                  |
-```
-
 ## Create a Virtual Environment
 
 It is strongly recommended to create a Python [virtual environment](https://docs.python.org/3/tutorial/venv.html) for the development of your plugin, as opposed to using system-wide packages. This will afford you complete control over the installed versions of all dependencies and avoid conflict with system packages. This environment can live wherever you'd like;however, it should be excluded from revision control. (A popular convention is to keep all virtual environments in the user's home directory, e.g. `~/.virtualenvs/`.)

docs/plugins/development/migration-v4.md

@@ -325,14 +325,14 @@ class CircuitTypeType(OrganizationalObjectType):
 
 ### Change filters.py
 
-Filter classes should inherit from `netbox.graphql.filters.BaseModelFilter`.
+Strawberry currently doesn't directly support django-filter, so an explicit filters.py file will need to be created.  NetBox includes a new `autotype_decorator` used to automatically wrap FilterSets to reduce the required code to a minimum.
 
 ```python title="New"
 import strawberry
 import strawberry_django
 from circuits import filtersets, models
 
-from netbox.graphql.filters import BaseModelFilter
+from netbox.graphql.filter_mixins import autotype_decorator, BaseFilterMixin
 
 __all__ = (
     'CircuitFilter',
@@ -340,7 +340,8 @@ __all__ = (
 
 
 @strawberry_django.filter(models.Circuit, lookups=True)
-class CircuitFilter(BaseModelFilter):
+@autotype_decorator(filtersets.CircuitFilterSet)
+class CircuitFilter(BaseFilterMixin):
     pass
 
 ```

docs/plugins/development/models.md

@@ -67,46 +67,6 @@ class MyModel(ExportTemplatesMixin, TagsMixin, models.Model):
     ...
 ```
 
-### Additional Models
-
-In addition to the base NetBoxModel class, the following additional classes are provided for convenience.
-
-!!! info "These model classes were added to the plugins API in NetBox v4.5."
-
-#### PrimaryModel
-
-PrimaryModel is the go-to class for most object types. It extends NetBoxModel with `description` and `comments` fields, and it introduces support for ownership assignment.
-
-| Field         | Required | Unique | Description                                 |
-|---------------|----------|--------|---------------------------------------------|
-| `owner`       | No       | No     | The object's owner                          |
-| `description` | No       | No     | A human-friendly description for the object |
-| `comments`    | No       | No     | General comments                            |
-
-#### OrganizationalModel
-
-OrganizationalModel is used by object types whose function is primarily the organization of other objects.
-
-| Field         | Required | Unique | Description                                 |
-|---------------|----------|--------|---------------------------------------------|
-| `name`        | Yes      | Yes    | The name of the object                      |
-| `slug`        | Yes      | Yes    | A unique URL-friendly identifier            |
-| `owner`       | No       | No     | The object's owner                          |
-| `description` | No       | No     | A human-friendly description for the object |
-
-#### NestedGroupModel
-
-NestedGroupModel is used for objects which arrange into a recursive hierarchy (like regions and locations) via its self-referential `parent` foreign key.
-
-| Field         | Required | Unique | Description                                                     |
-|---------------|----------|--------|-----------------------------------------------------------------|
-| `name`        | Yes      | Yes    | The name of the object                                          |
-| `slug`        | Yes      | Yes    | A unique URL-friendly identifier                                |
-| `parent`      | No       | No     | The object (of the same type) under which this object is nested |
-| `owner`       | No       | No     | The object's owner                                              |
-| `description` | No       | No     | A human-friendly description for the object                     |
-| `comments`    | No       | No     | General comments                                                |
-
 ## Database Migrations
 
 Once you have completed defining the model(s) for your plugin, you'll need to create the database schema migrations. A migration file is essentially a set of instructions for manipulating the PostgreSQL database to support your new model, or to alter existing models. Creating migrations can usually be done automatically using Django's `makemigrations` management command. (Ensure that your plugin has been installed and enabled first, otherwise it won't be found.)

docs/plugins/development/rest-api.md

@@ -27,14 +27,6 @@ Serializers are responsible for converting Python objects to JSON data suitable
 
 The default nested representation of an object is defined by the `brief_fields` attributes under the serializer's `Meta` class. (Older versions of NetBox required the definition of a separate nested serializer.)
 
-In addition to the base NetBoxModelSerializer class, the following serializer classes are also available for subclasses of standard base models.
-
-| Model Class           | Serializer Class                                       |
-|-----------------------|--------------------------------------------------------|
-| `PrimaryModel`        | `netbox.api.serializers.PrimaryModelSerializer`        |
-| `OrganizationalModel` | `netbox.api.serializers.OrganizationalModelSerializer` |
-| `NestedGroupModel`    | `netbox.api.serializers.NestedGroupModelSerializer`    |
-
 #### Example
 
 To create a serializer for a plugin model, subclass `NetBoxModelSerializer` in `api/serializers.py`. Specify the model class and the fields to include within the serializer's `Meta` class.

docs/plugins/development/tables.md

@@ -36,14 +36,6 @@ class MyModelTable(NetBoxTable):
         default_columns = ('pk', 'name', ...)
 ```
 
-In addition to the base NetBoxTable class, the following table classes are also available for subclasses of standard base models.
-
-| Model Class           | Table Class                              |
-|-----------------------|------------------------------------------|
-| `PrimaryModel`        | `netbox.tables.PrimaryModelTable`        |
-| `OrganizationalModel` | `netbox.tables.OrganizationalModelTable` |
-| `NestedGroupModel`    | `netbox.tables.NestedGroupModelTable`    |
-
 ### Table Configuration
 
 The NetBoxTable class features dynamic configuration to allow users to change their column display and ordering preferences. To configure a table for a specific request, simply call its `configure()` method and pass the current HTTPRequest object. For example:

docs/release-notes/index.md

@@ -10,14 +10,6 @@ Minor releases are published in April, August, and December of each calendar yea
 
 This page contains a history of all major and minor releases since NetBox v2.0. For more detail on a specific patch release, please see the release notes page for that specific minor release.
 
-#### [Version 4.5](./version-4.5.md) (January 2026)
-
-* Lookup Modifiers in Filter Forms ([#7604](https://github.com/netbox-community/netbox/issues/7604))
-* Improved API Authentication Tokens ([#20210](https://github.com/netbox-community/netbox/issues/20210))
-* Object Ownership ([#20304](https://github.com/netbox-community/netbox/issues/20304))
-* Advanced Port Mappings ([#20564](https://github.com/netbox-community/netbox/issues/20564))
-* Cable Profiles ([#20788](https://github.com/netbox-community/netbox/issues/20788))
-
 #### [Version 4.4](./version-4.4.md) (September 2025)
 
 * Background Jobs for Bulk Operations ([#19589](https://github.com/netbox-community/netbox/issues/19589), [#19891](https://github.com/netbox-community/netbox/issues/19891))

docs/release-notes/version-4.4.md

@@ -1,69 +1,5 @@
 # NetBox v4.4
 
-## v4.4.10 (2026-01-06)
-
-### Enhancements
-
-* [#20953](https://github.com/netbox-community/netbox/issues/20953) - Show reverse bridge relationships on interface detail pages
-* [#21071](https://github.com/netbox-community/netbox/issues/21071) - Include request method & URL when displaying server errors
-
-### Bug Fixes
-
-* [#19506](https://github.com/netbox-community/netbox/issues/19506) - Add filter forms for component templates to ensure object selector support
-* [#20044](https://github.com/netbox-community/netbox/issues/20044) - Fix dark mode support for rack elevations
-* [#20320](https://github.com/netbox-community/netbox/issues/20320) - Restore support for selecting related interfaces when bulk editing device interfaces
-* [#20817](https://github.com/netbox-community/netbox/issues/20817) - Re-enable sync button when disabling scheduled syncing for a data source
-* [#21045](https://github.com/netbox-community/netbox/issues/21045) - Fix `ValueError` exception when saving a site with an assigned prefix
-* [#21049](https://github.com/netbox-community/netbox/issues/21049) - Ignore stale custom field data when validating an object
-* [#21063](https://github.com/netbox-community/netbox/issues/21063) - Check for duplicate choice values when validating a custom field choice set
-* [#21064](https://github.com/netbox-community/netbox/issues/21064) - Ensures that extra choices in custom field choice sets preserve escaped colons
-
----
-
-## v4.4.9 (2025-12-23)
-
-### Enhancements
-
-* [#20309](https://github.com/netbox-community/netbox/issues/20309) - Support ASDOT notation for ASN ranges
-* [#20720](https://github.com/netbox-community/netbox/issues/20720) - Add Latvian translations
-* [#20900](https://github.com/netbox-community/netbox/issues/20900) - Allow filtering custom choice fields by multiple values in the UI
-
-### Bug Fixes
-
-* [#17976](https://github.com/netbox-community/netbox/issues/17976) - Remove `devicetype_count` from nested manufacturer to correct OpenAPI schema
-* [#20011](https://github.com/netbox-community/netbox/issues/20011) - Provide a clear message when encountering duplicate object IDs during bulk import
-* [#20114](https://github.com/netbox-community/netbox/issues/20114) - Preserve `parent_bay` during device bulk import when tags are present
-* [#20491](https://github.com/netbox-community/netbox/issues/20491) - Improve handling of numeric ranges in tests
-* [#20873](https://github.com/netbox-community/netbox/issues/20873) - Fix `AttributeError` exception triggered by event rules associated with an object that supports file attachments
-* [#20875](https://github.com/netbox-community/netbox/issues/20875) - Ensure that parent object relations are cached (for filtering) on device/module components during instantiation
-* [#20876](https://github.com/netbox-community/netbox/issues/20876) - Allow editing an IP address that resides within a range marked as populated
-* [#20912](https://github.com/netbox-community/netbox/issues/20912) - Fix inconsistent clearing of `module` field on ModuleBay
-* [#20944](https://github.com/netbox-community/netbox/issues/20944) - Ensure cached scope is updated on child objects when a parent region/site/location is changed
-* [#20948](https://github.com/netbox-community/netbox/issues/20948) - Handle the deletion of related objects with `on_delete=RESTRICT` the same as `CASCADE`
-* [#20966](https://github.com/netbox-community/netbox/issues/20966) - Fix UI rendering issue when scrolling list of object types in permissions form
-* [#20969](https://github.com/netbox-community/netbox/issues/20969) - Fix querying of front port templates by `rear_port_id`
-* [#21011](https://github.com/netbox-community/netbox/issues/21011) - Avoid writing to the database when loading active ConfigRevision
-* [#21032](https://github.com/netbox-community/netbox/issues/21032) - Avoid SQL subquery in RestrictedQuerySet where unnecessary
-
----
-
-## v4.4.8 (2025-12-09)
-
-### Enhancements
-
-* [#20068](https://github.com/netbox-community/netbox/issues/20068) - Support the assignment of module type profile attributes via bulk import
-* [#20914](https://github.com/netbox-community/netbox/issues/20914) - Enable filtering device components by tenant assigned to device
-
-### Bug Fixes
-
-* [#19918](https://github.com/netbox-community/netbox/issues/19918) - Fix support for `{module}` resolution of components of child modules
-* [#20759](https://github.com/netbox-community/netbox/issues/20759) - Improve legibility of object types in permissions form
-* [#20860](https://github.com/netbox-community/netbox/issues/20860) - Ensure user-provided changelog message is recorded when creating device components via the UI
-* [#20878](https://github.com/netbox-community/netbox/issues/20878) - Use the active database connection when executing custom scripts
-* [#20888](https://github.com/netbox-community/netbox/issues/20888) - Resolve warnings about non-decimal values for min/max latitude & longitude fields
-
----
-
 ## v4.4.7 (2025-11-25)
 
 ### Enhancements

docs/release-notes/version-4.5.md

@@ -1,187 +0,0 @@
-# NetBox v4.5
-
-## v4.5.1 (2026-01-20)
-
-### Enhancements
-
-* [#21018](https://github.com/netbox-community/netbox/issues/21018) - Enable filtering prefixes by location/site/site group/region directly via GraphQL API
-* [#21142](https://github.com/netbox-community/netbox/issues/21142) - Enable filtering device components by site/location/rack directly via GraphQL API
-* [#21144](https://github.com/netbox-community/netbox/issues/21144) - Enable specifying a prefix length for IP addresses when utilizing the `/api/ipam/prefixes/<id>/available-ips/` REST API endpoint
-* [#21165](https://github.com/netbox-community/netbox/issues/21165) - VLAN selector should default to group (instead of site)
-* [#21178](https://github.com/netbox-community/netbox/issues/21178) - Improve consistency of rack measurements in UI
-
-### Bug Fixes
-
-* [#19901](https://github.com/netbox-community/netbox/issues/19901) - Fix `RelatedObjectDoesNotExist` exception when importing modules into unnamed devices
-* [#20239](https://github.com/netbox-community/netbox/issues/20239) - Prevent shared mutable state in PluginMenuItem & PluginMenuButton
-* [#20933](https://github.com/netbox-community/netbox/issues/20933) - Fix writable `data_file` assignment for ConfigContext and ConfigContextProfile via the REST API
-* [#21039](https://github.com/netbox-community/netbox/issues/21039) - Fix support for AVIF image uploads
-* [#21050](https://github.com/netbox-community/netbox/issues/21050) - Clear device OOB IP assignments when reassigning IP addresses
-* [#21051](https://github.com/netbox-community/netbox/issues/21051) - Remove irrelevant object types from permissions form
-* [#21097](https://github.com/netbox-community/netbox/issues/21097) - Fix comparison lookups for ID filters in GraphQL API
-* [#21102](https://github.com/netbox-community/netbox/issues/21102) - Fix GraphiQL explorer UI
-* [#21117](https://github.com/netbox-community/netbox/issues/21117) - Avoid `ValueError` exception when `API_TOKEN_PEPPERS` is not defined
-* [#21118](https://github.com/netbox-community/netbox/issues/21118) - Address performance issue when saving sites with many assigned objects
-* [#21124](https://github.com/netbox-community/netbox/issues/21124) - Fix front/rear port mapping for module types
-* [#21134](https://github.com/netbox-community/netbox/issues/21134) - Fix bulk renaming for module types
-* [#21139](https://github.com/netbox-community/netbox/issues/21139) - Support `fields` parameter for job, object change, and object type REST API endpoints
-* [#21140](https://github.com/netbox-community/netbox/issues/21140) - Restore translation for object attribute labels on several UI views
-* [#21160](https://github.com/netbox-community/netbox/issues/21160) - Fix performance issue loading UI views caused by unintended `APISelect` choices resolution
-* [#21166](https://github.com/netbox-community/netbox/issues/21166) - Fix support for 32-bit ASN filtering in GraphQL API
-* [#21175](https://github.com/netbox-community/netbox/issues/21175) - Fix pending migrations warning when `DEFAULT_LANGUAGE` is set
-* [#21181](https://github.com/netbox-community/netbox/issues/21181) - Handle `AuthenticationFailed` exception when using an invalid API token to fetch media files
-* [#21213](https://github.com/netbox-community/netbox/issues/21213) - Tag weight field should be marked as required in UI forms
-* [#21231](https://github.com/netbox-community/netbox/issues/21231) - Presence of object types table should be checked only during migration (performance improvement)
-
----
-
-## v4.5.0 (2026-01-06)
-
-### Breaking Changes
-
-* Python 3.10 and 3.11 are no longer supported. NetBox now requires Python 3.12, 3.13, or 3.14.
-* GraphQL API queries which filter by object IDs or enums must now specify a filter lookup similar to other fields. For example, `id: 123` becomes `id: {exact: 123 }`.
-* Rendering a device or virtual machine configuration is now restricted to users with the `render_config` permission for the applicable object type.
-* Retrieval of API token plaintexts is no longer supported. The `ALLOW_TOKEN_RETRIEVAL` config parameter has been removed.
-* API tokens can no longer be reassigned from one user to another.
-* A config context assigned to a platform will now also apply to any children of that platform. (Although this is typically desired behavior, it may introduce unanticipated changes for existing deployments.)
-* The `/api/dcim/cable-terminations/` REST API endpoint is now read-only. Cable terminations must be set on cables directly via the `/api/dcim/cables/` endpoint.
-* The UI view dedicated to swapping A/Z circuit terminations has been removed.
-* The experimental HTMX navigation feature has been removed.
-* The obsolete boolean field `is_staff` has been removed from the `User` model.
-* Removal of deprecated behavior
-    * The `/api/extras/object-types/` REST API endpoint has been removed. (Use `/api/core/object-types/` instead.)
-    * Webhooks no longer specify a `model` in payload data. (Reference `object_type` instead, which includes the parent app label.)
-    * The obsolete module `core.models.contenttypes` has been removed (replaced in v4.4 by `core.models.object_types`).
-    * The `load_yaml()` and `load_json()` utility methods have been removed from the base class for custom scripts.
-
-### New Features
-
-#### Lookup Modifiers in Filter Forms ([#7604](https://github.com/netbox-community/netbox/issues/7604))
-
-Most object list filters within the UI have been extended to include optional lookup modifiers to support more complex queries. For instance, filters for numeric values now include a dropdown where a user can select "less than," "greater than," or "not" in addition to the default equivalency match. The specific modifiers available depend on the type of each filter. Plugins can register their own filtersets using the `register_filterset()` decorator to enable this new functionality.
-
-(Note that this feature does not introduce any new filters. Rather, it makes available in the UI filters which already exist.)
-
-#### Improved API Authentication Tokens ([#20210](https://github.com/netbox-community/netbox/issues/20210))
-
-This release introduces a new version of API token (v2) which implements several security improvements. HMAC hashing with a cryptographic pepper is used to authenticate these tokens, obviating the need to store plaintexts. The new tokens also employ a non-sensitive key which can be shared to identify tokens without divulging their plaintexts. We've also adopted the standard "bearer" HTTP header format, as shown below.
-
-```
-# v1 token header
-Authorization: Token <TOKEN>
-
-# v2 token header
-Authorization: Bearer nbt_<KEY>.<TOKEN>
-```
-
-Note that v2 token keys are prefixed with the fixed string `nbt_`, which can be used to aid in secret detection.
-
-Backward compatibility with legacy (v1) tokens is retained in this release. However, users are strongly encouraged to begin using only v2 tokens, as support for legacy tokens will be removed in NetBox v4.7.
-
-#### Object Ownership ([#20304](https://github.com/netbox-community/netbox/issues/20304))
-
-An optional `owner` foreign key field has been added to most models. This enables the assignment of objects to a new Owner model, which represents a set of users and/or groups. Through this relationship, we can now convey ownership of objects within NetBox natively, without needing to rely on the assignment of tags or custom fields.
-
-(Note that ownership differs significantly in function from tenancy. Ownership determines the parties responsible for the maintenance of an object, whereas as tenancy conveys an operational dependency.)
-
-#### Advanced Port Mappings ([#20564](https://github.com/netbox-community/netbox/issues/20564))
-
-The previous many-to-one mapping of front to rear ports has been expanded to support bidirectional mappings. The `rear_port` and `rear_port_position` fields on the FrontPort model have been replaced with an intermediary PortMapping model, which supports any number of assignments between front port/position pair and a rear port/position pair. This change unlocks the ability to model complex inline devices that swap individual fiber pairs between cables.
-
-#### Cable Profiles ([#20788](https://github.com/netbox-community/netbox/issues/20788))
-
-Cables can now be assigned profiles which determine how they are treated for path tracing. A profile indicates the number of discrete parallel channels or lanes carried by the cable among its endpoints. For example, a 1-to-4 breakout cable has four lanes, shared at one end via a common termination and split out at the other end to four separate terminations. Profiles, when assigned, enable NetBox to more accurately trace a specific connection within a cable, rather than the cable as a whole.
-
-The assignment of cable profiles is optional: Cable tracing will continue to operate as before for cables with no profile assigned.
-
-### Enhancements
-
-* [#16681](https://github.com/netbox-community/netbox/issues/16681) - Introduce a `render_config` permission, which is now required to render a device or virtual machine configuration
-* [#18658](https://github.com/netbox-community/netbox/issues/18658) - Add a `start_on_boot` choice field for virtual machines
-* [#19095](https://github.com/netbox-community/netbox/issues/19095) - Add support for Python 3.13 and 3.14
-* [#19338](https://github.com/netbox-community/netbox/issues/19338) - Enable filter lookups for object IDs and enums in GraphQL API queries
-* [#19523](https://github.com/netbox-community/netbox/issues/19523) - Cache the number of instances for device, module, and rack types, and enable filtering by these counts
-* [#20417](https://github.com/netbox-community/netbox/issues/20417) - Add an optional `color` field for device type power outlets
-* [#20476](https://github.com/netbox-community/netbox/issues/20476) - Once provisioned, the owner of an API token cannot be changed
-* [#20492](https://github.com/netbox-community/netbox/issues/20492) - Completely disabled the means to retrieve legacy API token plaintexts (removed the `ALLOW_TOKEN_RETRIEVAL` config parameter)
-* [#20639](https://github.com/netbox-community/netbox/issues/20639) - Apply config contexts to devices/VMs assigned any child platform of the parent platform
-* [#20834](https://github.com/netbox-community/netbox/issues/20834) - Add an `enabled` boolean field to API tokens
-* [#20917](https://github.com/netbox-community/netbox/issues/20917) - Include usage reference on API token views
-* [#20925](https://github.com/netbox-community/netbox/issues/20925) - Add optional `comments` field to all subclasses of `OrganizationalModel`
-* [#20929](https://github.com/netbox-community/netbox/issues/20929) - Require the `render_config` permission to view a rendered device/VM configuration in the UI
-* [#20936](https://github.com/netbox-community/netbox/issues/20936) - Introduce the `/api/authentication-check/` REST API endpoint for validating authentication tokens
-* [#20959](https://github.com/netbox-community/netbox/issues/20959) - Include a count of related module types for a manufacturer in the REST API
-
-### Plugins
-
-* [#13182](https://github.com/netbox-community/netbox/issues/13182) - Added `PrimaryModel`, `OrganizationalModel`, and `NestedGroupModel` to the plugins API, as well as their respective base classes for various resources
-
-### Other Changes
-
-* [#16137](https://github.com/netbox-community/netbox/issues/16137) - Remove the obsolete boolean field `is_staff` from the `User` model
-* [#17571](https://github.com/netbox-community/netbox/issues/17571) - Remove the experimental HTMX navigation feature
-* [#17936](https://github.com/netbox-community/netbox/issues/17936) - Introduce a dedicated `GFKSerializerField` for representing generic foreign keys in API serializers
-* [#19889](https://github.com/netbox-community/netbox/issues/19889) - Drop support for Python 3.10 and 3.11
-* [#19898](https://github.com/netbox-community/netbox/issues/19898) - Remove the obsolete REST API endpoint `/api/extras/object-types/`
-* [#20088](https://github.com/netbox-community/netbox/issues/20088) - Remove the non-deterministic `model` key from webhook payload data
-* [#20095](https://github.com/netbox-community/netbox/issues/20095) - Remove the obsolete module `core.models.contenttypes`
-* [#20096](https://github.com/netbox-community/netbox/issues/20096) - Remove the `load_yaml()` and `load_json()` utility methods from the `BaseScript` class
-* [#20204](https://github.com/netbox-community/netbox/issues/20204) - Started migrating object views from custom HTML templates to declarative layouts
-* [#20295](https://github.com/netbox-community/netbox/issues/20295) - Cable terminations may be modified via the REST API only by modifying the cable itself
-* [#20617](https://github.com/netbox-community/netbox/issues/20617) - Introduce `BaseModel` as the global base class for models
-* [#20683](https://github.com/netbox-community/netbox/issues/20683) - Remove the UI view dedicated to swapping A/Z circuit terminations
-* [#20926](https://github.com/netbox-community/netbox/issues/20926) - Standardize naming of GraphQL filters
-
-### REST API Changes
-
-* Most objects now include an optional `owner` foreign key field.
-* The `/api/dcim/cable-terminations` endpoint is now read-only.
-* Introduced the `/api/authentication-check/` endpoint to test REST API credentials
-* `circuits.CircuitGroup`
-    * Add optional `comments` field
-* `circuits.CircuitType`
-    * Add optional `comments` field
-* `circuits.VirtualCircuitType`
-    * Add optional `comments` field
-* `dcim.Cable`
-    * Add the optional `profile` choice field
-* `dcim.FrontPort`
-    * Removed the `rear_port` and `rear_port_position` fields
-    * Add the `positions` integer field
-    * Add the `rear_ports` list for port mappings
-* `dcim.InventoryItemRole`
-    * Add optional `comments` field
-* `dcim.Manufacturer`
-    * Add optional `comments` field
-    * Add read-only `moduletype_count` integer field
-* `dcim.ModuleType`
-    * Add read-only `module_count` integer field
-* `dcim.PowerOutletTemplate`
-    * Add optional `color` field
-* `dcim.RackRole`
-    * Add optional `comments` field
-* `dcim.RackType`
-    * Add read-only `rack_count` integer field
-* `dcim.RearPort`
-    * Add the `front_ports` list for port mappings
-* `ipam.ASNRange`
-    * Add optional `comments` field
-* `ipam.RIR`
-    * Add optional `comments` field
-* `ipam.Role`
-    * Add optional `comments` field
-* `ipam.VLANGroup`
-    * Add optional `comments` field
-* `tenancy.ContactRole`
-    * Add optional `comments` field
-* `users.Token`
-    * Add `enabled` boolean field
-* `virtualization.ClusterGroup`
-    * Add optional `comments` field
-* `virtualization.ClusterType`
-    * Add optional `comments` field
-* `virtualization.VirtualMachine`
-    * Add optional `start_on_boot` choice field
-* `vpn.TunnelGroup`
-    * Add optional `comments` field

mkdocs.yml

@@ -322,7 +322,6 @@ nav:
         - git Cheat Sheet: 'development/git-cheat-sheet.md'
     - Release Notes:
         - Summary: 'release-notes/index.md'
-        - Version 4.5: 'release-notes/version-4.5.md'
         - Version 4.4: 'release-notes/version-4.4.md'
         - Version 4.3: 'release-notes/version-4.3.md'
         - Version 4.2: 'release-notes/version-4.2.md'

netbox/circuits/api/serializers_/circuits.py

@@ -38,7 +38,7 @@ class CircuitTypeSerializer(OrganizationalModelSerializer):
     class Meta:
         model = CircuitType
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'slug', 'color', 'description', 'owner', 'comments', 'tags',
+            'id', 'url', 'display_url', 'display', 'name', 'slug', 'color', 'description', 'owner', 'tags',
             'custom_fields', 'created', 'last_updated', 'circuit_count',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'slug', 'description', 'circuit_count')
@@ -71,7 +71,7 @@ class CircuitGroupSerializer(OrganizationalModelSerializer):
     class Meta:
         model = CircuitGroup
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'slug', 'description', 'tenant', 'owner', 'comments', 'tags',
+            'id', 'url', 'display_url', 'display', 'name', 'slug', 'description', 'tenant', 'owner', 'tags',
             'custom_fields', 'created', 'last_updated', 'circuit_count'
         ]
         brief_fields = ('id', 'url', 'display', 'name')
@@ -161,7 +161,7 @@ class VirtualCircuitTypeSerializer(OrganizationalModelSerializer):
     class Meta:
         model = VirtualCircuitType
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'slug', 'color', 'description', 'owner', 'comments', 'tags',
+            'id', 'url', 'display_url', 'display', 'name', 'slug', 'color', 'description', 'owner', 'tags',
             'custom_fields', 'created', 'last_updated', 'virtual_circuit_count',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'slug', 'description', 'virtual_circuit_count')

netbox/circuits/filtersets.py

@@ -7,11 +7,11 @@ from dcim.filtersets import CabledObjectFilterSet
 from dcim.models import Interface, Location, Region, Site, SiteGroup
 from ipam.models import ASN
 from netbox.filtersets import NetBoxModelFilterSet, OrganizationalModelFilterSet, PrimaryModelFilterSet
+from netbox.plugins.registration import register_filterset
 from tenancy.filtersets import ContactModelFilterSet, TenancyFilterSet
 from utilities.filters import (
     ContentTypeFilter, MultiValueCharFilter, MultiValueNumberFilter, TreeNodeMultipleChoiceFilter,
 )
-from utilities.filtersets import register_filterset
 from .choices import *
 from .models import *
 
@@ -353,7 +353,7 @@ class CircuitTerminationFilterSet(NetBoxModelFilterSet, CabledObjectFilterSet):
         model = CircuitTermination
         fields = (
             'id', 'termination_id', 'term_side', 'port_speed', 'upstream_speed', 'xconnect_id', 'description',
-            'mark_connected', 'pp_info', 'cable_end', 'cable_connector',
+            'mark_connected', 'pp_info', 'cable_end', 'cable_position',
         )
 
     def search(self, queryset, name, value):

netbox/circuits/forms/bulk_edit.py

@@ -99,7 +99,7 @@ class CircuitTypeBulkEditForm(OrganizationalModelBulkEditForm):
     fieldsets = (
         FieldSet('color', 'description'),
     )
-    nullable_fields = ('color', 'description', 'comments')
+    nullable_fields = ('color', 'description')
 
 
 class CircuitBulkEditForm(PrimaryModelBulkEditForm):
@@ -241,7 +241,7 @@ class CircuitGroupBulkEditForm(OrganizationalModelBulkEditForm):
 
     model = CircuitGroup
     nullable_fields = (
-        'description', 'tenant', 'comments',
+        'description', 'tenant',
     )
 
 
@@ -274,7 +274,7 @@ class VirtualCircuitTypeBulkEditForm(OrganizationalModelBulkEditForm):
     fieldsets = (
         FieldSet('color', 'description'),
     )
-    nullable_fields = ('color', 'description', 'comments')
+    nullable_fields = ('color', 'description')
 
 
 class VirtualCircuitBulkEditForm(PrimaryModelBulkEditForm):

netbox/circuits/forms/bulk_import.py

@@ -73,7 +73,7 @@ class CircuitTypeImportForm(OrganizationalModelImportForm):
 
     class Meta:
         model = CircuitType
-        fields = ('name', 'slug', 'color', 'description', 'owner', 'comments', 'tags')
+        fields = ('name', 'slug', 'color', 'description', 'owner', 'tags')
 
 
 class CircuitImportForm(PrimaryModelImportForm):
@@ -176,7 +176,7 @@ class CircuitGroupImportForm(OrganizationalModelImportForm):
 
     class Meta:
         model = CircuitGroup
-        fields = ('name', 'slug', 'description', 'tenant', 'owner', 'comments', 'tags')
+        fields = ('name', 'slug', 'description', 'tenant', 'owner', 'tags')
 
 
 class CircuitGroupAssignmentImportForm(NetBoxModelImportForm):
@@ -199,7 +199,7 @@ class VirtualCircuitTypeImportForm(OrganizationalModelImportForm):
 
     class Meta:
         model = VirtualCircuitType
-        fields = ('name', 'slug', 'color', 'description', 'owner', 'comments', 'tags')
+        fields = ('name', 'slug', 'color', 'description', 'owner', 'tags')
 
 
 class VirtualCircuitImportForm(PrimaryModelImportForm):

netbox/circuits/forms/model_forms.py

@@ -97,7 +97,7 @@ class CircuitTypeForm(OrganizationalModelForm):
     class Meta:
         model = CircuitType
         fields = [
-            'name', 'slug', 'color', 'description', 'comments', 'tags',
+            'name', 'slug', 'color', 'description', 'tags',
         ]
 
 
@@ -236,7 +236,7 @@ class CircuitGroupForm(TenancyForm, OrganizationalModelForm):
     class Meta:
         model = CircuitGroup
         fields = [
-            'name', 'slug', 'description', 'tenant_group', 'tenant', 'owner', 'comments', 'tags',
+            'name', 'slug', 'description', 'tenant_group', 'tenant', 'owner', 'tags',
         ]
 
 
@@ -307,7 +307,7 @@ class VirtualCircuitTypeForm(OrganizationalModelForm):
     class Meta:
         model = VirtualCircuitType
         fields = [
-            'name', 'slug', 'color', 'description', 'owner', 'comments', 'tags',
+            'name', 'slug', 'color', 'description', 'owner', 'tags',
         ]
 
 

netbox/circuits/graphql/filter_mixins.py

@@ -3,18 +3,17 @@ from typing import Annotated, TYPE_CHECKING
 
 import strawberry
 import strawberry_django
-from strawberry_django import BaseFilterLookup
+
+from netbox.graphql.filter_mixins import OrganizationalModelFilterMixin
 
 if TYPE_CHECKING:
     from netbox.graphql.enums import ColorEnum
 
 __all__ = (
-    'CircuitTypeFilterMixin',
+    'BaseCircuitTypeFilterMixin',
 )
 
 
 @dataclass
-class CircuitTypeFilterMixin:
-    color: BaseFilterLookup[Annotated['ColorEnum', strawberry.lazy('netbox.graphql.enums')]] | None = (
-        strawberry_django.filter_field()
-    )
+class BaseCircuitTypeFilterMixin(OrganizationalModelFilterMixin):
+    color: Annotated['ColorEnum', strawberry.lazy('netbox.graphql.enums')] | None = strawberry_django.filter_field()

netbox/circuits/graphql/filters.py

@@ -7,12 +7,17 @@ from strawberry.scalars import ID
 from strawberry_django import BaseFilterLookup, FilterLookup, DateFilterLookup
 
 from circuits import models
-from circuits.graphql.filter_mixins import CircuitTypeFilterMixin
+from core.graphql.filter_mixins import BaseObjectTypeFilterMixin, ChangeLogFilterMixin
 from dcim.graphql.filter_mixins import CabledObjectModelFilterMixin
 from extras.graphql.filter_mixins import CustomFieldsFilterMixin, TagsFilterMixin
-from netbox.graphql.filter_mixins import DistanceFilterMixin, ImageAttachmentFilterMixin
-from netbox.graphql.filters import ChangeLoggedModelFilter, OrganizationalModelFilter, PrimaryModelFilter
+from netbox.graphql.filter_mixins import (
+    DistanceFilterMixin,
+    ImageAttachmentFilterMixin,
+    OrganizationalModelFilterMixin,
+    PrimaryModelFilterMixin,
+)
 from tenancy.graphql.filter_mixins import ContactFilterMixin, TenancyFilterMixin
+from .filter_mixins import BaseCircuitTypeFilterMixin
 
 if TYPE_CHECKING:
     from core.graphql.filters import ContentTypeFilter
@@ -38,9 +43,10 @@ __all__ = (
 
 @strawberry_django.filter_type(models.CircuitTermination, lookups=True)
 class CircuitTerminationFilter(
+    BaseObjectTypeFilterMixin,
     CustomFieldsFilterMixin,
     TagsFilterMixin,
-    ChangeLoggedModelFilter,
+    ChangeLogFilterMixin,
     CabledObjectModelFilterMixin,
 ):
     circuit: Annotated['CircuitFilter', strawberry.lazy('circuits.graphql.filters')] | None = (
@@ -89,7 +95,7 @@ class CircuitFilter(
     ImageAttachmentFilterMixin,
     DistanceFilterMixin,
     TenancyFilterMixin,
-    PrimaryModelFilter
+    PrimaryModelFilterMixin
 ):
     cid: FilterLookup[str] | None = strawberry_django.filter_field()
     provider: Annotated['ProviderFilter', strawberry.lazy('circuits.graphql.filters')] | None = (
@@ -118,17 +124,19 @@ class CircuitFilter(
 
 
 @strawberry_django.filter_type(models.CircuitType, lookups=True)
-class CircuitTypeFilter(CircuitTypeFilterMixin, OrganizationalModelFilter):
+class CircuitTypeFilter(BaseCircuitTypeFilterMixin):
     pass
 
 
 @strawberry_django.filter_type(models.CircuitGroup, lookups=True)
-class CircuitGroupFilter(TenancyFilterMixin, OrganizationalModelFilter):
+class CircuitGroupFilter(TenancyFilterMixin, OrganizationalModelFilterMixin):
     pass
 
 
 @strawberry_django.filter_type(models.CircuitGroupAssignment, lookups=True)
-class CircuitGroupAssignmentFilter(CustomFieldsFilterMixin, TagsFilterMixin, ChangeLoggedModelFilter):
+class CircuitGroupAssignmentFilter(
+    BaseObjectTypeFilterMixin, CustomFieldsFilterMixin, TagsFilterMixin, ChangeLogFilterMixin
+):
     member_type: Annotated['ContentTypeFilter', strawberry.lazy('core.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -143,7 +151,7 @@ class CircuitGroupAssignmentFilter(CustomFieldsFilterMixin, TagsFilterMixin, Cha
 
 
 @strawberry_django.filter_type(models.Provider, lookups=True)
-class ProviderFilter(ContactFilterMixin, PrimaryModelFilter):
+class ProviderFilter(ContactFilterMixin, PrimaryModelFilterMixin):
     name: FilterLookup[str] | None = strawberry_django.filter_field()
     slug: FilterLookup[str] | None = strawberry_django.filter_field()
     asns: Annotated['ASNFilter', strawberry.lazy('ipam.graphql.filters')] | None = strawberry_django.filter_field()
@@ -153,7 +161,7 @@ class ProviderFilter(ContactFilterMixin, PrimaryModelFilter):
 
 
 @strawberry_django.filter_type(models.ProviderAccount, lookups=True)
-class ProviderAccountFilter(ContactFilterMixin, PrimaryModelFilter):
+class ProviderAccountFilter(ContactFilterMixin, PrimaryModelFilterMixin):
     provider: Annotated['ProviderFilter', strawberry.lazy('circuits.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -163,7 +171,7 @@ class ProviderAccountFilter(ContactFilterMixin, PrimaryModelFilter):
 
 
 @strawberry_django.filter_type(models.ProviderNetwork, lookups=True)
-class ProviderNetworkFilter(PrimaryModelFilter):
+class ProviderNetworkFilter(PrimaryModelFilterMixin):
     name: FilterLookup[str] | None = strawberry_django.filter_field()
     provider: Annotated['ProviderFilter', strawberry.lazy('circuits.graphql.filters')] | None = (
         strawberry_django.filter_field()
@@ -173,12 +181,12 @@ class ProviderNetworkFilter(PrimaryModelFilter):
 
 
 @strawberry_django.filter_type(models.VirtualCircuitType, lookups=True)
-class VirtualCircuitTypeFilter(CircuitTypeFilterMixin, OrganizationalModelFilter):
+class VirtualCircuitTypeFilter(BaseCircuitTypeFilterMixin):
     pass
 
 
 @strawberry_django.filter_type(models.VirtualCircuit, lookups=True)
-class VirtualCircuitFilter(TenancyFilterMixin, PrimaryModelFilter):
+class VirtualCircuitFilter(TenancyFilterMixin, PrimaryModelFilterMixin):
     cid: FilterLookup[str] | None = strawberry_django.filter_field()
     provider_network: Annotated['ProviderNetworkFilter', strawberry.lazy('circuits.graphql.filters')] | None = (
         strawberry_django.filter_field()
@@ -201,7 +209,9 @@ class VirtualCircuitFilter(TenancyFilterMixin, PrimaryModelFilter):
 
 
 @strawberry_django.filter_type(models.VirtualCircuitTermination, lookups=True)
-class VirtualCircuitTerminationFilter(CustomFieldsFilterMixin, TagsFilterMixin, ChangeLoggedModelFilter):
+class VirtualCircuitTerminationFilter(
+    BaseObjectTypeFilterMixin, CustomFieldsFilterMixin, TagsFilterMixin, ChangeLogFilterMixin
+):
     virtual_circuit: Annotated['VirtualCircuitFilter', strawberry.lazy('circuits.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )

netbox/circuits/migrations/0054_cable_connector_positions.py

@@ -1,39 +0,0 @@
-import django.contrib.postgres.fields
-import django.core.validators
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-    dependencies = [
-        ('circuits', '0053_owner'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='circuittermination',
-            name='cable_connector',
-            field=models.PositiveSmallIntegerField(
-                blank=True,
-                null=True,
-                validators=[
-                    django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(256)
-                ],
-            ),
-        ),
-        migrations.AddField(
-            model_name='circuittermination',
-            name='cable_positions',
-            field=django.contrib.postgres.fields.ArrayField(
-                base_field=models.PositiveSmallIntegerField(
-                    validators=[
-                        django.core.validators.MinValueValidator(1),
-                        django.core.validators.MaxValueValidator(1024),
-                    ]
-                ),
-                blank=True,
-                null=True,
-                size=None,
-            ),
-        ),
-    ]

netbox/circuits/migrations/0054_cable_position.py

@@ -0,0 +1,23 @@
+import django.core.validators
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ('circuits', '0053_owner'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='circuittermination',
+            name='cable_position',
+            field=models.PositiveIntegerField(
+                blank=True,
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(1),
+                    django.core.validators.MaxValueValidator(1024),
+                ],
+            ),
+        ),
+    ]

netbox/circuits/migrations/0055_add_comments_to_organizationalmodel.py

@@ -1,28 +0,0 @@
-# Generated by Django 5.2.8 on 2025-12-08 17:38
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('circuits', '0054_cable_connector_positions'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='circuitgroup',
-            name='comments',
-            field=models.TextField(blank=True),
-        ),
-        migrations.AddField(
-            model_name='circuittype',
-            name='comments',
-            field=models.TextField(blank=True),
-        ),
-        migrations.AddField(
-            model_name='virtualcircuittype',
-            name='comments',
-            field=models.TextField(blank=True),
-        ),
-    ]

netbox/circuits/migrations/0056_gfk_indexes.py

@@ -1,17 +0,0 @@
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-    dependencies = [
-        ('circuits', '0055_add_comments_to_organizationalmodel'),
-        ('contenttypes', '0002_remove_content_type_name'),
-        ('dcim', '0224_add_comments_to_organizationalmodel'),
-        ('extras', '0134_owner'),
-    ]
-
-    operations = [
-        migrations.AddIndex(
-            model_name='circuittermination',
-            index=models.Index(fields=['termination_type', 'termination_id'], name='circuits_ci_termina_505dda_idx'),
-        ),
-    ]

netbox/circuits/models/circuits.py

@@ -335,9 +335,6 @@ class CircuitTermination(
                 name='%(app_label)s_%(class)s_unique_circuit_term_side'
             ),
         )
-        indexes = (
-            models.Index(fields=('termination_type', 'termination_id')),
-        )
         verbose_name = _('circuit termination')
         verbose_name_plural = _('circuit terminations')
 

netbox/circuits/search.py

@@ -20,7 +20,6 @@ class CircuitGroupIndex(SearchIndex):
         ('name', 100),
         ('slug', 110),
         ('description', 500),
-        ('comments', 5000),
     )
     display_attrs = ('description',)
 
@@ -45,7 +44,6 @@ class CircuitTypeIndex(SearchIndex):
         ('name', 100),
         ('slug', 110),
         ('description', 500),
-        ('comments', 5000),
     )
     display_attrs = ('description',)
 
@@ -111,6 +109,5 @@ class VirtualCircuitTypeIndex(SearchIndex):
         ('name', 100),
         ('slug', 110),
         ('description', 500),
-        ('comments', 5000),
     )
     display_attrs = ('description',)

netbox/circuits/tables/circuits.py

@@ -40,8 +40,8 @@ class CircuitTypeTable(OrganizationalModelTable):
     class Meta(OrganizationalModelTable.Meta):
         model = CircuitType
         fields = (
-            'pk', 'id', 'name', 'circuit_count', 'color', 'description', 'slug', 'comments', 'tags', 'created',
-            'last_updated', 'actions',
+            'pk', 'id', 'name', 'circuit_count', 'color', 'description', 'slug', 'tags', 'created', 'last_updated',
+            'actions',
         )
         default_columns = ('pk', 'name', 'circuit_count', 'color', 'description')
 
@@ -175,7 +175,7 @@ class CircuitGroupTable(OrganizationalModelTable):
     class Meta(OrganizationalModelTable.Meta):
         model = CircuitGroup
         fields = (
-            'pk', 'name', 'description', 'circuit_group_assignment_count', 'comments', 'tags',
+            'pk', 'name', 'description', 'circuit_group_assignment_count', 'tags',
             'created', 'last_updated', 'actions',
         )
         default_columns = ('pk', 'name', 'description', 'circuit_group_assignment_count')

netbox/circuits/tests/test_filtersets.py

@@ -433,7 +433,7 @@ class CircuitTestCase(TestCase, ChangeLoggedFilterSetTests):
 class CircuitTerminationTestCase(TestCase, ChangeLoggedFilterSetTests):
     queryset = CircuitTermination.objects.all()
     filterset = CircuitTerminationFilterSet
-    ignore_fields = ('cable', 'cable_positions')
+    ignore_fields = ('cable',)
 
     @classmethod
     def setUpTestData(cls):

netbox/core/api/serializers_/data.py

@@ -44,4 +44,3 @@ class DataFileSerializer(NetBoxModelSerializer):
             'id', 'url', 'display_url', 'display', 'source', 'path', 'last_updated', 'size', 'hash',
         ]
         brief_fields = ('id', 'url', 'display', 'path')
-        read_only_fields = ['path', 'last_updated', 'size', 'hash']

netbox/core/api/views.py

@@ -11,6 +11,7 @@ from rest_framework.decorators import action
 from rest_framework.exceptions import PermissionDenied
 from rest_framework.response import Response
 from rest_framework.routers import APIRootView
+from rest_framework.viewsets import ReadOnlyModelViewSet
 from rq.job import Job as RQ_Job
 from rq.worker import Worker
 
@@ -63,7 +64,7 @@ class DataFileViewSet(NetBoxReadOnlyModelViewSet):
     filterset_class = filtersets.DataFileFilterSet
 
 
-class JobViewSet(NetBoxReadOnlyModelViewSet):
+class JobViewSet(ReadOnlyModelViewSet):
     """
     Retrieve a list of job results
     """
@@ -72,20 +73,19 @@ class JobViewSet(NetBoxReadOnlyModelViewSet):
     filterset_class = filtersets.JobFilterSet
 
 
-class ObjectChangeViewSet(NetBoxReadOnlyModelViewSet):
+class ObjectChangeViewSet(ReadOnlyModelViewSet):
     """
     Retrieve a list of recent changes.
     """
     metadata_class = ContentTypeMetadata
-    queryset = ObjectChange.objects.all()
     serializer_class = serializers.ObjectChangeSerializer
     filterset_class = filtersets.ObjectChangeFilterSet
 
     def get_queryset(self):
-        return super().get_queryset().valid_models()
+        return ObjectChange.objects.valid_models()
 
 
-class ObjectTypeViewSet(NetBoxReadOnlyModelViewSet):
+class ObjectTypeViewSet(ReadOnlyModelViewSet):
     """
     Read-only list of ObjectTypes.
     """
@@ -94,16 +94,6 @@ class ObjectTypeViewSet(NetBoxReadOnlyModelViewSet):
     serializer_class = serializers.ObjectTypeSerializer
     filterset_class = filtersets.ObjectTypeFilterSet
 
-    def initial(self, request, *args, **kwargs):
-        """
-        Override initial() to skip the restrict() call since ObjectType (a ContentType proxy)
-        doesn't use RestrictedQuerySet and is publicly accessible metadata.
-        """
-        # Call GenericViewSet.initial() directly, skipping BaseViewSet.initial()
-        # which would try to call restrict() on the queryset
-        from rest_framework.viewsets import GenericViewSet
-        GenericViewSet.initial(self, request, *args, **kwargs)
-
 
 class BaseRQViewSet(viewsets.ViewSet):
     """

netbox/core/filtersets.py

@@ -4,10 +4,10 @@ from django.db.models import Q
 from django.utils.translation import gettext as _
 
 from netbox.filtersets import BaseFilterSet, ChangeLoggedModelFilterSet, PrimaryModelFilterSet
+from netbox.plugins.registration import register_filterset
 from netbox.utils import get_data_backend_choices
 from users.models import User
 from utilities.filters import ContentTypeFilter
-from utilities.filtersets import register_filterset
 from .choices import *
 from .models import *
 

netbox/core/graphql/filter_mixins.py

@@ -4,18 +4,31 @@ from typing import Annotated, TYPE_CHECKING
 
 import strawberry
 import strawberry_django
-from strawberry_django import DatetimeFilterLookup
+from strawberry import ID
+from strawberry_django import FilterLookup, DatetimeFilterLookup
 
 if TYPE_CHECKING:
     from .filters import *
 
 __all__ = (
-    'ChangeLoggingMixin',
+    'BaseFilterMixin',
+    'BaseObjectTypeFilterMixin',
+    'ChangeLogFilterMixin',
 )
 
 
+# @strawberry.input
+class BaseFilterMixin: ...
+
+
 @dataclass
-class ChangeLoggingMixin:
+class BaseObjectTypeFilterMixin(BaseFilterMixin):
+    id: FilterLookup[ID] | None = strawberry_django.filter_field()
+
+
+@dataclass
+class ChangeLogFilterMixin(BaseFilterMixin):
+    id: FilterLookup[ID] | None = strawberry_django.filter_field()
     # TODO: "changelog" is not a valid field name; needs to be updated for ObjectChange
     changelog: Annotated['ObjectChangeFilter', strawberry.lazy('core.graphql.filters')] | None = (
         strawberry_django.filter_field()

netbox/core/graphql/filters.py

@@ -8,7 +8,8 @@ from strawberry.scalars import ID
 from strawberry_django import BaseFilterLookup, DatetimeFilterLookup, FilterLookup
 
 from core import models
-from netbox.graphql.filters import BaseModelFilter, PrimaryModelFilter
+from core.graphql.filter_mixins import BaseFilterMixin
+from netbox.graphql.filter_mixins import PrimaryModelFilterMixin
 from .enums import *
 
 if TYPE_CHECKING:
@@ -24,7 +25,8 @@ __all__ = (
 
 
 @strawberry_django.filter_type(models.DataFile, lookups=True)
-class DataFileFilter(BaseModelFilter):
+class DataFileFilter(BaseFilterMixin):
+    id: FilterLookup[ID] | None = strawberry_django.filter_field()
     created: DatetimeFilterLookup[datetime] | None = strawberry_django.filter_field()
     last_updated: DatetimeFilterLookup[datetime] | None = strawberry_django.filter_field()
     source: Annotated['DataSourceFilter', strawberry.lazy('core.graphql.filters')] | None = (
@@ -39,7 +41,7 @@ class DataFileFilter(BaseModelFilter):
 
 
 @strawberry_django.filter_type(models.DataSource, lookups=True)
-class DataSourceFilter(PrimaryModelFilter):
+class DataSourceFilter(PrimaryModelFilterMixin):
     name: FilterLookup[str] | None = strawberry_django.filter_field()
     type: FilterLookup[str] | None = strawberry_django.filter_field()
     source_url: FilterLookup[str] | None = strawberry_django.filter_field()
@@ -58,7 +60,8 @@ class DataSourceFilter(PrimaryModelFilter):
 
 
 @strawberry_django.filter_type(models.ObjectChange, lookups=True)
-class ObjectChangeFilter(BaseModelFilter):
+class ObjectChangeFilter(BaseFilterMixin):
+    id: FilterLookup[ID] | None = strawberry_django.filter_field()
     time: DatetimeFilterLookup[datetime] | None = strawberry_django.filter_field()
     user: Annotated['UserFilter', strawberry.lazy('users.graphql.filters')] | None = strawberry_django.filter_field()
     user_name: FilterLookup[str] | None = strawberry_django.filter_field()
@@ -85,6 +88,7 @@ class ObjectChangeFilter(BaseModelFilter):
 
 
 @strawberry_django.filter_type(DjangoContentType, lookups=True)
-class ContentTypeFilter(BaseModelFilter):
+class ContentTypeFilter(BaseFilterMixin):
+    id: FilterLookup[ID] | None = strawberry_django.filter_field()
     app_label: FilterLookup[str] | None = strawberry_django.filter_field()
     model: FilterLookup[str] | None = strawberry_django.filter_field()

netbox/core/models/config.py

@@ -63,20 +63,16 @@ class ConfigRevision(models.Model):
             return reverse('core:config')  # Default config view
         return reverse('core:configrevision', args=[self.pk])
 
-    def activate(self, update_db=True):
+    def activate(self):
         """
         Cache the configuration data.
-
-        Parameters:
-            update_db: Mark the ConfigRevision as active in the database (default: True)
         """
         cache.set('config', self.data, None)
         cache.set('config_version', self.pk, None)
 
-        if update_db:
-            # Set all instances of ConfigRevision to false and set this instance to true
-            ConfigRevision.objects.all().update(active=False)
-            ConfigRevision.objects.filter(pk=self.pk).update(active=True)
+        # Set all instances of ConfigRevision to false and set this instance to true
+        ConfigRevision.objects.all().update(active=False)
+        ConfigRevision.objects.filter(pk=self.pk).update(active=True)
 
     activate.alters_data = True
 

netbox/core/models/data.py

@@ -12,7 +12,7 @@ from django.core.validators import RegexValidator
 from django.db import models
 from django.urls import reverse
 from django.utils import timezone
-from django.utils.translation import gettext_lazy as _
+from django.utils.translation import gettext as _
 
 from netbox.constants import CENSOR_TOKEN, CENSOR_TOKEN_CHANGED
 from netbox.models import PrimaryModel
@@ -128,24 +128,9 @@ class DataSource(JobsMixin, PrimaryModel):
         # Ensure URL scheme matches selected type
         if self.backend_class.is_local and self.url_scheme not in ('file', ''):
             raise ValidationError({
-                'source_url': _("URLs for local sources must start with {scheme} (or specify no scheme)").format(
-                    scheme='file://'
-                )
+                'source_url': "URLs for local sources must start with file:// (or specify no scheme)"
             })
 
-    def save(self, *args, **kwargs):
-
-        # If recurring sync is disabled for an existing DataSource, clear any pending sync jobs for it and reset its
-        # "queued" status
-        if not self._state.adding and not self.sync_interval:
-            self.jobs.filter(status=JobStatusChoices.STATUS_PENDING).delete()
-            if self.status == DataSourceStatusChoices.QUEUED and self.last_synced:
-                self.status = DataSourceStatusChoices.COMPLETED
-            elif self.status == DataSourceStatusChoices.QUEUED:
-                self.status = DataSourceStatusChoices.NEW
-
-        super().save(*args, **kwargs)
-
     def to_objectchange(self, action):
         objectchange = super().to_objectchange(action)
 

netbox/core/models/object_types.py

@@ -35,10 +35,6 @@ class ObjectTypeQuerySet(models.QuerySet):
 
 class ObjectTypeManager(models.Manager):
 
-    # TODO: Remove this in NetBox v5.0
-    # Cache the result of introspection to avoid repeated queries.
-    _table_exists = False
-
     def get_queryset(self):
         return ObjectTypeQuerySet(self.model, using=self._db)
 
@@ -73,12 +69,10 @@ class ObjectTypeManager(models.Manager):
         # TODO: Remove this in NetBox v5.0
         # If the ObjectType table has not yet been provisioned (e.g. because we're in a pre-v4.4 migration),
         # fall back to ContentType.
-        if not ObjectTypeManager._table_exists:
-            if 'core_objecttype' not in connection.introspection.table_names():
-                ct = ContentType.objects.get_for_model(model, for_concrete_model=for_concrete_model)
-                ct.features = get_model_features(ct.model_class())
-                return ct
-            ObjectTypeManager._table_exists = True
+        if 'core_objecttype' not in connection.introspection.table_names():
+            ct = ContentType.objects.get_for_model(model, for_concrete_model=for_concrete_model)
+            ct.features = get_model_features(ct.model_class())
+            return ct
 
         if not inspect.isclass(model):
             model = model.__class__

netbox/core/signals.py

@@ -3,7 +3,7 @@ from threading import local
 
 from django.contrib.contenttypes.models import ContentType
 from django.core.exceptions import ObjectDoesNotExist, ValidationError
-from django.db.models import CASCADE, RESTRICT
+from django.db.models import CASCADE
 from django.db.models.fields.reverse_related import ManyToManyRel, ManyToOneRel
 from django.db.models.signals import m2m_changed, post_migrate, post_save, pre_delete
 from django.dispatch import receiver, Signal
@@ -221,7 +221,7 @@ def handle_deleted_object(sender, instance, **kwargs):
             obj.snapshot()  # Ensure the change record includes the "before" state
             if type(relation) is ManyToManyRel:
                 getattr(obj, related_field_name).remove(instance)
-            elif type(relation) is ManyToOneRel and relation.null and relation.on_delete not in (CASCADE, RESTRICT):
+            elif type(relation) is ManyToOneRel and relation.null and relation.on_delete is not CASCADE:
                 setattr(obj, related_field_name, None)
                 obj.save()
 

netbox/dcim/api/serializers_/base.py

@@ -2,12 +2,10 @@ from drf_spectacular.types import OpenApiTypes
 from drf_spectacular.utils import extend_schema_field
 from rest_framework import serializers
 
-from dcim.models import FrontPort, FrontPortTemplate, PortMapping, PortTemplateMapping, RearPort, RearPortTemplate
 from utilities.api import get_serializer_for_model
 
 __all__ = (
     'ConnectedEndpointsSerializer',
-    'PortSerializer',
 )
 
 
@@ -37,53 +35,3 @@ class ConnectedEndpointsSerializer(serializers.ModelSerializer):
     @extend_schema_field(serializers.BooleanField)
     def get_connected_endpoints_reachable(self, obj):
         return obj._path and obj._path.is_complete and obj._path.is_active
-
-
-class PortSerializer(serializers.ModelSerializer):
-    """
-    Base serializer for front & rear port and port templates.
-    """
-    @property
-    def _mapper(self):
-        """
-        Return the model and ForeignKey field name used to track port mappings for this model.
-        """
-        if self.Meta.model is FrontPort:
-            return PortMapping, 'front_port'
-        if self.Meta.model is RearPort:
-            return PortMapping, 'rear_port'
-        if self.Meta.model is FrontPortTemplate:
-            return PortTemplateMapping, 'front_port'
-        if self.Meta.model is RearPortTemplate:
-            return PortTemplateMapping, 'rear_port'
-        raise ValueError(f"Could not determine mapping details for {self.__class__}")
-
-    def create(self, validated_data):
-        mappings = validated_data.pop('mappings', [])
-        instance = super().create(validated_data)
-
-        # Create port mappings
-        mapping_model, fk_name = self._mapper
-        for attrs in mappings:
-            mapping_model.objects.create(**{
-                fk_name: instance,
-                **attrs,
-            })
-
-        return instance
-
-    def update(self, instance, validated_data):
-        mappings = validated_data.pop('mappings', None)
-        instance = super().update(instance, validated_data)
-
-        if mappings is not None:
-            # Update port mappings
-            mapping_model, fk_name = self._mapper
-            mapping_model.objects.filter(**{fk_name: instance}).delete()
-            for attrs in mappings:
-                mapping_model.objects.create(**{
-                    fk_name: instance,
-                    **attrs,
-                })
-
-        return instance

netbox/dcim/api/serializers_/cables.py

@@ -61,12 +61,11 @@ class CableTerminationSerializer(NetBoxModelSerializer):
         model = CableTermination
         fields = [
             'id', 'url', 'display', 'cable', 'cable_end', 'termination_type', 'termination_id',
-            'termination', 'connector', 'positions', 'created', 'last_updated',
+            'termination', 'position', 'created', 'last_updated',
         ]
         read_only_fields = fields
         brief_fields = (
-            'id', 'url', 'display', 'cable', 'cable_end', 'connector', 'positions', 'termination_type',
-            'termination_id',
+            'id', 'url', 'display', 'cable', 'cable_end', 'position', 'termination_type', 'termination_id',
         )
 
 

netbox/dcim/api/serializers_/device_components.py

@@ -1,26 +1,25 @@
-from django.contrib.contenttypes.models import ContentType
 from django.utils.translation import gettext as _
+from django.contrib.contenttypes.models import ContentType
 from rest_framework import serializers
 
 from dcim.choices import *
 from dcim.constants import *
 from dcim.models import (
-    ConsolePort, ConsoleServerPort, DeviceBay, FrontPort, Interface, InventoryItem, ModuleBay, PortMapping,
-    PowerOutlet, PowerPort, RearPort, VirtualDeviceContext,
+    ConsolePort, ConsoleServerPort, DeviceBay, FrontPort, Interface, InventoryItem, ModuleBay, PowerOutlet, PowerPort,
+    RearPort, VirtualDeviceContext,
 )
 from ipam.api.serializers_.vlans import VLANSerializer, VLANTranslationPolicySerializer
 from ipam.api.serializers_.vrfs import VRFSerializer
 from ipam.models import VLAN
 from netbox.api.fields import ChoiceField, ContentTypeField, SerializedPKRelatedField
 from netbox.api.gfk_fields import GFKSerializerField
-from netbox.api.serializers import NetBoxModelSerializer
-from users.api.serializers_.mixins import OwnerMixin
+from netbox.api.serializers import NetBoxModelSerializer, WritableNestedSerializer
 from vpn.api.serializers_.l2vpn import L2VPNTerminationSerializer
 from wireless.api.serializers_.nested import NestedWirelessLinkSerializer
 from wireless.api.serializers_.wirelesslans import WirelessLANSerializer
 from wireless.choices import *
 from wireless.models import WirelessLAN
-from .base import ConnectedEndpointsSerializer, PortSerializer
+from .base import ConnectedEndpointsSerializer
 from .cables import CabledObjectSerializer
 from .devices import DeviceSerializer, MACAddressSerializer, ModuleSerializer, VirtualDeviceContextSerializer
 from .manufacturers import ManufacturerSerializer
@@ -41,12 +40,7 @@ __all__ = (
 )
 
 
-class ConsoleServerPortSerializer(
-    OwnerMixin,
-    NetBoxModelSerializer,
-    CabledObjectSerializer,
-    ConnectedEndpointsSerializer
-):
+class ConsoleServerPortSerializer(NetBoxModelSerializer, CabledObjectSerializer, ConnectedEndpointsSerializer):
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -70,18 +64,13 @@ class ConsoleServerPortSerializer(
         fields = [
             'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'speed', 'description',
             'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type', 'connected_endpoints',
-            'connected_endpoints_type', 'connected_endpoints_reachable', 'owner', 'tags', 'custom_fields', 'created',
+            'connected_endpoints_type', 'connected_endpoints_reachable', 'tags', 'custom_fields', 'created',
             'last_updated', '_occupied',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description', 'cable', '_occupied')
 
 
-class ConsolePortSerializer(
-    OwnerMixin,
-    NetBoxModelSerializer,
-    CabledObjectSerializer,
-    ConnectedEndpointsSerializer
-):
+class ConsolePortSerializer(NetBoxModelSerializer, CabledObjectSerializer, ConnectedEndpointsSerializer):
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -105,18 +94,13 @@ class ConsolePortSerializer(
         fields = [
             'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'speed', 'description',
             'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type', 'connected_endpoints',
-            'connected_endpoints_type', 'connected_endpoints_reachable', 'owner', 'tags', 'custom_fields', 'created',
+            'connected_endpoints_type', 'connected_endpoints_reachable', 'tags', 'custom_fields', 'created',
             'last_updated', '_occupied',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description', 'cable', '_occupied')
 
 
-class PowerPortSerializer(
-    OwnerMixin,
-    NetBoxModelSerializer,
-    CabledObjectSerializer,
-    ConnectedEndpointsSerializer
-):
+class PowerPortSerializer(NetBoxModelSerializer, CabledObjectSerializer, ConnectedEndpointsSerializer):
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -136,18 +120,13 @@ class PowerPortSerializer(
         fields = [
             'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'maximum_draw',
             'allocated_draw', 'description', 'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type',
-            'connected_endpoints', 'connected_endpoints_type', 'connected_endpoints_reachable', 'owner', 'tags',
-            'custom_fields', 'created', 'last_updated', '_occupied',
+            'connected_endpoints', 'connected_endpoints_type', 'connected_endpoints_reachable', 'tags', 'custom_fields',
+            'created', 'last_updated', '_occupied',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description', 'cable', '_occupied')
 
 
-class PowerOutletSerializer(
-    OwnerMixin,
-    NetBoxModelSerializer,
-    CabledObjectSerializer,
-    ConnectedEndpointsSerializer
-):
+class PowerOutletSerializer(NetBoxModelSerializer, CabledObjectSerializer, ConnectedEndpointsSerializer):
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -180,17 +159,12 @@ class PowerOutletSerializer(
             'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'status', 'color',
             'power_port', 'feed_leg', 'description', 'mark_connected', 'cable', 'cable_end', 'link_peers',
             'link_peers_type', 'connected_endpoints', 'connected_endpoints_type', 'connected_endpoints_reachable',
-            'owner', 'tags', 'custom_fields', 'created', 'last_updated', '_occupied',
+            'tags', 'custom_fields', 'created', 'last_updated', '_occupied',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description', 'cable', '_occupied')
 
 
-class InterfaceSerializer(
-    OwnerMixin,
-    NetBoxModelSerializer,
-    CabledObjectSerializer,
-    ConnectedEndpointsSerializer
-):
+class InterfaceSerializer(NetBoxModelSerializer, CabledObjectSerializer, ConnectedEndpointsSerializer):
     device = DeviceSerializer(nested=True)
     vdcs = SerializedPKRelatedField(
         queryset=VirtualDeviceContext.objects.all(),
@@ -208,7 +182,6 @@ class InterfaceSerializer(
     type = ChoiceField(choices=InterfaceTypeChoices)
     parent = NestedInterfaceSerializer(required=False, allow_null=True)
     bridge = NestedInterfaceSerializer(required=False, allow_null=True)
-    bridge_interfaces = NestedInterfaceSerializer(many=True, read_only=True)
     lag = NestedInterfaceSerializer(required=False, allow_null=True)
     mode = ChoiceField(choices=InterfaceModeChoices, required=False, allow_blank=True)
     duplex = ChoiceField(choices=InterfaceDuplexChoices, required=False, allow_blank=True, allow_null=True)
@@ -248,13 +221,13 @@ class InterfaceSerializer(
         model = Interface
         fields = [
             'id', 'url', 'display_url', 'display', 'device', 'vdcs', 'module', 'name', 'label', 'type', 'enabled',
-            'parent', 'bridge', 'bridge_interfaces', 'lag', 'mtu', 'mac_address', 'primary_mac_address',
-            'mac_addresses', 'speed', 'duplex', 'wwn', 'mgmt_only', 'description', 'mode', 'rf_role', 'rf_channel',
-            'poe_mode', 'poe_type', 'rf_channel_frequency', 'rf_channel_width', 'tx_power', 'untagged_vlan',
-            'tagged_vlans', 'qinq_svlan', 'vlan_translation_policy', 'mark_connected', 'cable', 'cable_end',
-            'wireless_link', 'link_peers', 'link_peers_type', 'wireless_lans', 'vrf', 'l2vpn_termination',
-            'connected_endpoints', 'connected_endpoints_type', 'connected_endpoints_reachable', 'owner', 'tags',
-            'custom_fields', 'created', 'last_updated', 'count_ipaddresses', 'count_fhrp_groups', '_occupied',
+            'parent', 'bridge', 'lag', 'mtu', 'mac_address', 'primary_mac_address', 'mac_addresses', 'speed', 'duplex',
+            'wwn', 'mgmt_only', 'description', 'mode', 'rf_role', 'rf_channel', 'poe_mode', 'poe_type',
+            'rf_channel_frequency', 'rf_channel_width', 'tx_power', 'untagged_vlan', 'tagged_vlans', 'qinq_svlan',
+            'vlan_translation_policy', 'mark_connected', 'cable', 'cable_end', 'wireless_link', 'link_peers',
+            'link_peers_type', 'wireless_lans', 'vrf', 'l2vpn_termination', 'connected_endpoints',
+            'connected_endpoints_type', 'connected_endpoints_reachable', 'tags', 'custom_fields', 'created',
+            'last_updated', 'count_ipaddresses', 'count_fhrp_groups', '_occupied',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description', 'cable', '_occupied')
 
@@ -321,20 +294,7 @@ class InterfaceSerializer(
         return super().validate(data)
 
 
-class RearPortMappingSerializer(serializers.ModelSerializer):
-    position = serializers.IntegerField(
-        source='rear_port_position'
-    )
-    front_port = serializers.PrimaryKeyRelatedField(
-        queryset=FrontPort.objects.all(),
-    )
-
-    class Meta:
-        model = PortMapping
-        fields = ('position', 'front_port', 'front_port_position')
-
-
-class RearPortSerializer(OwnerMixin, NetBoxModelSerializer, CabledObjectSerializer, PortSerializer):
+class RearPortSerializer(NetBoxModelSerializer, CabledObjectSerializer):
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -343,36 +303,28 @@ class RearPortSerializer(OwnerMixin, NetBoxModelSerializer, CabledObjectSerializ
         allow_null=True
     )
     type = ChoiceField(choices=PortTypeChoices)
-    front_ports = RearPortMappingSerializer(
-        source='mappings',
-        many=True,
-        required=False,
-    )
 
     class Meta:
         model = RearPort
         fields = [
             'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'color', 'positions',
-            'front_ports', 'description', 'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type',
-            'owner', 'tags', 'custom_fields', 'created', 'last_updated', '_occupied',
+            'description', 'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type', 'tags',
+            'custom_fields', 'created', 'last_updated', '_occupied',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description', 'cable', '_occupied')
 
 
-class FrontPortMappingSerializer(serializers.ModelSerializer):
-    position = serializers.IntegerField(
-        source='front_port_position'
-    )
-    rear_port = serializers.PrimaryKeyRelatedField(
-        queryset=RearPort.objects.all(),
-    )
+class FrontPortRearPortSerializer(WritableNestedSerializer):
+    """
+    NestedRearPortSerializer but with parent device omitted (since front and rear ports must belong to same device)
+    """
 
     class Meta:
-        model = PortMapping
-        fields = ('position', 'rear_port', 'rear_port_position')
+        model = RearPort
+        fields = ['id', 'url', 'display_url', 'display', 'name', 'label', 'description']
 
 
-class FrontPortSerializer(OwnerMixin, NetBoxModelSerializer, CabledObjectSerializer, PortSerializer):
+class FrontPortSerializer(NetBoxModelSerializer, CabledObjectSerializer):
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -381,23 +333,19 @@ class FrontPortSerializer(OwnerMixin, NetBoxModelSerializer, CabledObjectSeriali
         allow_null=True
     )
     type = ChoiceField(choices=PortTypeChoices)
-    rear_ports = FrontPortMappingSerializer(
-        source='mappings',
-        many=True,
-        required=False,
-    )
+    rear_port = FrontPortRearPortSerializer()
 
     class Meta:
         model = FrontPort
         fields = [
-            'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'color', 'positions',
-            'rear_ports', 'description', 'mark_connected', 'cable', 'cable_end', 'link_peers', 'link_peers_type',
-            'owner', 'tags', 'custom_fields', 'created', 'last_updated', '_occupied',
+            'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'label', 'type', 'color', 'rear_port',
+            'rear_port_position', 'description', 'mark_connected', 'cable', 'cable_end', 'link_peers',
+            'link_peers_type', 'tags', 'custom_fields', 'created', 'last_updated', '_occupied',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description', 'cable', '_occupied')
 
 
-class ModuleBaySerializer(OwnerMixin, NetBoxModelSerializer):
+class ModuleBaySerializer(NetBoxModelSerializer):
     device = DeviceSerializer(nested=True)
     module = ModuleSerializer(
         nested=True,
@@ -417,12 +365,12 @@ class ModuleBaySerializer(OwnerMixin, NetBoxModelSerializer):
         model = ModuleBay
         fields = [
             'id', 'url', 'display_url', 'display', 'device', 'module', 'name', 'installed_module', 'label', 'position',
-            'description', 'owner', 'tags', 'custom_fields', 'created', 'last_updated',
+            'description', 'tags', 'custom_fields', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'installed_module', 'name', 'description')
 
 
-class DeviceBaySerializer(OwnerMixin, NetBoxModelSerializer):
+class DeviceBaySerializer(NetBoxModelSerializer):
     device = DeviceSerializer(nested=True)
     installed_device = DeviceSerializer(nested=True, required=False, allow_null=True)
 
@@ -430,12 +378,12 @@ class DeviceBaySerializer(OwnerMixin, NetBoxModelSerializer):
         model = DeviceBay
         fields = [
             'id', 'url', 'display_url', 'display', 'device', 'name', 'label', 'description', 'installed_device',
-            'owner', 'tags', 'custom_fields', 'created', 'last_updated',
+            'tags', 'custom_fields', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description')
 
 
-class InventoryItemSerializer(OwnerMixin, NetBoxModelSerializer):
+class InventoryItemSerializer(NetBoxModelSerializer):
     device = DeviceSerializer(nested=True)
     parent = serializers.PrimaryKeyRelatedField(queryset=InventoryItem.objects.all(), allow_null=True, default=None)
     role = InventoryItemRoleSerializer(nested=True, required=False, allow_null=True)
@@ -454,6 +402,6 @@ class InventoryItemSerializer(OwnerMixin, NetBoxModelSerializer):
         fields = [
             'id', 'url', 'display_url', 'display', 'device', 'parent', 'name', 'label', 'status', 'role',
             'manufacturer', 'part_id', 'serial', 'asset_tag', 'discovered', 'description', 'component_type',
-            'component_id', 'component', 'owner', 'tags', 'custom_fields', 'created', 'last_updated', '_depth',
+            'component_id', 'component', 'tags', 'custom_fields', 'created', 'last_updated', '_depth',
         ]
         brief_fields = ('id', 'url', 'display', 'device', 'name', 'description', '_depth')

netbox/dcim/api/serializers_/devices.py

@@ -111,7 +111,7 @@ class DeviceWithConfigContextSerializer(DeviceSerializer):
             'id', 'url', 'display_url', 'display', 'name', 'device_type', 'role', 'tenant', 'platform', 'serial',
             'asset_tag', 'site', 'location', 'rack', 'position', 'face', 'latitude', 'longitude', 'parent_device',
             'status', 'airflow', 'primary_ip', 'primary_ip4', 'primary_ip6', 'oob_ip', 'cluster', 'virtual_chassis',
-            'vc_position', 'vc_priority', 'description', 'owner', 'comments', 'config_template', 'config_context',
+            'vc_position', 'vc_priority', 'description', 'comments', 'config_template', 'config_context',
             'local_context_data', 'tags', 'custom_fields', 'created', 'last_updated', 'console_port_count',
             'console_server_port_count', 'power_port_count', 'power_outlet_count', 'interface_count',
             'front_port_count', 'rear_port_count', 'device_bay_count', 'module_bay_count', 'inventory_item_count',

netbox/dcim/api/serializers_/devicetype_components.py

@@ -5,14 +5,12 @@ from dcim.choices import *
 from dcim.constants import *
 from dcim.models import (
     ConsolePortTemplate, ConsoleServerPortTemplate, DeviceBayTemplate, FrontPortTemplate, InterfaceTemplate,
-    InventoryItemTemplate, ModuleBayTemplate, PortTemplateMapping, PowerOutletTemplate, PowerPortTemplate,
-    RearPortTemplate,
+    InventoryItemTemplate, ModuleBayTemplate, PowerOutletTemplate, PowerPortTemplate, RearPortTemplate,
 )
 from netbox.api.fields import ChoiceField, ContentTypeField
 from netbox.api.gfk_fields import GFKSerializerField
 from netbox.api.serializers import ChangeLogMessageSerializer, ValidatedModelSerializer
 from wireless.choices import *
-from .base import PortSerializer
 from .devicetypes import DeviceTypeSerializer, ModuleTypeSerializer
 from .manufacturers import ManufacturerSerializer
 from .nested import NestedInterfaceTemplateSerializer
@@ -207,20 +205,7 @@ class InterfaceTemplateSerializer(ComponentTemplateSerializer):
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
-class RearPortTemplateMappingSerializer(serializers.ModelSerializer):
-    position = serializers.IntegerField(
-        source='rear_port_position'
-    )
-    front_port = serializers.PrimaryKeyRelatedField(
-        queryset=FrontPortTemplate.objects.all(),
-    )
-
-    class Meta:
-        model = PortTemplateMapping
-        fields = ('position', 'front_port', 'front_port_position')
-
-
-class RearPortTemplateSerializer(ComponentTemplateSerializer, PortSerializer):
+class RearPortTemplateSerializer(ComponentTemplateSerializer):
     device_type = DeviceTypeSerializer(
         required=False,
         nested=True,
@@ -234,35 +219,17 @@ class RearPortTemplateSerializer(ComponentTemplateSerializer, PortSerializer):
         default=None
     )
     type = ChoiceField(choices=PortTypeChoices)
-    front_ports = RearPortTemplateMappingSerializer(
-        source='mappings',
-        many=True,
-        required=False,
-    )
 
     class Meta:
         model = RearPortTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'color', 'positions',
-            'front_ports', 'description', 'created', 'last_updated',
+            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'color',
+            'positions', 'description', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 
 
-class FrontPortTemplateMappingSerializer(serializers.ModelSerializer):
-    position = serializers.IntegerField(
-        source='front_port_position'
-    )
-    rear_port = serializers.PrimaryKeyRelatedField(
-        queryset=RearPortTemplate.objects.all(),
-    )
-
-    class Meta:
-        model = PortTemplateMapping
-        fields = ('position', 'rear_port', 'rear_port_position')
-
-
-class FrontPortTemplateSerializer(ComponentTemplateSerializer, PortSerializer):
+class FrontPortTemplateSerializer(ComponentTemplateSerializer):
     device_type = DeviceTypeSerializer(
         nested=True,
         required=False,
@@ -276,17 +243,13 @@ class FrontPortTemplateSerializer(ComponentTemplateSerializer, PortSerializer):
         default=None
     )
     type = ChoiceField(choices=PortTypeChoices)
-    rear_ports = FrontPortTemplateMappingSerializer(
-        source='mappings',
-        many=True,
-        required=False,
-    )
+    rear_port = RearPortTemplateSerializer(nested=True)
 
     class Meta:
         model = FrontPortTemplate
         fields = [
-            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'color', 'positions',
-            'rear_ports', 'description', 'created', 'last_updated',
+            'id', 'url', 'display', 'device_type', 'module_type', 'name', 'label', 'type', 'color',
+            'rear_port', 'rear_port_position', 'description', 'created', 'last_updated',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'description')
 

netbox/dcim/api/serializers_/manufacturers.py

@@ -11,15 +11,13 @@ class ManufacturerSerializer(OrganizationalModelSerializer):
 
     # Related object counts
     devicetype_count = RelatedObjectCountField('device_types')
-    moduletype_count = RelatedObjectCountField('module_types')
     inventoryitem_count = RelatedObjectCountField('inventory_items')
     platform_count = RelatedObjectCountField('platforms')
 
     class Meta:
         model = Manufacturer
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'slug', 'description', 'owner', 'comments', 'tags',
-            'custom_fields', 'created', 'last_updated', 'devicetype_count', 'moduletype_count', 'inventoryitem_count',
-            'platform_count',
+            'id', 'url', 'display_url', 'display', 'name', 'slug', 'description', 'owner', 'tags', 'custom_fields',
+            'created', 'last_updated', 'devicetype_count', 'inventoryitem_count', 'platform_count',
         ]
-        brief_fields = ('id', 'url', 'display', 'name', 'slug', 'description')
+        brief_fields = ('id', 'url', 'display', 'name', 'slug', 'description', 'devicetype_count')

netbox/dcim/api/serializers_/racks.py

@@ -30,7 +30,7 @@ class RackRoleSerializer(OrganizationalModelSerializer):
     class Meta:
         model = RackRole
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'slug', 'color', 'description', 'owner', 'comments', 'tags',
+            'id', 'url', 'display_url', 'display', 'name', 'slug', 'color', 'description', 'owner', 'tags',
             'custom_fields', 'created', 'last_updated', 'rack_count',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'slug', 'description', 'rack_count')

netbox/dcim/api/serializers_/roles.py

@@ -38,7 +38,7 @@ class InventoryItemRoleSerializer(OrganizationalModelSerializer):
     class Meta:
         model = InventoryItemRole
         fields = [
-            'id', 'url', 'display_url', 'display', 'name', 'slug', 'color', 'description', 'owner', 'comments', 'tags',
+            'id', 'url', 'display_url', 'display', 'name', 'slug', 'color', 'description', 'owner', 'tags',
             'custom_fields', 'created', 'last_updated', 'inventoryitem_count',
         ]
         brief_fields = ('id', 'url', 'display', 'name', 'slug', 'description', 'inventoryitem_count')

netbox/dcim/cable_profiles.py

@@ -1,390 +1,108 @@
 from django.core.exceptions import ValidationError
 from django.utils.translation import gettext_lazy as _
 
-from dcim.choices import CableEndChoices
 from dcim.models import CableTermination
 
 
 class BaseCableProfile:
-    """Base class for representing a cable profile."""
-
-    # Mappings of connectors to the number of positions presented by each, at either end of the cable. For example, a
-    # 12-strand MPO fiber cable would have one connector at either end with six positions (six bidirectional fiber
-    # pairs).
-    a_connectors = {}
-    b_connectors = {}
-
-    # Defined a mapping of A/B connector & position pairings. If not defined, all positions are presumed to be
-    # symmetrical (i.e. 1:1 on side A maps to 1:1 on side B). If defined, it must be constructed as a dictionary of
-    # two-item tuples, e.g. {(1, 1): (1, 1)}.
-    _mapping = None
+    # Maximum number of terminations allowed per side
+    a_max_connections = None
+    b_max_connections = None
 
     def clean(self, cable):
-        # Enforce maximum terminations limits
-        a_terminations_count = len(cable.a_terminations)
-        b_terminations_count = len(cable.b_terminations)
-        max_a_terminations = len(self.a_connectors)
-        max_b_terminations = len(self.b_connectors)
-        if a_terminations_count > max_a_terminations:
+        # Enforce maximum connection limits
+        if self.a_max_connections and len(cable.a_terminations) > self.a_max_connections:
             raise ValidationError({
                 'a_terminations': _(
-                    'A side of cable has {count} terminations but only {max} are permitted for profile {profile}'
+                    'Maximum A side connections for profile {profile}: {max}'
                 ).format(
-                    count=a_terminations_count,
                     profile=cable.get_profile_display(),
-                    max=max_a_terminations,
+                    max=self.a_max_connections,
                 )
             })
-        if b_terminations_count > max_b_terminations:
+        if self.b_max_connections and len(cable.b_terminations) > self.b_max_connections:
             raise ValidationError({
                 'b_terminations': _(
-                    'B side of cable has {count} terminations but only {max} are permitted for profile {profile}'
+                    'Maximum B side connections for profile {profile}: {max}'
                 ).format(
-                    count=b_terminations_count,
                     profile=cable.get_profile_display(),
-                    max=max_b_terminations,
+                    max=self.b_max_connections,
                 )
             })
 
-    def get_mapped_position(self, side, connector, position):
+    def get_mapped_position(self, side, position):
         """
-        Return the mapped far-end connector & position for a given cable end the local connector & position.
+        Return the mapped position for a given cable end and position.
+
+        By default, assume all positions are symmetrical.
         """
-        # By default, assume all positions are symmetrical.
-        if self._mapping:
-            return self._mapping.get((connector, position))
-        return connector, position
+        return position
 
-    def get_peer_termination(self, termination, position):
-        """
-        Given a terminating object, return the peer terminating object (if any) on the opposite end of the cable.
-        """
-        try:
-            connector, position = self.get_mapped_position(
-                termination.cable_end,
-                termination.cable_connector,
-                position
-            )
-        except TypeError:
-            raise ValueError(
-                f"Could not map connector {termination.cable_connector} position {position} on side "
-                f"{termination.cable_end}"
-            )
-        try:
-            ct = CableTermination.objects.get(
-                cable=termination.cable,
-                cable_end=termination.opposite_cable_end,
-                connector=connector,
-                positions__contains=[position],
-            )
-            return ct.termination, position
-        except CableTermination.DoesNotExist:
-            return None, None
+    def get_peer_terminations(self, terminations, position_stack):
+        local_end = terminations[0].cable_end
+        qs = CableTermination.objects.filter(
+            cable=terminations[0].cable,
+            cable_end=terminations[0].opposite_cable_end
+        )
 
-    @staticmethod
-    def get_position_list(n):
-        """Return a list of integers from 1 to n, inclusive."""
-        return list(range(1, n + 1))
+        # TODO: Optimize this to use a single query under any condition
+        if position_stack:
+            # Attempt to find a peer termination at the same position currently in the stack. Pop the stack only if
+            # we find one. Otherwise, return any peer terminations with a null position.
+            position = self.get_mapped_position(local_end, position_stack[-1][0])
+            if peers := qs.filter(position=position):
+                position_stack.pop()
+                return peers
+
+        return qs.filter(position=None)
 
 
-# Profile naming:
-#  - Single: One connector per side, with one or more positions
-#  - Trunk: Two or more connectors per side, with one or more positions per connector
-#  - Breakout: One or more connectors on the A side which map to a greater number of B side connectors
-#  - Shuffle: A cable with nonlinear position mappings between sides
+class StraightSingleCableProfile(BaseCableProfile):
+    a_max_connections = 1
+    b_max_connections = 1
 
-class Single1C1PCableProfile(BaseCableProfile):
-    a_connectors = {
+
+class StraightMultiCableProfile(BaseCableProfile):
+    a_max_connections = None
+    b_max_connections = None
+
+
+class Shuffle2x2MPO8CableProfile(BaseCableProfile):
+    a_max_connections = 8
+    b_max_connections = 8
+    _mapping = {
         1: 1,
-    }
-    b_connectors = a_connectors
-
-
-class Single1C2PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 2,
-    }
-    b_connectors = a_connectors
-
-
-class Single1C4PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 4,
-    }
-    b_connectors = a_connectors
-
-
-class Single1C6PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 6,
-    }
-    b_connectors = a_connectors
-
-
-class Single1C8PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 8,
-    }
-    b_connectors = a_connectors
-
-
-class Single1C12PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 12,
-    }
-    b_connectors = a_connectors
-
-
-class Single1C16PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 16,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk2C1PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 1,
-        2: 1,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk2C2PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 2,
         2: 2,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk2C4PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 4,
-        2: 4,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk2C6PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 6,
-        2: 6,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk2C8PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 8,
-        2: 8,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk2C12PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 12,
-        2: 12,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk4C1PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 1,
-        2: 1,
-        3: 1,
-        4: 1,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk4C2PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 2,
-        2: 2,
-        3: 2,
-        4: 2,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk4C4PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 4,
-        2: 4,
-        3: 4,
-        4: 4,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk4C6PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 6,
-        2: 6,
-        3: 6,
+        3: 5,
         4: 6,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk4C8PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 8,
-        2: 8,
-        3: 8,
-        4: 8,
-    }
-    b_connectors = a_connectors
-
-
-class Trunk8C4PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 4,
-        2: 4,
-        3: 4,
-        4: 4,
-        5: 4,
+        5: 3,
         6: 4,
-        7: 4,
-        8: 4,
-    }
-    b_connectors = a_connectors
-
-
-class Breakout1C4Px4C1PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 4,
-    }
-    b_connectors = {
-        1: 1,
-        2: 1,
-        3: 1,
-        4: 1,
-    }
-    _mapping = {
-        (1, 1): (1, 1),
-        (1, 2): (2, 1),
-        (1, 3): (3, 1),
-        (1, 4): (4, 1),
-        (2, 1): (1, 2),
-        (3, 1): (1, 3),
-        (4, 1): (1, 4),
+        7: 7,
+        8: 8,
     }
 
-
-class Breakout1C6Px6C1PCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 6,
-    }
-    b_connectors = {
-        1: 1,
-        2: 1,
-        3: 1,
-        4: 1,
-        5: 1,
-        6: 1,
-    }
-    _mapping = {
-        (1, 1): (1, 1),
-        (1, 2): (2, 1),
-        (1, 3): (3, 1),
-        (1, 4): (4, 1),
-        (1, 5): (5, 1),
-        (1, 6): (6, 1),
-        (2, 1): (1, 2),
-        (3, 1): (1, 3),
-        (4, 1): (1, 4),
-        (5, 1): (1, 5),
-        (6, 1): (1, 6),
-    }
+    def get_mapped_position(self, side, position):
+        return self._mapping.get(position)
 
 
-class Trunk2C4PShuffleCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 4,
-        2: 4,
-    }
-    b_connectors = a_connectors
-    _mapping = {
-        (1, 1): (1, 1),
-        (1, 2): (1, 2),
-        (1, 3): (2, 1),
-        (1, 4): (2, 2),
-        (2, 1): (1, 3),
-        (2, 2): (1, 4),
-        (2, 3): (2, 3),
-        (2, 4): (2, 4),
-    }
-
-
-class Trunk4C4PShuffleCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 4,
-        2: 4,
-        3: 4,
-        4: 4,
-    }
-    b_connectors = a_connectors
-    _mapping = {
-        (1, 1): (1, 1),
-        (1, 2): (2, 1),
-        (1, 3): (3, 1),
-        (1, 4): (4, 1),
-        (2, 1): (1, 2),
-        (2, 2): (2, 2),
-        (2, 3): (3, 2),
-        (2, 4): (4, 2),
-        (3, 1): (1, 3),
-        (3, 2): (2, 3),
-        (3, 3): (3, 3),
-        (3, 4): (4, 3),
-        (4, 1): (1, 4),
-        (4, 2): (2, 4),
-        (4, 3): (3, 4),
-        (4, 4): (4, 4),
-    }
-
-
-class Breakout2C4Px8C1PShuffleCableProfile(BaseCableProfile):
-    a_connectors = {
-        1: 4,
-        2: 4,
-    }
-    b_connectors = {
-        1: 1,
-        2: 1,
-        3: 1,
-        4: 1,
-        5: 1,
-        6: 1,
-        7: 1,
-        8: 1,
-    }
+class Shuffle4x4MPO8CableProfile(BaseCableProfile):
+    a_max_connections = 8
+    b_max_connections = 8
+    # A side to B side position mapping
     _a_mapping = {
-        (1, 1): (1, 1),
-        (1, 2): (2, 1),
-        (1, 3): (5, 1),
-        (1, 4): (6, 1),
-        (2, 1): (3, 1),
-        (2, 2): (4, 1),
-        (2, 3): (7, 1),
-        (2, 4): (8, 1),
-    }
-    _b_mapping = {
-        (1, 1): (1, 1),
-        (2, 1): (1, 2),
-        (3, 1): (2, 1),
-        (4, 1): (2, 2),
-        (5, 1): (1, 3),
-        (6, 1): (1, 4),
-        (7, 1): (2, 3),
-        (8, 1): (2, 4),
+        1: 1,
+        2: 3,
+        3: 5,
+        4: 7,
+        5: 2,
+        6: 4,
+        7: 6,
+        8: 8,
     }
+    # B side to A side position mapping (reverse of _a_mapping)
+    _b_mapping = {v: k for k, v in _a_mapping.items()}
 
-    def get_mapped_position(self, side, connector, position):
-        if side.upper() == CableEndChoices.SIDE_A:
-            return self._a_mapping.get((connector, position))
-        return self._b_mapping.get((connector, position))
+    def get_mapped_position(self, side, position):
+        if side.lower() == 'b':
+            return self._b_mapping.get(position)
+        return self._a_mapping.get(position)

netbox/dcim/choices.py

@@ -1722,74 +1722,16 @@ class PortTypeChoices(ChoiceSet):
 #
 
 class CableProfileChoices(ChoiceSet):
-    # Singles
-    SINGLE_1C1P = 'single-1c1p'
-    SINGLE_1C2P = 'single-1c2p'
-    SINGLE_1C4P = 'single-1c4p'
-    SINGLE_1C6P = 'single-1c6p'
-    SINGLE_1C8P = 'single-1c8p'
-    SINGLE_1C12P = 'single-1c12p'
-    SINGLE_1C16P = 'single-1c16p'
-    # Trunks
-    TRUNK_2C1P = 'trunk-2c1p'
-    TRUNK_2C2P = 'trunk-2c2p'
-    TRUNK_2C4P = 'trunk-2c4p'
-    TRUNK_2C4P_SHUFFLE = 'trunk-2c4p-shuffle'
-    TRUNK_2C6P = 'trunk-2c6p'
-    TRUNK_2C8P = 'trunk-2c8p'
-    TRUNK_2C12P = 'trunk-2c12p'
-    TRUNK_4C1P = 'trunk-4c1p'
-    TRUNK_4C2P = 'trunk-4c2p'
-    TRUNK_4C4P = 'trunk-4c4p'
-    TRUNK_4C4P_SHUFFLE = 'trunk-4c4p-shuffle'
-    TRUNK_4C6P = 'trunk-4c6p'
-    TRUNK_4C8P = 'trunk-4c8p'
-    TRUNK_8C4P = 'trunk-8c4p'
-    # Breakouts
-    BREAKOUT_1C4P_4C1P = 'breakout-1c4p-4c1p'
-    BREAKOUT_1C6P_6C1P = 'breakout-1c6p-6c1p'
-    BREAKOUT_2C4P_8C1P_SHUFFLE = 'breakout-2c4p-8c1p-shuffle'
+    STRAIGHT_SINGLE = 'straight-single'
+    STRAIGHT_MULTI = 'straight-multi'
+    SHUFFLE_2X2_MPO8 = 'shuffle-2x2-mpo8'
+    SHUFFLE_4X4_MPO8 = 'shuffle-4x4-mpo8'
 
     CHOICES = (
-        (
-            _('Single'),
-            (
-                (SINGLE_1C1P, _('1C1P')),
-                (SINGLE_1C2P, _('1C2P')),
-                (SINGLE_1C4P, _('1C4P')),
-                (SINGLE_1C6P, _('1C6P')),
-                (SINGLE_1C8P, _('1C8P')),
-                (SINGLE_1C12P, _('1C12P')),
-                (SINGLE_1C16P, _('1C16P')),
-            ),
-        ),
-        (
-            _('Trunk'),
-            (
-                (TRUNK_2C1P, _('2C1P trunk')),
-                (TRUNK_2C2P, _('2C2P trunk')),
-                (TRUNK_2C4P, _('2C4P trunk')),
-                (TRUNK_2C4P_SHUFFLE, _('2C4P trunk (shuffle)')),
-                (TRUNK_2C6P, _('2C6P trunk')),
-                (TRUNK_2C8P, _('2C8P trunk')),
-                (TRUNK_2C12P, _('2C12P trunk')),
-                (TRUNK_4C1P, _('4C1P trunk')),
-                (TRUNK_4C2P, _('4C2P trunk')),
-                (TRUNK_4C4P, _('4C4P trunk')),
-                (TRUNK_4C4P_SHUFFLE, _('4C4P trunk (shuffle)')),
-                (TRUNK_4C6P, _('4C6P trunk')),
-                (TRUNK_4C8P, _('4C8P trunk')),
-                (TRUNK_8C4P, _('8C4P trunk')),
-            ),
-        ),
-        (
-            _('Breakout'),
-            (
-                (BREAKOUT_1C4P_4C1P, _('1C4P:4C1P breakout')),
-                (BREAKOUT_1C6P_6C1P, _('1C6P:6C1P breakout')),
-                (BREAKOUT_2C4P_8C1P_SHUFFLE, _('2C4P:8C1P breakout (shuffle)')),
-            ),
-        ),
+        (STRAIGHT_SINGLE, _('Straight (single position)')),
+        (STRAIGHT_MULTI, _('Straight (multi-position)')),
+        (SHUFFLE_2X2_MPO8, _('Shuffle (2x2 MPO8)')),
+        (SHUFFLE_4X4_MPO8, _('Shuffle (4x4 MPO8)')),
     )
 
 

netbox/dcim/constants.py

@@ -24,9 +24,6 @@ RACK_STARTING_UNIT_DEFAULT = 1
 # Cables
 #
 
-CABLE_CONNECTOR_MIN = 1
-CABLE_CONNECTOR_MAX = 256
-
 CABLE_POSITION_MIN = 1
 CABLE_POSITION_MAX = 1024
 
@@ -35,8 +32,8 @@ CABLE_POSITION_MAX = 1024
 # RearPorts
 #
 
-PORT_POSITION_MIN = 1
-PORT_POSITION_MAX = 1024
+REARPORT_POSITIONS_MIN = 1
+REARPORT_POSITIONS_MAX = 1024
 
 
 #

netbox/dcim/filtersets.py

@@ -14,6 +14,7 @@ from netbox.filtersets import (
     AttributeFiltersMixin, BaseFilterSet, ChangeLoggedModelFilterSet, NestedGroupModelFilterSet,
     OrganizationalModelFilterSet, PrimaryModelFilterSet, NetBoxModelFilterSet,
 )
+from netbox.plugins.registration import register_filterset
 from tenancy.filtersets import ContactModelFilterSet, TenancyFilterSet
 from tenancy.models import *
 from users.filterset_mixins import OwnerFilterMixin
@@ -22,7 +23,6 @@ from utilities.filters import (
     ContentTypeFilter, MultiValueCharFilter, MultiValueMACAddressFilter, MultiValueNumberFilter, MultiValueWWNFilter,
     NumericArrayFilter, TreeNodeMultipleChoiceFilter,
 )
-from utilities.filtersets import register_filterset
 from virtualization.models import Cluster, ClusterGroup, VirtualMachine, VMInterface
 from vpn.models import L2VPN
 from wireless.choices import WirelessChannelChoices, WirelessRoleChoices
@@ -904,15 +904,12 @@ class FrontPortTemplateFilterSet(ChangeLoggedModelFilterSet, ModularDeviceTypeCo
         null_value=None
     )
     rear_port_id = django_filters.ModelMultipleChoiceFilter(
-        field_name='mappings__rear_port',
-        queryset=RearPortTemplate.objects.all(),
-        to_field_name='rear_port',
-        label=_('Rear port (ID)'),
+        queryset=RearPort.objects.all()
     )
 
     class Meta:
         model = FrontPortTemplate
-        fields = ('id', 'name', 'label', 'type', 'color', 'positions', 'description')
+        fields = ('id', 'name', 'label', 'type', 'color', 'rear_port_position', 'description')
 
 
 @register_filterset
@@ -921,12 +918,6 @@ class RearPortTemplateFilterSet(ChangeLoggedModelFilterSet, ModularDeviceTypeCom
         choices=PortTypeChoices,
         null_value=None
     )
-    front_port_id = django_filters.ModelMultipleChoiceFilter(
-        field_name='mappings__front_port',
-        queryset=FrontPort.objects.all(),
-        to_field_name='front_port',
-        label=_('Front port (ID)'),
-    )
 
     class Meta:
         model = RearPortTemplate
@@ -1673,17 +1664,6 @@ class DeviceComponentFilterSet(OwnerFilterMixin, NetBoxModelFilterSet):
         choices=DeviceStatusChoices,
         field_name='device__status',
     )
-    tenant_id = django_filters.ModelMultipleChoiceFilter(
-        field_name='device__tenant',
-        queryset=Tenant.objects.all(),
-        label=_('Tenant (ID)'),
-    )
-    tenant = django_filters.ModelMultipleChoiceFilter(
-        field_name='device__tenant__slug',
-        queryset=Tenant.objects.all(),
-        to_field_name='slug',
-        label=_('Tenant (slug)'),
-    )
 
     def search(self, queryset, name, value):
         if not value.strip():
@@ -1748,9 +1728,7 @@ class ConsolePortFilterSet(ModularDeviceComponentFilterSet, CabledObjectFilterSe
 
     class Meta:
         model = ConsolePort
-        fields = (
-            'id', 'name', 'label', 'speed', 'description', 'mark_connected', 'cable_end', 'cable_connector',
-        )
+        fields = ('id', 'name', 'label', 'speed', 'description', 'mark_connected', 'cable_end', 'cable_position')
 
 
 @register_filterset
@@ -1762,9 +1740,7 @@ class ConsoleServerPortFilterSet(ModularDeviceComponentFilterSet, CabledObjectFi
 
     class Meta:
         model = ConsoleServerPort
-        fields = (
-            'id', 'name', 'label', 'speed', 'description', 'mark_connected', 'cable_end', 'cable_connector',
-        )
+        fields = ('id', 'name', 'label', 'speed', 'description', 'mark_connected', 'cable_end', 'cable_position')
 
 
 @register_filterset
@@ -1778,7 +1754,7 @@ class PowerPortFilterSet(ModularDeviceComponentFilterSet, CabledObjectFilterSet,
         model = PowerPort
         fields = (
             'id', 'name', 'label', 'maximum_draw', 'allocated_draw', 'description', 'mark_connected', 'cable_end',
-            'cable_connector',
+            'cable_position',
         )
 
 
@@ -1805,7 +1781,7 @@ class PowerOutletFilterSet(ModularDeviceComponentFilterSet, CabledObjectFilterSe
         model = PowerOutlet
         fields = (
             'id', 'name', 'status', 'label', 'feed_leg', 'description', 'color', 'mark_connected', 'cable_end',
-            'cable_connector',
+            'cable_position',
         )
 
 
@@ -2115,7 +2091,7 @@ class InterfaceFilterSet(
         fields = (
             'id', 'name', 'label', 'type', 'enabled', 'mtu', 'mgmt_only', 'poe_mode', 'poe_type', 'mode', 'rf_role',
             'rf_channel', 'rf_channel_frequency', 'rf_channel_width', 'tx_power', 'description', 'mark_connected',
-            'cable_id', 'cable_end', 'cable_connector',
+            'cable_id', 'cable_end', 'cable_position',
         )
 
     def filter_virtual_chassis_member_or_master(self, queryset, name, value):
@@ -2161,17 +2137,14 @@ class FrontPortFilterSet(ModularDeviceComponentFilterSet, CabledObjectFilterSet)
         null_value=None
     )
     rear_port_id = django_filters.ModelMultipleChoiceFilter(
-        field_name='mappings__rear_port',
-        queryset=RearPort.objects.all(),
-        to_field_name='rear_port',
-        label=_('Rear port (ID)'),
+        queryset=RearPort.objects.all()
     )
 
     class Meta:
         model = FrontPort
         fields = (
-            'id', 'name', 'label', 'type', 'color', 'positions', 'description', 'mark_connected', 'cable_end',
-            'cable_connector',
+            'id', 'name', 'label', 'type', 'color', 'rear_port_position', 'description', 'mark_connected', 'cable_end',
+            'cable_position',
         )
 
 
@@ -2181,18 +2154,12 @@ class RearPortFilterSet(ModularDeviceComponentFilterSet, CabledObjectFilterSet):
         choices=PortTypeChoices,
         null_value=None
     )
-    front_port_id = django_filters.ModelMultipleChoiceFilter(
-        field_name='mappings__front_port',
-        queryset=FrontPort.objects.all(),
-        to_field_name='front_port',
-        label=_('Front port (ID)'),
-    )
 
     class Meta:
         model = RearPort
         fields = (
             'id', 'name', 'label', 'type', 'color', 'positions', 'description', 'mark_connected', 'cable_end',
-            'cable_connector',
+            'cable_position',
         )
 
 
@@ -2548,7 +2515,7 @@ class CableTerminationFilterSet(ChangeLoggedModelFilterSet):
 
     class Meta:
         model = CableTermination
-        fields = ('id', 'cable', 'cable_end', 'termination_type', 'termination_id')
+        fields = ('id', 'cable', 'cable_end', 'position', 'termination_type', 'termination_id')
 
 
 @register_filterset
@@ -2667,7 +2634,7 @@ class PowerFeedFilterSet(PrimaryModelFilterSet, CabledObjectFilterSet, PathEndpo
         model = PowerFeed
         fields = (
             'id', 'name', 'status', 'type', 'supply', 'phase', 'voltage', 'amperage', 'max_utilization',
-            'available_power', 'mark_connected', 'cable_end', 'cable_connector', 'description',
+            'available_power', 'mark_connected', 'cable_end', 'cable_position', 'description',
         )
 
     def search(self, queryset, name, value):

netbox/dcim/forms/bulk_edit.py

@@ -208,7 +208,7 @@ class RackRoleBulkEditForm(OrganizationalModelBulkEditForm):
     fieldsets = (
         FieldSet('color', 'description'),
     )
-    nullable_fields = ('color', 'description', 'comments')
+    nullable_fields = ('color', 'description')
 
 
 class RackTypeBulkEditForm(PrimaryModelBulkEditForm):
@@ -474,7 +474,7 @@ class ManufacturerBulkEditForm(OrganizationalModelBulkEditForm):
     fieldsets = (
         FieldSet('description'),
     )
-    nullable_fields = ('description', 'comments')
+    nullable_fields = ('description',)
 
 
 class DeviceTypeBulkEditForm(PrimaryModelBulkEditForm):
@@ -1719,7 +1719,7 @@ class InventoryItemRoleBulkEditForm(OrganizationalModelBulkEditForm):
     fieldsets = (
         FieldSet('color', 'description'),
     )
-    nullable_fields = ('color', 'description', 'comments')
+    nullable_fields = ('color', 'description')
 
 
 class VirtualDeviceContextBulkEditForm(PrimaryModelBulkEditForm):

netbox/dcim/forms/bulk_import.py

@@ -183,7 +183,7 @@ class RackRoleImportForm(OrganizationalModelImportForm):
 
     class Meta:
         model = RackRole
-        fields = ('name', 'slug', 'color', 'description', 'owner', 'comments', 'tags')
+        fields = ('name', 'slug', 'color', 'description', 'owner', 'tags')
 
 
 class RackTypeImportForm(PrimaryModelImportForm):
@@ -400,7 +400,7 @@ class ManufacturerImportForm(OrganizationalModelImportForm):
 
     class Meta:
         model = Manufacturer
-        fields = ('name', 'slug', 'description', 'owner', 'comments', 'tags')
+        fields = ('name', 'slug', 'description', 'owner', 'tags')
 
 
 class DeviceTypeImportForm(PrimaryModelImportForm):
@@ -476,30 +476,14 @@ class ModuleTypeImportForm(PrimaryModelImportForm):
         required=False,
         help_text=_('Unit for module weight')
     )
-    attribute_data = forms.JSONField(
-        label=_('Attributes'),
-        required=False,
-        help_text=_('Attribute values for the assigned profile, passed as a dictionary')
-    )
 
     class Meta:
         model = ModuleType
         fields = [
             'manufacturer', 'model', 'part_number', 'description', 'airflow', 'weight', 'weight_unit', 'profile',
-            'attribute_data', 'owner', 'comments', 'tags',
+            'owner', 'comments', 'tags'
         ]
 
-    def clean(self):
-        super().clean()
-
-        # Attribute data may be included only if a profile is specified
-        if self.cleaned_data.get('attribute_data') and not self.cleaned_data.get('profile'):
-            raise forms.ValidationError(_("Profile must be specified if attribute data is provided."))
-
-        # Default attribute_data to an empty dictionary if a profile is specified (to enforce schema validation)
-        if self.cleaned_data.get('profile') and not self.cleaned_data.get('attribute_data'):
-            self.cleaned_data['attribute_data'] = {}
-
 
 class DeviceRoleImportForm(NestedGroupModelImportForm):
     parent = CSVModelChoiceField(
@@ -1091,6 +1075,12 @@ class FrontPortImportForm(OwnerCSVMixin, NetBoxModelImportForm):
         queryset=Device.objects.all(),
         to_field_name='name'
     )
+    rear_port = CSVModelChoiceField(
+        label=_('Rear port'),
+        queryset=RearPort.objects.all(),
+        to_field_name='name',
+        help_text=_('Corresponding rear port')
+    )
     type = CSVChoiceField(
         label=_('Type'),
         choices=PortTypeChoices,
@@ -1100,9 +1090,32 @@ class FrontPortImportForm(OwnerCSVMixin, NetBoxModelImportForm):
     class Meta:
         model = FrontPort
         fields = (
-            'device', 'name', 'label', 'type', 'color', 'mark_connected', 'positions', 'description', 'owner', 'tags'
+            'device', 'name', 'label', 'type', 'color', 'mark_connected', 'rear_port', 'rear_port_position',
+            'description', 'owner', 'tags'
         )
 
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+
+        # Limit RearPort choices to those belonging to this device (or VC master)
+        if self.is_bound and 'device' in self.data:
+            try:
+                device = self.fields['device'].to_python(self.data['device'])
+            except forms.ValidationError:
+                device = None
+        else:
+            try:
+                device = self.instance.device
+            except Device.DoesNotExist:
+                device = None
+
+        if device:
+            self.fields['rear_port'].queryset = RearPort.objects.filter(
+                device__in=[device, device.get_vc_master()]
+            )
+        else:
+            self.fields['rear_port'].queryset = RearPort.objects.none()
+
 
 class RearPortImportForm(OwnerCSVMixin, NetBoxModelImportForm):
     device = CSVModelChoiceField(
@@ -1298,7 +1311,7 @@ class InventoryItemRoleImportForm(OrganizationalModelImportForm):
 
     class Meta:
         model = InventoryItemRole
-        fields = ('name', 'slug', 'color', 'description', 'owner', 'comments')
+        fields = ('name', 'slug', 'color', 'description')
 
 
 #

netbox/dcim/forms/filtersets.py

@@ -13,7 +13,6 @@ from netbox.forms import (
     PrimaryModelFilterSetForm,
 )
 from tenancy.forms import ContactModelFilterForm, TenancyFilterForm
-from tenancy.models import Tenant
 from users.models import Owner, User
 from utilities.forms import BOOLEAN_WITH_BLANK_CHOICES, FilterForm, add_blank_choice
 from utilities.forms.fields import ColorField, DynamicModelChoiceField, DynamicModelMultipleChoiceField, TagFilterField
@@ -27,45 +26,35 @@ __all__ = (
     'CableFilterForm',
     'ConsoleConnectionFilterForm',
     'ConsolePortFilterForm',
-    'ConsolePortTemplateFilterForm',
     'ConsoleServerPortFilterForm',
-    'ConsoleServerPortTemplateFilterForm',
     'DeviceBayFilterForm',
-    'DeviceBayTemplateFilterForm',
     'DeviceFilterForm',
     'DeviceRoleFilterForm',
     'DeviceTypeFilterForm',
     'FrontPortFilterForm',
-    'FrontPortTemplateFilterForm',
     'InterfaceConnectionFilterForm',
     'InterfaceFilterForm',
-    'InterfaceTemplateFilterForm',
     'InventoryItemFilterForm',
-    'InventoryItemTemplateFilterForm',
     'InventoryItemRoleFilterForm',
     'LocationFilterForm',
     'MACAddressFilterForm',
     'ManufacturerFilterForm',
     'ModuleFilterForm',
     'ModuleBayFilterForm',
-    'ModuleBayTemplateFilterForm',
     'ModuleTypeFilterForm',
     'ModuleTypeProfileFilterForm',
     'PlatformFilterForm',
     'PowerConnectionFilterForm',
     'PowerFeedFilterForm',
     'PowerOutletFilterForm',
-    'PowerOutletTemplateFilterForm',
     'PowerPanelFilterForm',
     'PowerPortFilterForm',
-    'PowerPortTemplateFilterForm',
     'RackFilterForm',
     'RackElevationFilterForm',
     'RackReservationFilterForm',
     'RackRoleFilterForm',
     'RackTypeFilterForm',
     'RearPortFilterForm',
-    'RearPortTemplateFilterForm',
     'RegionFilterForm',
     'SiteFilterForm',
     'SiteGroupFilterForm',
@@ -134,11 +123,6 @@ class DeviceComponentFilterForm(NetBoxModelFilterSetForm):
         required=False,
         label=_('Device role')
     )
-    tenant_id = DynamicModelMultipleChoiceField(
-        queryset=Tenant.objects.all(),
-        required=False,
-        label=_('Tenant')
-    )
     device_id = DynamicModelMultipleChoiceField(
         queryset=Device.objects.all(),
         required=False,
@@ -147,8 +131,7 @@ class DeviceComponentFilterForm(NetBoxModelFilterSetForm):
             'location_id': '$location_id',
             'virtual_chassis_id': '$virtual_chassis_id',
             'device_type_id': '$device_type_id',
-            'role_id': '$role_id',
-            'tenant_id': '$tenant_id'
+            'role_id': '$role_id'
         },
         label=_('Device')
     )
@@ -1343,23 +1326,6 @@ class PowerFeedFilterForm(TenancyFilterForm, PrimaryModelFilterSetForm):
 # Device components
 #
 
-class DeviceComponentTemplateFilterForm(NetBoxModelFilterSetForm):
-    device_type_id = DynamicModelMultipleChoiceField(
-        queryset=DeviceType.objects.all(),
-        required=False,
-        label=_('Device type'),
-    )
-
-
-class ModularDeviceComponentTemplateFilterForm(DeviceComponentTemplateFilterForm):
-    module_type_id = DynamicModelMultipleChoiceField(
-        queryset=ModuleType.objects.all(),
-        required=False,
-        query_params={'manufacturer_id': '$manufacturer_id'},
-        label=_('Module Type'),
-    )
-
-
 class CabledFilterForm(forms.Form):
     cabled = forms.NullBooleanField(
         label=_('Cabled'),
@@ -1394,8 +1360,7 @@ class ConsolePortFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
         FieldSet('name', 'label', 'type', 'speed', name=_('Attributes')),
         FieldSet('region_id', 'site_group_id', 'site_id', 'location_id', 'rack_id', name=_('Location')),
         FieldSet(
-            'tenant_id', 'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
-            name=_('Device')
+            'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id', name=_('Device')
         ),
         FieldSet('cabled', 'connected', 'occupied', name=_('Connection')),
     )
@@ -1412,20 +1377,6 @@ class ConsolePortFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
     tag = TagFilterField(model)
 
 
-class ConsolePortTemplateFilterForm(ModularDeviceComponentTemplateFilterForm):
-    model = ConsolePortTemplate
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('name', 'label', 'type', name=_('Attributes')),
-        FieldSet('device_type_id', 'module_type_id', name=_('Device')),
-    )
-    type = forms.MultipleChoiceField(
-        label=_('Type'),
-        choices=ConsolePortTypeChoices,
-        required=False
-    )
-
-
 class ConsoleServerPortFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
     model = ConsoleServerPort
     fieldsets = (
@@ -1433,7 +1384,7 @@ class ConsoleServerPortFilterForm(PathEndpointFilterForm, DeviceComponentFilterF
         FieldSet('name', 'label', 'type', 'speed', name=_('Attributes')),
         FieldSet('region_id', 'site_group_id', 'site_id', 'location_id', 'rack_id', name=_('Location')),
         FieldSet(
-            'tenant_id', 'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
+            'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
             name=_('Device')
         ),
         FieldSet('cabled', 'connected', 'occupied', name=_('Connection')),
@@ -1451,20 +1402,6 @@ class ConsoleServerPortFilterForm(PathEndpointFilterForm, DeviceComponentFilterF
     tag = TagFilterField(model)
 
 
-class ConsoleServerPortTemplateFilterForm(ModularDeviceComponentTemplateFilterForm):
-    model = ConsoleServerPortTemplate
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('name', 'label', 'type', name=_('Attributes')),
-        FieldSet('device_type_id', 'module_type_id', name=_('Device')),
-    )
-    type = forms.MultipleChoiceField(
-        label=_('Type'),
-        choices=ConsolePortTypeChoices,
-        required=False
-    )
-
-
 class PowerPortFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
     model = PowerPort
     fieldsets = (
@@ -1472,8 +1409,7 @@ class PowerPortFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
         FieldSet('name', 'label', 'type', name=_('Attributes')),
         FieldSet('region_id', 'site_group_id', 'site_id', 'location_id', 'rack_id', name=_('Location')),
         FieldSet(
-            'tenant_id', 'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
-            name=_('Device')
+            'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id', name=_('Device')
         ),
         FieldSet('cabled', 'connected', 'occupied', name=_('Connection')),
     )
@@ -1485,20 +1421,6 @@ class PowerPortFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
     tag = TagFilterField(model)
 
 
-class PowerPortTemplateFilterForm(ModularDeviceComponentTemplateFilterForm):
-    model = PowerPortTemplate
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('name', 'label', 'type', name=_('Attributes')),
-        FieldSet('device_type_id', 'module_type_id', name=_('Device')),
-    )
-    type = forms.MultipleChoiceField(
-        label=_('Type'),
-        choices=PowerPortTypeChoices,
-        required=False
-    )
-
-
 class PowerOutletFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
     model = PowerOutlet
     fieldsets = (
@@ -1506,7 +1428,7 @@ class PowerOutletFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
         FieldSet('name', 'label', 'type', 'color', 'status', name=_('Attributes')),
         FieldSet('region_id', 'site_group_id', 'site_id', 'location_id', 'rack_id', name=_('Location')),
         FieldSet(
-            'tenant_id', 'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
+            'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
             name=_('Device')
         ),
         FieldSet('cabled', 'connected', 'occupied', name=_('Connection')),
@@ -1528,20 +1450,6 @@ class PowerOutletFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
     )
 
 
-class PowerOutletTemplateFilterForm(ModularDeviceComponentTemplateFilterForm):
-    model = PowerOutletTemplate
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('name', 'label', 'type', name=_('Attributes')),
-        FieldSet('device_type_id', 'module_type_id', name=_('Device')),
-    )
-    type = forms.MultipleChoiceField(
-        label=_('Type'),
-        choices=PowerOutletTypeChoices,
-        required=False
-    )
-
-
 class InterfaceFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
     model = Interface
     fieldsets = (
@@ -1553,8 +1461,7 @@ class InterfaceFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
         FieldSet('rf_role', 'rf_channel', 'rf_channel_width', 'tx_power', name=_('Wireless')),
         FieldSet('region_id', 'site_group_id', 'site_id', 'location_id', 'rack_id', name=_('Location')),
         FieldSet(
-            'tenant_id', 'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
-            'vdc_id',
+            'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id', 'vdc_id',
             name=_('Device')
         ),
         FieldSet('cabled', 'connected', 'occupied', name=_('Connection')),
@@ -1669,59 +1576,13 @@ class InterfaceFilterForm(PathEndpointFilterForm, DeviceComponentFilterForm):
     tag = TagFilterField(model)
 
 
-class InterfaceTemplateFilterForm(ModularDeviceComponentTemplateFilterForm):
-    model = InterfaceTemplate
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('name', 'label', 'type', 'enabled', 'mgmt_only', name=_('Attributes')),
-        FieldSet('poe_mode', 'poe_type', name=_('PoE')),
-        FieldSet('rf_role', name=_('Wireless')),
-        FieldSet('device_type_id', 'module_type_id', name=_('Device')),
-    )
-    type = forms.MultipleChoiceField(
-        label=_('Type'),
-        choices=InterfaceTypeChoices,
-        required=False
-    )
-    enabled = forms.NullBooleanField(
-        label=_('Enabled'),
-        required=False,
-        widget=forms.Select(
-            choices=BOOLEAN_WITH_BLANK_CHOICES
-        )
-    )
-    mgmt_only = forms.NullBooleanField(
-        label=_('Management only'),
-        required=False,
-        widget=forms.Select(
-            choices=BOOLEAN_WITH_BLANK_CHOICES
-        )
-    )
-    poe_mode = forms.MultipleChoiceField(
-        choices=InterfacePoEModeChoices,
-        required=False,
-        label=_('PoE mode')
-    )
-    poe_type = forms.MultipleChoiceField(
-        choices=InterfacePoETypeChoices,
-        required=False,
-        label=_('PoE type')
-    )
-    rf_role = forms.MultipleChoiceField(
-        choices=WirelessRoleChoices,
-        required=False,
-        label=_('Wireless role')
-    )
-
-
 class FrontPortFilterForm(CabledFilterForm, DeviceComponentFilterForm):
     fieldsets = (
         FieldSet('q', 'filter_id', 'tag', 'owner_id'),
         FieldSet('name', 'label', 'type', 'color', name=_('Attributes')),
         FieldSet('region_id', 'site_group_id', 'site_id', 'location_id', 'rack_id', name=_('Location')),
         FieldSet(
-            'tenant_id', 'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
-            name=_('Device')
+            'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id', name=_('Device')
         ),
         FieldSet('cabled', 'occupied', name=_('Cable')),
     )
@@ -1738,24 +1599,6 @@ class FrontPortFilterForm(CabledFilterForm, DeviceComponentFilterForm):
     tag = TagFilterField(model)
 
 
-class FrontPortTemplateFilterForm(ModularDeviceComponentTemplateFilterForm):
-    model = FrontPortTemplate
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('name', 'label', 'type', 'color', name=_('Attributes')),
-        FieldSet('device_type_id', 'module_type_id', name=_('Device')),
-    )
-    type = forms.MultipleChoiceField(
-        label=_('Type'),
-        choices=PortTypeChoices,
-        required=False
-    )
-    color = ColorField(
-        label=_('Color'),
-        required=False
-    )
-
-
 class RearPortFilterForm(CabledFilterForm, DeviceComponentFilterForm):
     model = RearPort
     fieldsets = (
@@ -1763,7 +1606,7 @@ class RearPortFilterForm(CabledFilterForm, DeviceComponentFilterForm):
         FieldSet('name', 'label', 'type', 'color', name=_('Attributes')),
         FieldSet('region_id', 'site_group_id', 'site_id', 'location_id', 'rack_id', name=_('Location')),
         FieldSet(
-            'tenant_id', 'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
+            'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
             name=_('Device')
         ),
         FieldSet('cabled', 'occupied', name=_('Cable')),
@@ -1780,24 +1623,6 @@ class RearPortFilterForm(CabledFilterForm, DeviceComponentFilterForm):
     tag = TagFilterField(model)
 
 
-class RearPortTemplateFilterForm(ModularDeviceComponentTemplateFilterForm):
-    model = RearPortTemplate
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('name', 'label', 'type', 'color', name=_('Attributes')),
-        FieldSet('device_type_id', 'module_type_id', name=_('Device')),
-    )
-    type = forms.MultipleChoiceField(
-        label=_('Type'),
-        choices=PortTypeChoices,
-        required=False
-    )
-    color = ColorField(
-        label=_('Color'),
-        required=False
-    )
-
-
 class ModuleBayFilterForm(DeviceComponentFilterForm):
     model = ModuleBay
     fieldsets = (
@@ -1805,7 +1630,7 @@ class ModuleBayFilterForm(DeviceComponentFilterForm):
         FieldSet('name', 'label', 'position', name=_('Attributes')),
         FieldSet('region_id', 'site_group_id', 'site_id', 'location_id', 'rack_id', name=_('Location')),
         FieldSet(
-            'tenant_id', 'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
+            'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
             name=_('Device')
         ),
     )
@@ -1816,19 +1641,6 @@ class ModuleBayFilterForm(DeviceComponentFilterForm):
     )
 
 
-class ModuleBayTemplateFilterForm(ModularDeviceComponentTemplateFilterForm):
-    model = ModuleBayTemplate
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('name', 'label', 'position', name=_('Attributes')),
-        FieldSet('device_type_id', 'module_type_id', name=_('Device')),
-    )
-    position = forms.CharField(
-        label=_('Position'),
-        required=False,
-    )
-
-
 class DeviceBayFilterForm(DeviceComponentFilterForm):
     model = DeviceBay
     fieldsets = (
@@ -1836,22 +1648,13 @@ class DeviceBayFilterForm(DeviceComponentFilterForm):
         FieldSet('name', 'label', name=_('Attributes')),
         FieldSet('region_id', 'site_group_id', 'site_id', 'location_id', 'rack_id', name=_('Location')),
         FieldSet(
-            'tenant_id', 'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
+            'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
             name=_('Device')
         ),
     )
     tag = TagFilterField(model)
 
 
-class DeviceBayTemplateFilterForm(DeviceComponentTemplateFilterForm):
-    model = DeviceBayTemplate
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('name', 'label', name=_('Attributes')),
-        FieldSet('device_type_id', name=_('Device')),
-    )
-
-
 class InventoryItemFilterForm(DeviceComponentFilterForm):
     model = InventoryItem
     fieldsets = (
@@ -1862,7 +1665,7 @@ class InventoryItemFilterForm(DeviceComponentFilterForm):
         ),
         FieldSet('region_id', 'site_group_id', 'site_id', 'location_id', 'rack_id', name=_('Location')),
         FieldSet(
-            'tenant_id', 'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
+            'device_type_id', 'device_role_id', 'device_id', 'device_status', 'virtual_chassis_id',
             name=_('Device')
         ),
     )
@@ -1899,25 +1702,6 @@ class InventoryItemFilterForm(DeviceComponentFilterForm):
     tag = TagFilterField(model)
 
 
-class InventoryItemTemplateFilterForm(DeviceComponentTemplateFilterForm):
-    model = InventoryItemTemplate
-    fieldsets = (
-        FieldSet('q', 'filter_id', 'tag'),
-        FieldSet('name', 'label', 'role_id', 'manufacturer_id', name=_('Attributes')),
-        FieldSet('device_type_id', name=_('Device')),
-    )
-    role_id = DynamicModelMultipleChoiceField(
-        queryset=InventoryItemRole.objects.all(),
-        required=False,
-        label=_('Role')
-    )
-    manufacturer_id = DynamicModelMultipleChoiceField(
-        queryset=Manufacturer.objects.all(),
-        required=False,
-        label=_('Manufacturer')
-    )
-
-
 #
 # Device component roles
 #

netbox/dcim/forms/mixins.py

@@ -1,12 +1,10 @@
 from django import forms
 from django.contrib.contenttypes.models import ContentType
 from django.core.exceptions import ObjectDoesNotExist, ValidationError
-from django.db import connection
-from django.db.models.signals import post_save
 from django.utils.translation import gettext_lazy as _
 
 from dcim.constants import LOCATION_SCOPE_TYPES
-from dcim.models import PortMapping, PortTemplateMapping, Site
+from dcim.models import Site
 from utilities.forms import get_field_value
 from utilities.forms.fields import (
     ContentTypeChoiceField, CSVContentTypeField, DynamicModelChoiceField,
@@ -15,7 +13,6 @@ from utilities.templatetags.builtins.filters import bettertitle
 from utilities.forms.widgets import HTMXSelect
 
 __all__ = (
-    'FrontPortFormMixin',
     'ScopedBulkEditForm',
     'ScopedForm',
     'ScopedImportForm',
@@ -131,91 +128,3 @@ class ScopedImportForm(forms.Form):
                     "Please select a {scope_type}."
                 ).format(scope_type=scope_type.model_class()._meta.model_name)
             })
-
-
-class FrontPortFormMixin(forms.Form):
-    rear_ports = forms.MultipleChoiceField(
-        choices=[],
-        label=_('Rear ports'),
-        widget=forms.SelectMultiple(attrs={'size': 8})
-    )
-
-    def clean(self):
-        super().clean()
-
-        # Check that the total number of FrontPorts and positions matches the selected number of RearPort:position
-        # mappings. Note that `name` will be a list under FrontPortCreateForm, in which cases we multiply the number of
-        # FrontPorts being creation by the number of positions.
-        positions = self.cleaned_data['positions']
-        frontport_count = len(self.cleaned_data['name']) if type(self.cleaned_data['name']) is list else 1
-        rearport_count = len(self.cleaned_data['rear_ports'])
-        if frontport_count * positions != rearport_count:
-            raise forms.ValidationError({
-                'rear_ports': _(
-                    "The total number of front port positions ({frontport_count}) must match the selected number of "
-                    "rear port positions ({rearport_count})."
-                ).format(
-                    frontport_count=frontport_count,
-                    rearport_count=rearport_count
-                )
-            })
-
-    def _save_m2m(self):
-        super()._save_m2m()
-
-        # TODO: Can this be made more efficient?
-        # Delete existing rear port mappings
-        self.port_mapping_model.objects.filter(front_port_id=self.instance.pk).delete()
-
-        # Create new rear port mappings
-        mappings = []
-        if self.port_mapping_model is PortTemplateMapping:
-            params = {
-                'device_type_id': self.instance.device_type_id,
-                'module_type_id': self.instance.module_type_id,
-            }
-        else:
-            params = {
-                'device_id': self.instance.device_id,
-            }
-        for i, rp_position in enumerate(self.cleaned_data['rear_ports'], start=1):
-            rear_port_id, rear_port_position = rp_position.split(':')
-            mappings.append(
-                self.port_mapping_model(**{
-                    **params,
-                    'front_port_id': self.instance.pk,
-                    'front_port_position': i,
-                    'rear_port_id': rear_port_id,
-                    'rear_port_position': rear_port_position,
-                })
-            )
-        self.port_mapping_model.objects.bulk_create(mappings)
-        # Send post_save signals
-        for mapping in mappings:
-            post_save.send(
-                sender=PortMapping,
-                instance=mapping,
-                created=True,
-                raw=False,
-                using=connection,
-                update_fields=None
-            )
-
-    def _get_rear_port_choices(self, parent_filter, front_port):
-        """
-        Return a list of choices representing each available rear port & position pair on the parent object (identified
-        by a Q filter), excluding those assigned to the specified instance.
-        """
-        occupied_rear_port_positions = [
-            f'{mapping.rear_port_id}:{mapping.rear_port_position}'
-            for mapping in self.port_mapping_model.objects.filter(parent_filter).exclude(front_port=front_port.pk)
-        ]
-
-        choices = []
-        for rear_port in self.rear_port_model.objects.filter(parent_filter):
-            for i in range(1, rear_port.positions + 1):
-                pair_id = f'{rear_port.pk}:{i}'
-                if pair_id not in occupied_rear_port_positions:
-                    pair_label = f'{rear_port.name}:{i}'
-                    choices.append((pair_id, pair_label))
-        return choices

netbox/dcim/forms/model_forms.py

@@ -6,7 +6,6 @@ from timezone_field import TimeZoneFormField
 
 from dcim.choices import *
 from dcim.constants import *
-from dcim.forms.mixins import FrontPortFormMixin
 from dcim.models import *
 from extras.models import ConfigTemplate
 from ipam.choices import VLANQinQRoleChoices
@@ -202,7 +201,7 @@ class RackRoleForm(OrganizationalModelForm):
     class Meta:
         model = RackRole
         fields = [
-            'name', 'slug', 'color', 'description', 'owner', 'comments', 'tags',
+            'name', 'slug', 'color', 'description', 'owner', 'tags',
         ]
 
 
@@ -345,7 +344,7 @@ class ManufacturerForm(OrganizationalModelForm):
     class Meta:
         model = Manufacturer
         fields = [
-            'name', 'slug', 'description', 'owner', 'comments', 'tags',
+            'name', 'slug', 'description', 'owner', 'tags',
         ]
 
 
@@ -1113,45 +1112,34 @@ class InterfaceTemplateForm(ModularComponentTemplateForm):
         ]
 
 
-class FrontPortTemplateForm(FrontPortFormMixin, ModularComponentTemplateForm):
+class FrontPortTemplateForm(ModularComponentTemplateForm):
+    rear_port = DynamicModelChoiceField(
+        label=_('Rear port'),
+        queryset=RearPortTemplate.objects.all(),
+        required=False,
+        query_params={
+            'device_type_id': '$device_type',
+            'module_type_id': '$module_type',
+        }
+    )
+
     fieldsets = (
         FieldSet(
             TabbedGroups(
                 FieldSet('device_type', name=_('Device Type')),
                 FieldSet('module_type', name=_('Module Type')),
             ),
-            'name', 'label', 'type', 'positions', 'rear_ports', 'description',
+            'name', 'label', 'type', 'color', 'rear_port', 'rear_port_position', 'description',
         ),
     )
 
-    port_mapping_model = PortTemplateMapping
-    rear_port_model = RearPortTemplate
-
     class Meta:
         model = FrontPortTemplate
         fields = [
-            'device_type', 'module_type', 'name', 'label', 'type', 'color', 'positions', 'description',
+            'device_type', 'module_type', 'name', 'label', 'type', 'color', 'rear_port', 'rear_port_position',
+            'description',
         ]
 
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-
-        # Populate rear port choices based on parent DeviceType or ModuleType
-        if device_type_id := self.data.get('device_type') or self.initial.get('device_type'):
-            parent_filter = Q(device_type=device_type_id)
-        elif module_type_id := self.data.get('module_type') or self.initial.get('module_type'):
-            parent_filter = Q(module_type=module_type_id)
-        else:
-            return
-        self.fields['rear_ports'].choices = self._get_rear_port_choices(parent_filter, self.instance)
-
-        # Set initial rear port mappings
-        if self.instance.pk:
-            self.initial['rear_ports'] = [
-                f'{mapping.rear_port_id}:{mapping.rear_port_position}'
-                for mapping in PortTemplateMapping.objects.filter(front_port_id=self.instance.pk)
-            ]
-
 
 class RearPortTemplateForm(ModularComponentTemplateForm):
     fieldsets = (
@@ -1590,41 +1578,28 @@ class InterfaceForm(InterfaceCommonForm, ModularDeviceComponentForm):
         }
 
 
-class FrontPortForm(FrontPortFormMixin, ModularDeviceComponentForm):
+class FrontPortForm(ModularDeviceComponentForm):
+    rear_port = DynamicModelChoiceField(
+        queryset=RearPort.objects.all(),
+        query_params={
+            'device_id': '$device',
+        }
+    )
+
     fieldsets = (
         FieldSet(
-            'device', 'module', 'name', 'label', 'type', 'color', 'positions', 'rear_ports', 'mark_connected',
+            'device', 'module', 'name', 'label', 'type', 'color', 'rear_port', 'rear_port_position', 'mark_connected',
             'description', 'tags',
         ),
     )
 
-    port_mapping_model = PortMapping
-    rear_port_model = RearPort
-
     class Meta:
         model = FrontPort
         fields = [
-            'device', 'module', 'name', 'label', 'type', 'color', 'positions', 'mark_connected', 'description', 'owner',
-            'tags',
+            'device', 'module', 'name', 'label', 'type', 'color', 'rear_port', 'rear_port_position', 'mark_connected',
+            'description', 'owner', 'tags',
         ]
 
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-
-        # Populate rear port choices
-        if device_id := self.data.get('device') or self.initial.get('device'):
-            parent_filter = Q(device=device_id)
-        else:
-            return
-        self.fields['rear_ports'].choices = self._get_rear_port_choices(parent_filter, self.instance)
-
-        # Set initial rear port mappings
-        if self.instance.pk:
-            self.initial['rear_ports'] = [
-                f'{mapping.rear_port_id}:{mapping.rear_port_position}'
-                for mapping in PortMapping.objects.filter(front_port_id=self.instance.pk)
-            ]
-
 
 class RearPortForm(ModularDeviceComponentForm):
     fieldsets = (
@@ -1840,7 +1815,7 @@ class InventoryItemRoleForm(OrganizationalModelForm):
     class Meta:
         model = InventoryItemRole
         fields = [
-            'name', 'slug', 'color', 'description', 'owner', 'comments', 'tags',
+            'name', 'slug', 'color', 'description', 'owner', 'tags',
         ]
 
 

netbox/dcim/forms/object_create.py

@@ -109,30 +109,85 @@ class InterfaceTemplateCreateForm(ComponentCreateForm, model_forms.InterfaceTemp
 
 
 class FrontPortTemplateCreateForm(ComponentCreateForm, model_forms.FrontPortTemplateForm):
+    rear_port = forms.MultipleChoiceField(
+        choices=[],
+        label=_('Rear ports'),
+        help_text=_('Select one rear port assignment for each front port being created.'),
+        widget=forms.SelectMultiple(attrs={'size': 6})
+    )
 
-    # Override fieldsets from FrontPortTemplateForm
+    # Override fieldsets from FrontPortTemplateForm to omit rear_port_position
     fieldsets = (
         FieldSet(
             TabbedGroups(
                 FieldSet('device_type', name=_('Device Type')),
                 FieldSet('module_type', name=_('Module Type')),
             ),
-            'name', 'label', 'type', 'color', 'positions', 'rear_ports', 'description',
+            'name', 'label', 'type', 'color', 'rear_port', 'description',
         ),
     )
 
-    class Meta:
-        model = FrontPortTemplate
-        fields = (
-            'device_type', 'module_type', 'type', 'color', 'positions', 'description',
-        )
+    class Meta(model_forms.FrontPortTemplateForm.Meta):
+        exclude = ('name', 'label', 'rear_port', 'rear_port_position')
+
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+
+        # TODO: This needs better validation
+        if 'device_type' in self.initial or self.data.get('device_type'):
+            parent = DeviceType.objects.get(
+                pk=self.initial.get('device_type') or self.data.get('device_type')
+            )
+        elif 'module_type' in self.initial or self.data.get('module_type'):
+            parent = ModuleType.objects.get(
+                pk=self.initial.get('module_type') or self.data.get('module_type')
+            )
+        else:
+            return
+
+        # Determine which rear port positions are occupied. These will be excluded from the list of available mappings.
+        occupied_port_positions = [
+            (front_port.rear_port_id, front_port.rear_port_position)
+            for front_port in parent.frontporttemplates.all()
+        ]
+
+        # Populate rear port choices
+        choices = []
+        rear_ports = parent.rearporttemplates.all()
+        for rear_port in rear_ports:
+            for i in range(1, rear_port.positions + 1):
+                if (rear_port.pk, i) not in occupied_port_positions:
+                    choices.append(
+                        ('{}:{}'.format(rear_port.pk, i), '{}:{}'.format(rear_port.name, i))
+                    )
+        self.fields['rear_port'].choices = choices
+
+    def clean(self):
+        super().clean()
+
+        # Check that the number of FrontPortTemplates to be created matches the selected number of RearPortTemplate
+        # positions
+        frontport_count = len(self.cleaned_data['name'])
+        rearport_count = len(self.cleaned_data['rear_port'])
+        if frontport_count != rearport_count:
+            raise forms.ValidationError({
+                'rear_port': _(
+                    "The number of front port templates to be created ({frontport_count}) must match the selected "
+                    "number of rear port positions ({rearport_count})."
+                ).format(
+                    frontport_count=frontport_count,
+                    rearport_count=rearport_count
+                )
+            })
 
     def get_iterative_data(self, iteration):
-        positions = self.cleaned_data['positions']
-        offset = positions * iteration
+
+        # Assign rear port and position from selected set
+        rear_port, position = self.cleaned_data['rear_port'][iteration].split(':')
 
         return {
-            'rear_ports': self.cleaned_data['rear_ports'][offset:offset + positions]
+            'rear_port': int(rear_port),
+            'rear_port_position': int(position),
         }
 
 
@@ -214,26 +269,74 @@ class FrontPortCreateForm(ComponentCreateForm, model_forms.FrontPortForm):
             }
         )
     )
+    rear_port = forms.MultipleChoiceField(
+        choices=[],
+        label=_('Rear ports'),
+        help_text=_('Select one rear port assignment for each front port being created.'),
+        widget=forms.SelectMultiple(attrs={'size': 6})
+    )
 
     # Override fieldsets from FrontPortForm to omit rear_port_position
     fieldsets = (
         FieldSet(
-            'device', 'module', 'name', 'label', 'type', 'color', 'positions', 'rear_ports', 'mark_connected',
-            'description', 'tags',
+            'device', 'module', 'name', 'label', 'type', 'color', 'rear_port', 'mark_connected', 'description', 'tags',
         ),
     )
 
-    class Meta:
-        model = FrontPort
-        fields = [
-            'device', 'module', 'type', 'color', 'positions', 'mark_connected', 'description', 'owner', 'tags',
+    class Meta(model_forms.FrontPortForm.Meta):
+        exclude = ('name', 'label', 'rear_port', 'rear_port_position')
+
+    def __init__(self, *args, **kwargs):
+        super().__init__(*args, **kwargs)
+
+        if device_id := self.data.get('device') or self.initial.get('device'):
+            device = Device.objects.get(pk=device_id)
+        else:
+            return
+
+        # Determine which rear port positions are occupied. These will be excluded from the list of available
+        # mappings.
+        occupied_port_positions = [
+            (front_port.rear_port_id, front_port.rear_port_position)
+            for front_port in device.frontports.all()
         ]
 
+        # Populate rear port choices
+        choices = []
+        rear_ports = RearPort.objects.filter(device=device)
+        for rear_port in rear_ports:
+            for i in range(1, rear_port.positions + 1):
+                if (rear_port.pk, i) not in occupied_port_positions:
+                    choices.append(
+                        ('{}:{}'.format(rear_port.pk, i), '{}:{}'.format(rear_port.name, i))
+                    )
+        self.fields['rear_port'].choices = choices
+
+    def clean(self):
+        super().clean()
+
+        # Check that the number of FrontPorts to be created matches the selected number of RearPort positions
+        frontport_count = len(self.cleaned_data['name'])
+        rearport_count = len(self.cleaned_data['rear_port'])
+        if frontport_count != rearport_count:
+            raise forms.ValidationError({
+                'rear_port': _(
+                    "The number of front ports to be created ({frontport_count}) must match the selected number of "
+                    "rear port positions ({rearport_count})."
+                ).format(
+                    frontport_count=frontport_count,
+                    rearport_count=rearport_count
+                )
+            })
+
     def get_iterative_data(self, iteration):
-        positions = self.cleaned_data['positions']
-        offset = positions * iteration
+
+        # Assign rear port and position from selected set
+        rear_port, position = self.cleaned_data['rear_port'][iteration].split(':')
+
         return {
-            'rear_ports': self.cleaned_data['rear_ports'][offset:offset + positions]
+            'rear_port': int(rear_port),
+            'rear_port_position': int(position),
         }
 
 

netbox/dcim/forms/object_import.py

@@ -13,7 +13,6 @@ __all__ = (
     'InterfaceTemplateImportForm',
     'InventoryItemTemplateImportForm',
     'ModuleBayTemplateImportForm',
-    'PortTemplateMappingImportForm',
     'PowerOutletTemplateImportForm',
     'PowerPortTemplateImportForm',
     'RearPortTemplateImportForm',
@@ -114,11 +113,31 @@ class FrontPortTemplateImportForm(forms.ModelForm):
         label=_('Type'),
         choices=PortTypeChoices.CHOICES
     )
+    rear_port = forms.ModelChoiceField(
+        label=_('Rear port'),
+        queryset=RearPortTemplate.objects.all(),
+        to_field_name='name'
+    )
+
+    def clean_device_type(self):
+        if device_type := self.cleaned_data['device_type']:
+            rear_port = self.fields['rear_port']
+            rear_port.queryset = rear_port.queryset.filter(device_type=device_type)
+
+        return device_type
+
+    def clean_module_type(self):
+        if module_type := self.cleaned_data['module_type']:
+            rear_port = self.fields['rear_port']
+            rear_port.queryset = rear_port.queryset.filter(module_type=module_type)
+
+        return module_type
 
     class Meta:
         model = FrontPortTemplate
         fields = [
-            'device_type', 'module_type', 'name', 'type', 'color', 'positions', 'label', 'description',
+            'device_type', 'module_type', 'name', 'type', 'color', 'rear_port', 'rear_port_position', 'label',
+            'description',
         ]
 
 
@@ -135,25 +154,6 @@ class RearPortTemplateImportForm(forms.ModelForm):
         ]
 
 
-class PortTemplateMappingImportForm(forms.ModelForm):
-    front_port = forms.ModelChoiceField(
-        label=_('Front port'),
-        queryset=FrontPortTemplate.objects.all(),
-        to_field_name='name',
-    )
-    rear_port = forms.ModelChoiceField(
-        label=_('Rear port'),
-        queryset=RearPortTemplate.objects.all(),
-        to_field_name='name',
-    )
-
-    class Meta:
-        model = PortTemplateMapping
-        fields = [
-            'front_port', 'front_port_position', 'rear_port', 'rear_port_position',
-        ]
-
-
 class ModuleBayTemplateImportForm(forms.ModelForm):
 
     class Meta:

netbox/dcim/graphql/filter_mixins.py

@@ -6,14 +6,15 @@ import strawberry_django
 from strawberry import ID
 from strawberry_django import BaseFilterLookup, FilterLookup
 
+from core.graphql.filter_mixins import BaseFilterMixin, ChangeLogFilterMixin
 from core.graphql.filters import ContentTypeFilter
+from netbox.graphql.filter_mixins import NetBoxModelFilterMixin, PrimaryModelFilterMixin, WeightFilterMixin
 from .enums import *
 
 if TYPE_CHECKING:
     from netbox.graphql.filter_lookups import IntegerLookup
     from extras.graphql.filters import ConfigTemplateFilter
     from ipam.graphql.filters import VLANFilter, VLANTranslationPolicyFilter
-    from dcim.graphql.filters import LocationFilter, RegionFilter, SiteFilter, SiteGroupFilter
     from .filters import *
 
 __all__ = (
@@ -21,47 +22,24 @@ __all__ = (
     'ComponentModelFilterMixin',
     'ComponentTemplateFilterMixin',
     'InterfaceBaseFilterMixin',
-    'ModularComponentFilterMixin',
+    'ModularComponentModelFilterMixin',
     'ModularComponentTemplateFilterMixin',
-    'RackFilterMixin',
+    'RackBaseFilterMixin',
     'RenderConfigFilterMixin',
     'ScopedFilterMixin',
 )
 
 
 @dataclass
-class ScopedFilterMixin:
+class ScopedFilterMixin(BaseFilterMixin):
     scope_type: Annotated['ContentTypeFilter', strawberry.lazy('core.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
     scope_id: ID | None = strawberry_django.filter_field()
 
-    # Cached relations
-    _location: Annotated['LocationFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
-        strawberry_django.filter_field(name='location')
-    )
-    _region: Annotated['RegionFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
-        strawberry_django.filter_field(name='region')
-    )
-    _site_group: Annotated['SiteGroupFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
-        strawberry_django.filter_field(name='site_group')
-    )
-    _site: Annotated['SiteFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
-        strawberry_django.filter_field(name='site')
-    )
-
 
 @dataclass
-class ComponentModelFilterMixin:
-    _site: Annotated['SiteFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
-        strawberry_django.filter_field(name='site')
-    )
-    _location: Annotated['LocationFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
-        strawberry_django.filter_field(name='location')
-    )
-    _rack: Annotated['RackFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
-        strawberry_django.filter_field(name='rack')
-    )
+class ComponentModelFilterMixin(NetBoxModelFilterMixin):
     device: Annotated['DeviceFilter', strawberry.lazy('dcim.graphql.filters')] | None = strawberry_django.filter_field()
     device_id: ID | None = strawberry_django.filter_field()
     name: FilterLookup[str] | None = strawberry_django.filter_field()
@@ -70,7 +48,7 @@ class ComponentModelFilterMixin:
 
 
 @dataclass
-class ModularComponentFilterMixin(ComponentModelFilterMixin):
+class ModularComponentModelFilterMixin(ComponentModelFilterMixin):
     module: Annotated['ModuleFilter', strawberry.lazy('dcim.graphql.filters')] | None = strawberry_django.filter_field()
     module_id: ID | None = strawberry_django.filter_field()
     inventory_items: Annotated['InventoryItemFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
@@ -79,7 +57,7 @@ class ModularComponentFilterMixin(ComponentModelFilterMixin):
 
 
 @dataclass
-class CabledObjectModelFilterMixin:
+class CabledObjectModelFilterMixin(BaseFilterMixin):
     cable: Annotated['CableFilter', strawberry.lazy('dcim.graphql.filters')] | None = strawberry_django.filter_field()
     cable_id: ID | None = strawberry_django.filter_field()
     cable_end: (
@@ -89,7 +67,7 @@ class CabledObjectModelFilterMixin:
 
 
 @dataclass
-class ComponentTemplateFilterMixin:
+class ComponentTemplateFilterMixin(ChangeLogFilterMixin):
     device_type: Annotated['DeviceTypeFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -107,7 +85,7 @@ class ModularComponentTemplateFilterMixin(ComponentTemplateFilterMixin):
 
 
 @dataclass
-class RenderConfigFilterMixin:
+class RenderConfigFilterMixin(BaseFilterMixin):
     config_template: Annotated['ConfigTemplateFilter', strawberry.lazy('extras.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -115,7 +93,7 @@ class RenderConfigFilterMixin:
 
 
 @dataclass
-class InterfaceBaseFilterMixin:
+class InterfaceBaseFilterMixin(BaseFilterMixin):
     enabled: FilterLookup[bool] | None = strawberry_django.filter_field()
     mtu: Annotated['IntegerLookup', strawberry.lazy('netbox.graphql.filter_lookups')] | None = (
         strawberry_django.filter_field()
@@ -146,7 +124,7 @@ class InterfaceBaseFilterMixin:
 
 
 @dataclass
-class RackFilterMixin:
+class RackBaseFilterMixin(WeightFilterMixin, PrimaryModelFilterMixin):
     width: BaseFilterLookup[Annotated['RackWidthEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )

netbox/dcim/graphql/filters.py

@@ -6,25 +6,29 @@ import strawberry_django
 from strawberry.scalars import ID
 from strawberry_django import BaseFilterLookup, ComparisonFilterLookup, FilterLookup
 
+from core.graphql.filter_mixins import ChangeLogFilterMixin
 from dcim import models
 from dcim.constants import *
 from dcim.graphql.enums import InterfaceKindEnum
-from dcim.graphql.filter_mixins import (
-    ComponentModelFilterMixin, ComponentTemplateFilterMixin, ModularComponentFilterMixin,
-    ModularComponentTemplateFilterMixin, RackFilterMixin,
-)
 from extras.graphql.filter_mixins import ConfigContextFilterMixin
-from netbox.graphql.filter_mixins import ImageAttachmentFilterMixin, WeightFilterMixin
-from netbox.graphql.filters import (
-    BaseModelFilter, ChangeLoggedModelFilter, NestedGroupModelFilter, OrganizationalModelFilter, PrimaryModelFilter,
-    NetBoxModelFilter,
+from netbox.graphql.filter_mixins import (
+    PrimaryModelFilterMixin,
+    OrganizationalModelFilterMixin,
+    NestedGroupModelFilterMixin,
+    ImageAttachmentFilterMixin,
+    WeightFilterMixin,
 )
 from tenancy.graphql.filter_mixins import ContactFilterMixin, TenancyFilterMixin
 from virtualization.models import VMInterface
 
 from .filter_mixins import (
     CabledObjectModelFilterMixin,
+    ComponentModelFilterMixin,
+    ComponentTemplateFilterMixin,
     InterfaceBaseFilterMixin,
+    ModularComponentModelFilterMixin,
+    ModularComponentTemplateFilterMixin,
+    RackBaseFilterMixin,
     RenderConfigFilterMixin,
 )
 
@@ -71,8 +75,6 @@ __all__ = (
     'ModuleTypeFilter',
     'ModuleTypeProfileFilter',
     'PlatformFilter',
-    'PortMappingFilter',
-    'PortTemplateMappingFilter',
     'PowerFeedFilter',
     'PowerOutletFilter',
     'PowerOutletTemplateFilter',
@@ -94,7 +96,7 @@ __all__ = (
 
 
 @strawberry_django.filter_type(models.Cable, lookups=True)
-class CableFilter(TenancyFilterMixin, PrimaryModelFilter):
+class CableFilter(PrimaryModelFilterMixin, TenancyFilterMixin):
     type: BaseFilterLookup[Annotated['CableTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -117,7 +119,7 @@ class CableFilter(TenancyFilterMixin, PrimaryModelFilter):
 
 
 @strawberry_django.filter_type(models.CableTermination, lookups=True)
-class CableTerminationFilter(ChangeLoggedModelFilter):
+class CableTerminationFilter(ChangeLogFilterMixin):
     cable: Annotated['CableFilter', strawberry.lazy('dcim.graphql.filters')] | None = strawberry_django.filter_field()
     cable_id: ID | None = strawberry_django.filter_field()
     cable_end: BaseFilterLookup[Annotated['CableEndEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
@@ -130,7 +132,7 @@ class CableTerminationFilter(ChangeLoggedModelFilter):
 
 
 @strawberry_django.filter_type(models.ConsolePort, lookups=True)
-class ConsolePortFilter(ModularComponentFilterMixin, CabledObjectModelFilterMixin, NetBoxModelFilter):
+class ConsolePortFilter(ModularComponentModelFilterMixin, CabledObjectModelFilterMixin):
     type: BaseFilterLookup[Annotated['ConsolePortTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -140,14 +142,14 @@ class ConsolePortFilter(ModularComponentFilterMixin, CabledObjectModelFilterMixi
 
 
 @strawberry_django.filter_type(models.ConsolePortTemplate, lookups=True)
-class ConsolePortTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLoggedModelFilter):
+class ConsolePortTemplateFilter(ModularComponentTemplateFilterMixin):
     type: BaseFilterLookup[Annotated['ConsolePortTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
 
 
 @strawberry_django.filter_type(models.ConsoleServerPort, lookups=True)
-class ConsoleServerPortFilter(ModularComponentFilterMixin, CabledObjectModelFilterMixin, NetBoxModelFilter):
+class ConsoleServerPortFilter(ModularComponentModelFilterMixin, CabledObjectModelFilterMixin):
     type: BaseFilterLookup[Annotated['ConsolePortTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -157,7 +159,7 @@ class ConsoleServerPortFilter(ModularComponentFilterMixin, CabledObjectModelFilt
 
 
 @strawberry_django.filter_type(models.ConsoleServerPortTemplate, lookups=True)
-class ConsoleServerPortTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLoggedModelFilter):
+class ConsoleServerPortTemplateFilter(ModularComponentTemplateFilterMixin):
     type: BaseFilterLookup[Annotated['ConsolePortTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -170,7 +172,7 @@ class DeviceFilter(
     ImageAttachmentFilterMixin,
     RenderConfigFilterMixin,
     ConfigContextFilterMixin,
-    PrimaryModelFilter,
+    PrimaryModelFilterMixin,
 ):
     device_type: Annotated['DeviceTypeFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
@@ -283,7 +285,7 @@ class DeviceFilter(
 
 
 @strawberry_django.filter_type(models.DeviceBay, lookups=True)
-class DeviceBayFilter(ComponentModelFilterMixin, NetBoxModelFilter):
+class DeviceBayFilter(ComponentModelFilterMixin):
     installed_device: Annotated['DeviceFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -291,12 +293,12 @@ class DeviceBayFilter(ComponentModelFilterMixin, NetBoxModelFilter):
 
 
 @strawberry_django.filter_type(models.DeviceBayTemplate, lookups=True)
-class DeviceBayTemplateFilter(ComponentTemplateFilterMixin, ChangeLoggedModelFilter):
+class DeviceBayTemplateFilter(ComponentTemplateFilterMixin):
     pass
 
 
 @strawberry_django.filter_type(models.InventoryItemTemplate, lookups=True)
-class InventoryItemTemplateFilter(ComponentTemplateFilterMixin, ChangeLoggedModelFilter):
+class InventoryItemTemplateFilter(ComponentTemplateFilterMixin):
     parent: Annotated['InventoryItemTemplateFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -316,7 +318,7 @@ class InventoryItemTemplateFilter(ComponentTemplateFilterMixin, ChangeLoggedMode
 
 
 @strawberry_django.filter_type(models.DeviceRole, lookups=True)
-class DeviceRoleFilter(RenderConfigFilterMixin, OrganizationalModelFilter):
+class DeviceRoleFilter(OrganizationalModelFilterMixin, RenderConfigFilterMixin):
     color: BaseFilterLookup[Annotated['ColorEnum', strawberry.lazy('netbox.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -324,7 +326,7 @@ class DeviceRoleFilter(RenderConfigFilterMixin, OrganizationalModelFilter):
 
 
 @strawberry_django.filter_type(models.DeviceType, lookups=True)
-class DeviceTypeFilter(ImageAttachmentFilterMixin, WeightFilterMixin, PrimaryModelFilter):
+class DeviceTypeFilter(ImageAttachmentFilterMixin, PrimaryModelFilterMixin, WeightFilterMixin):
     manufacturer: Annotated['ManufacturerFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -400,58 +402,41 @@ class DeviceTypeFilter(ImageAttachmentFilterMixin, WeightFilterMixin, PrimaryMod
 
 
 @strawberry_django.filter_type(models.FrontPort, lookups=True)
-class FrontPortFilter(ModularComponentFilterMixin, CabledObjectModelFilterMixin, NetBoxModelFilter):
+class FrontPortFilter(ModularComponentModelFilterMixin, CabledObjectModelFilterMixin):
     type: BaseFilterLookup[Annotated['PortTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
     color: BaseFilterLookup[Annotated['ColorEnum', strawberry.lazy('netbox.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
-
-
-@strawberry_django.filter_type(models.FrontPortTemplate, lookups=True)
-class FrontPortTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLoggedModelFilter):
-    type: BaseFilterLookup[Annotated['PortTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
-        strawberry_django.filter_field()
-    )
-    color: BaseFilterLookup[Annotated['ColorEnum', strawberry.lazy('netbox.graphql.enums')]] | None = (
-        strawberry_django.filter_field()
-    )
-
-
-@strawberry_django.filter_type(models.PortMapping, lookups=True)
-class PortMappingFilter(BaseModelFilter):
-    device: Annotated['DeviceFilter', strawberry.lazy('dcim.graphql.filters')] | None = strawberry_django.filter_field()
-    front_port: Annotated['FrontPortFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
-        strawberry_django.filter_field()
-    )
     rear_port: Annotated['RearPortFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
-    front_port_position: FilterLookup[int] | None = strawberry_django.filter_field()
-    rear_port_position: FilterLookup[int] | None = strawberry_django.filter_field()
-
-
-@strawberry_django.filter_type(models.PortTemplateMapping, lookups=True)
-class PortTemplateMappingFilter(BaseModelFilter):
-    device_type: Annotated['DeviceTypeFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
+    rear_port_id: ID | None = strawberry_django.filter_field()
+    rear_port_position: Annotated['IntegerLookup', strawberry.lazy('netbox.graphql.filter_lookups')] | None = (
         strawberry_django.filter_field()
     )
-    module_type: Annotated['ModuleTypeFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
+
+
+@strawberry_django.filter_type(models.FrontPortTemplate, lookups=True)
+class FrontPortTemplateFilter(ModularComponentTemplateFilterMixin):
+    type: BaseFilterLookup[Annotated['PortTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
-    front_port: Annotated['FrontPortTemplateFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
+    color: BaseFilterLookup[Annotated['ColorEnum', strawberry.lazy('netbox.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
     rear_port: Annotated['RearPortTemplateFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
-    front_port_position: FilterLookup[int] | None = strawberry_django.filter_field()
-    rear_port_position: FilterLookup[int] | None = strawberry_django.filter_field()
+    rear_port_id: ID | None = strawberry_django.filter_field()
+    rear_port_position: Annotated['IntegerLookup', strawberry.lazy('netbox.graphql.filter_lookups')] | None = (
+        strawberry_django.filter_field()
+    )
 
 
 @strawberry_django.filter_type(models.MACAddress, lookups=True)
-class MACAddressFilter(PrimaryModelFilter):
+class MACAddressFilter(PrimaryModelFilterMixin):
     mac_address: FilterLookup[str] | None = strawberry_django.filter_field()
     assigned_object_type: Annotated['ContentTypeFilter', strawberry.lazy('core.graphql.filters')] | None = (
         strawberry_django.filter_field()
@@ -478,12 +463,7 @@ class MACAddressFilter(PrimaryModelFilter):
 
 
 @strawberry_django.filter_type(models.Interface, lookups=True)
-class InterfaceFilter(
-    ModularComponentFilterMixin,
-    InterfaceBaseFilterMixin,
-    CabledObjectModelFilterMixin,
-    NetBoxModelFilter
-):
+class InterfaceFilter(ModularComponentModelFilterMixin, InterfaceBaseFilterMixin, CabledObjectModelFilterMixin):
     vcdcs: Annotated['VirtualDeviceContextFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -573,7 +553,7 @@ class InterfaceFilter(
 
 
 @strawberry_django.filter_type(models.InterfaceTemplate, lookups=True)
-class InterfaceTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLoggedModelFilter):
+class InterfaceTemplateFilter(ModularComponentTemplateFilterMixin):
     type: BaseFilterLookup[Annotated['InterfaceTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -595,7 +575,7 @@ class InterfaceTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLoggedM
 
 
 @strawberry_django.filter_type(models.InventoryItem, lookups=True)
-class InventoryItemFilter(ComponentModelFilterMixin, NetBoxModelFilter):
+class InventoryItemFilter(ComponentModelFilterMixin):
     parent: Annotated['InventoryItemFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -622,14 +602,14 @@ class InventoryItemFilter(ComponentModelFilterMixin, NetBoxModelFilter):
 
 
 @strawberry_django.filter_type(models.InventoryItemRole, lookups=True)
-class InventoryItemRoleFilter(OrganizationalModelFilter):
+class InventoryItemRoleFilter(OrganizationalModelFilterMixin):
     color: BaseFilterLookup[Annotated['ColorEnum', strawberry.lazy('netbox.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
 
 
 @strawberry_django.filter_type(models.Location, lookups=True)
-class LocationFilter(ContactFilterMixin, ImageAttachmentFilterMixin, TenancyFilterMixin, NestedGroupModelFilter):
+class LocationFilter(ContactFilterMixin, ImageAttachmentFilterMixin, TenancyFilterMixin, NestedGroupModelFilterMixin):
     site: Annotated['SiteFilter', strawberry.lazy('dcim.graphql.filters')] | None = strawberry_django.filter_field()
     site_id: ID | None = strawberry_django.filter_field()
     status: BaseFilterLookup[Annotated['LocationStatusEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
@@ -645,12 +625,12 @@ class LocationFilter(ContactFilterMixin, ImageAttachmentFilterMixin, TenancyFilt
 
 
 @strawberry_django.filter_type(models.Manufacturer, lookups=True)
-class ManufacturerFilter(ContactFilterMixin, OrganizationalModelFilter):
+class ManufacturerFilter(ContactFilterMixin, OrganizationalModelFilterMixin):
     pass
 
 
 @strawberry_django.filter_type(models.Module, lookups=True)
-class ModuleFilter(ConfigContextFilterMixin, PrimaryModelFilter):
+class ModuleFilter(PrimaryModelFilterMixin, ConfigContextFilterMixin):
     device: Annotated['DeviceFilter', strawberry.lazy('dcim.graphql.filters')] | None = strawberry_django.filter_field()
     device_id: ID | None = strawberry_django.filter_field()
     module_bay: Annotated['ModuleBayFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
@@ -699,7 +679,7 @@ class ModuleFilter(ConfigContextFilterMixin, PrimaryModelFilter):
 
 
 @strawberry_django.filter_type(models.ModuleBay, lookups=True)
-class ModuleBayFilter(ModularComponentFilterMixin, NetBoxModelFilter):
+class ModuleBayFilter(ModularComponentModelFilterMixin):
     parent: Annotated['ModuleBayFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -708,17 +688,17 @@ class ModuleBayFilter(ModularComponentFilterMixin, NetBoxModelFilter):
 
 
 @strawberry_django.filter_type(models.ModuleBayTemplate, lookups=True)
-class ModuleBayTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLoggedModelFilter):
+class ModuleBayTemplateFilter(ModularComponentTemplateFilterMixin):
     position: FilterLookup[str] | None = strawberry_django.filter_field()
 
 
 @strawberry_django.filter_type(models.ModuleTypeProfile, lookups=True)
-class ModuleTypeProfileFilter(PrimaryModelFilter):
+class ModuleTypeProfileFilter(PrimaryModelFilterMixin):
     name: FilterLookup[str] | None = strawberry_django.filter_field()
 
 
 @strawberry_django.filter_type(models.ModuleType, lookups=True)
-class ModuleTypeFilter(ImageAttachmentFilterMixin, WeightFilterMixin, PrimaryModelFilter):
+class ModuleTypeFilter(ImageAttachmentFilterMixin, PrimaryModelFilterMixin, WeightFilterMixin):
     manufacturer: Annotated['ManufacturerFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -769,7 +749,7 @@ class ModuleTypeFilter(ImageAttachmentFilterMixin, WeightFilterMixin, PrimaryMod
 
 
 @strawberry_django.filter_type(models.Platform, lookups=True)
-class PlatformFilter(OrganizationalModelFilter):
+class PlatformFilter(OrganizationalModelFilterMixin):
     manufacturer: Annotated['ManufacturerFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -781,7 +761,7 @@ class PlatformFilter(OrganizationalModelFilter):
 
 
 @strawberry_django.filter_type(models.PowerFeed, lookups=True)
-class PowerFeedFilter(CabledObjectModelFilterMixin, TenancyFilterMixin, PrimaryModelFilter):
+class PowerFeedFilter(CabledObjectModelFilterMixin, TenancyFilterMixin, PrimaryModelFilterMixin):
     power_panel: Annotated['PowerPanelFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -816,7 +796,7 @@ class PowerFeedFilter(CabledObjectModelFilterMixin, TenancyFilterMixin, PrimaryM
 
 
 @strawberry_django.filter_type(models.PowerOutlet, lookups=True)
-class PowerOutletFilter(ModularComponentFilterMixin, CabledObjectModelFilterMixin, NetBoxModelFilter):
+class PowerOutletFilter(ModularComponentModelFilterMixin, CabledObjectModelFilterMixin):
     type: BaseFilterLookup[Annotated['PowerOutletTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -836,7 +816,7 @@ class PowerOutletFilter(ModularComponentFilterMixin, CabledObjectModelFilterMixi
 
 
 @strawberry_django.filter_type(models.PowerOutletTemplate, lookups=True)
-class PowerOutletTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLoggedModelFilter):
+class PowerOutletTemplateFilter(ModularComponentModelFilterMixin):
     type: BaseFilterLookup[Annotated['PowerOutletTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -850,7 +830,7 @@ class PowerOutletTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLogge
 
 
 @strawberry_django.filter_type(models.PowerPanel, lookups=True)
-class PowerPanelFilter(ContactFilterMixin, ImageAttachmentFilterMixin, PrimaryModelFilter):
+class PowerPanelFilter(ContactFilterMixin, ImageAttachmentFilterMixin, PrimaryModelFilterMixin):
     site: Annotated['SiteFilter', strawberry.lazy('dcim.graphql.filters')] | None = strawberry_django.filter_field()
     site_id: ID | None = strawberry_django.filter_field()
     location: Annotated['LocationFilter', strawberry.lazy('dcim.graphql.filters')] | None = (
@@ -863,7 +843,7 @@ class PowerPanelFilter(ContactFilterMixin, ImageAttachmentFilterMixin, PrimaryMo
 
 
 @strawberry_django.filter_type(models.PowerPort, lookups=True)
-class PowerPortFilter(ModularComponentFilterMixin, CabledObjectModelFilterMixin, NetBoxModelFilter):
+class PowerPortFilter(ModularComponentModelFilterMixin, CabledObjectModelFilterMixin):
     type: BaseFilterLookup[Annotated['PowerPortTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -876,7 +856,7 @@ class PowerPortFilter(ModularComponentFilterMixin, CabledObjectModelFilterMixin,
 
 
 @strawberry_django.filter_type(models.PowerPortTemplate, lookups=True)
-class PowerPortTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLoggedModelFilter):
+class PowerPortTemplateFilter(ModularComponentTemplateFilterMixin):
     type: BaseFilterLookup[Annotated['PowerPortTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -889,7 +869,7 @@ class PowerPortTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLoggedM
 
 
 @strawberry_django.filter_type(models.RackType, lookups=True)
-class RackTypeFilter(RackFilterMixin, WeightFilterMixin, PrimaryModelFilter):
+class RackTypeFilter(RackBaseFilterMixin):
     form_factor: BaseFilterLookup[Annotated['RackFormFactorEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -904,14 +884,7 @@ class RackTypeFilter(RackFilterMixin, WeightFilterMixin, PrimaryModelFilter):
 
 
 @strawberry_django.filter_type(models.Rack, lookups=True)
-class RackFilter(
-    ContactFilterMixin,
-    ImageAttachmentFilterMixin,
-    TenancyFilterMixin,
-    WeightFilterMixin,
-    RackFilterMixin,
-    PrimaryModelFilter
-):
+class RackFilter(ContactFilterMixin, ImageAttachmentFilterMixin, TenancyFilterMixin, RackBaseFilterMixin):
     form_factor: BaseFilterLookup[Annotated['RackFormFactorEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -945,7 +918,7 @@ class RackFilter(
 
 
 @strawberry_django.filter_type(models.RackReservation, lookups=True)
-class RackReservationFilter(TenancyFilterMixin, PrimaryModelFilter):
+class RackReservationFilter(TenancyFilterMixin, PrimaryModelFilterMixin):
     rack: Annotated['RackFilter', strawberry.lazy('dcim.graphql.filters')] | None = strawberry_django.filter_field()
     rack_id: ID | None = strawberry_django.filter_field()
     units: Annotated['IntegerArrayLookup', strawberry.lazy('netbox.graphql.filter_lookups')] | None = (
@@ -960,14 +933,14 @@ class RackReservationFilter(TenancyFilterMixin, PrimaryModelFilter):
 
 
 @strawberry_django.filter_type(models.RackRole, lookups=True)
-class RackRoleFilter(OrganizationalModelFilter):
+class RackRoleFilter(OrganizationalModelFilterMixin):
     color: BaseFilterLookup[Annotated['ColorEnum', strawberry.lazy('netbox.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
 
 
 @strawberry_django.filter_type(models.RearPort, lookups=True)
-class RearPortFilter(ModularComponentFilterMixin, CabledObjectModelFilterMixin, NetBoxModelFilter):
+class RearPortFilter(ModularComponentModelFilterMixin, CabledObjectModelFilterMixin):
     type: BaseFilterLookup[Annotated['PortTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -980,7 +953,7 @@ class RearPortFilter(ModularComponentFilterMixin, CabledObjectModelFilterMixin,
 
 
 @strawberry_django.filter_type(models.RearPortTemplate, lookups=True)
-class RearPortTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLoggedModelFilter):
+class RearPortTemplateFilter(ModularComponentTemplateFilterMixin):
     type: BaseFilterLookup[Annotated['PortTypeEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
         strawberry_django.filter_field()
     )
@@ -993,7 +966,7 @@ class RearPortTemplateFilter(ModularComponentTemplateFilterMixin, ChangeLoggedMo
 
 
 @strawberry_django.filter_type(models.Region, lookups=True)
-class RegionFilter(ContactFilterMixin, NestedGroupModelFilter):
+class RegionFilter(ContactFilterMixin, NestedGroupModelFilterMixin):
     prefixes: Annotated['PrefixFilter', strawberry.lazy('ipam.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -1003,7 +976,7 @@ class RegionFilter(ContactFilterMixin, NestedGroupModelFilter):
 
 
 @strawberry_django.filter_type(models.Site, lookups=True)
-class SiteFilter(ContactFilterMixin, ImageAttachmentFilterMixin, TenancyFilterMixin, PrimaryModelFilter):
+class SiteFilter(ContactFilterMixin, ImageAttachmentFilterMixin, TenancyFilterMixin, PrimaryModelFilterMixin):
     name: FilterLookup[str] | None = strawberry_django.filter_field()
     slug: FilterLookup[str] | None = strawberry_django.filter_field()
     status: BaseFilterLookup[Annotated['SiteStatusEnum', strawberry.lazy('dcim.graphql.enums')]] | None = (
@@ -1039,7 +1012,7 @@ class SiteFilter(ContactFilterMixin, ImageAttachmentFilterMixin, TenancyFilterMi
 
 
 @strawberry_django.filter_type(models.SiteGroup, lookups=True)
-class SiteGroupFilter(ContactFilterMixin, NestedGroupModelFilter):
+class SiteGroupFilter(ContactFilterMixin, NestedGroupModelFilterMixin):
     prefixes: Annotated['PrefixFilter', strawberry.lazy('ipam.graphql.filters')] | None = (
         strawberry_django.filter_field()
     )
@@ -1049,7 +1022,7 @@ class SiteGroupFilter(ContactFilterMixin, NestedGroupModelFilter):
 
 
 @strawberry_django.filter_type(models.VirtualChassis, lookups=True)
-class VirtualChassisFilter(PrimaryModelFilter):
+class VirtualChassisFilter(PrimaryModelFilterMixin):
     master: Annotated['DeviceFilter', strawberry.lazy('dcim.graphql.filters')] | None = strawberry_django.filter_field()
     master_id: ID | None = strawberry_django.filter_field()
     name: FilterLookup[str] | None = strawberry_django.filter_field()
@@ -1061,7 +1034,7 @@ class VirtualChassisFilter(PrimaryModelFilter):
 
 
 @strawberry_django.filter_type(models.VirtualDeviceContext, lookups=True)
-class VirtualDeviceContextFilter(TenancyFilterMixin, PrimaryModelFilter):
+class VirtualDeviceContextFilter(TenancyFilterMixin, PrimaryModelFilterMixin):
     device: Annotated['DeviceFilter', strawberry.lazy('dcim.graphql.filters')] | None = strawberry_django.filter_field()
     device_id: ID | None = strawberry_django.filter_field()
     name: FilterLookup[str] | None = strawberry_django.filter_field()

netbox/dcim/graphql/types.py

@@ -385,8 +385,7 @@ class DeviceTypeType(PrimaryObjectType):
 )
 class FrontPortType(ModularComponentType, CabledObjectMixin):
     color: str
-
-    mappings: List[Annotated["PortMappingType", strawberry.lazy('dcim.graphql.types')]]
+    rear_port: Annotated["RearPortType", strawberry.lazy('dcim.graphql.types')]
 
 
 @strawberry_django.type(
@@ -397,8 +396,7 @@ class FrontPortType(ModularComponentType, CabledObjectMixin):
 )
 class FrontPortTemplateType(ModularComponentTemplateType):
     color: str
-
-    mappings: List[Annotated["PortMappingTemplateType", strawberry.lazy('dcim.graphql.types')]]
+    rear_port: Annotated["RearPortTemplateType", strawberry.lazy('dcim.graphql.types')]
 
 
 @strawberry_django.type(
@@ -638,28 +636,6 @@ class PlatformType(NestedGroupObjectType):
     devices: List[Annotated["DeviceType", strawberry.lazy('dcim.graphql.types')]]
 
 
-@strawberry_django.type(
-    models.PortMapping,
-    fields='__all__',
-    filters=PortMappingFilter,
-    pagination=True
-)
-class PortMappingType(ModularComponentTemplateType):
-    front_port: Annotated["FrontPortType", strawberry.lazy('dcim.graphql.types')]
-    rear_port: Annotated["RearPortType", strawberry.lazy('dcim.graphql.types')]
-
-
-@strawberry_django.type(
-    models.PortTemplateMapping,
-    fields='__all__',
-    filters=PortTemplateMappingFilter,
-    pagination=True
-)
-class PortMappingTemplateType(ModularComponentTemplateType):
-    front_port: Annotated["FrontPortTemplateType", strawberry.lazy('dcim.graphql.types')]
-    rear_port: Annotated["RearPortTemplateType", strawberry.lazy('dcim.graphql.types')]
-
-
 @strawberry_django.type(
     models.PowerFeed,
     exclude=['_path'],
@@ -792,7 +768,7 @@ class RackRoleType(OrganizationalObjectType):
 class RearPortType(ModularComponentType, CabledObjectMixin):
     color: str
 
-    mappings: List[Annotated["PortMappingType", strawberry.lazy('dcim.graphql.types')]]
+    frontports: List[Annotated["FrontPortType", strawberry.lazy('dcim.graphql.types')]]
 
 
 @strawberry_django.type(
@@ -804,7 +780,7 @@ class RearPortType(ModularComponentType, CabledObjectMixin):
 class RearPortTemplateType(ModularComponentTemplateType):
     color: str
 
-    mappings: List[Annotated["PortMappingTemplateType", strawberry.lazy('dcim.graphql.types')]]
+    frontport_templates: List[Annotated["FrontPortTemplateType", strawberry.lazy('dcim.graphql.types')]]
 
 
 @strawberry_django.type(

netbox/dcim/migrations/0216_latitude_longitude_validators.py

@@ -1,5 +1,3 @@
-import decimal
-
 import django.core.validators
 from django.db import migrations, models
 
@@ -19,8 +17,8 @@ class Migration(migrations.Migration):
                 max_digits=8,
                 null=True,
                 validators=[
-                    django.core.validators.MinValueValidator(decimal.Decimal('-90.0')),
-                    django.core.validators.MaxValueValidator(decimal.Decimal('90.0'))
+                    django.core.validators.MinValueValidator(-90.0),
+                    django.core.validators.MaxValueValidator(90.0),
                 ],
             ),
         ),
@@ -33,8 +31,8 @@ class Migration(migrations.Migration):
                 max_digits=9,
                 null=True,
                 validators=[
-                    django.core.validators.MinValueValidator(decimal.Decimal('-180.0')),
-                    django.core.validators.MaxValueValidator(decimal.Decimal('180.0'))
+                    django.core.validators.MinValueValidator(-180.0),
+                    django.core.validators.MaxValueValidator(180.0),
                 ],
             ),
         ),
@@ -47,8 +45,8 @@ class Migration(migrations.Migration):
                 max_digits=8,
                 null=True,
                 validators=[
-                    django.core.validators.MinValueValidator(decimal.Decimal('-90.0')),
-                    django.core.validators.MaxValueValidator(decimal.Decimal('90.0'))
+                    django.core.validators.MinValueValidator(-90.0),
+                    django.core.validators.MaxValueValidator(90.0),
                 ],
             ),
         ),
@@ -61,8 +59,8 @@ class Migration(migrations.Migration):
                 max_digits=9,
                 null=True,
                 validators=[
-                    django.core.validators.MinValueValidator(decimal.Decimal('-180.0')),
-                    django.core.validators.MaxValueValidator(decimal.Decimal('180.0'))
+                    django.core.validators.MinValueValidator(-180.0),
+                    django.core.validators.MaxValueValidator(180.0),
                 ],
             ),
         ),

netbox/dcim/migrations/0220_cable_profile.py

@@ -1,4 +1,3 @@
-import django.contrib.postgres.fields
 import django.core.validators
 from django.db import migrations, models
 
@@ -17,40 +16,25 @@ class Migration(migrations.Migration):
         ),
         migrations.AddField(
             model_name='cabletermination',
-            name='connector',
-            field=models.PositiveSmallIntegerField(
+            name='position',
+            field=models.PositiveIntegerField(
                 blank=True,
                 null=True,
                 validators=[
                     django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(256)
-                ]
-            ),
-        ),
-        migrations.AddField(
-            model_name='cabletermination',
-            name='positions',
-            field=django.contrib.postgres.fields.ArrayField(
-                base_field=models.PositiveSmallIntegerField(
-                    validators=[
-                        django.core.validators.MinValueValidator(1),
-                        django.core.validators.MaxValueValidator(1024)
-                    ]
-                ),
-                blank=True,
-                null=True,
-                size=None
+                    django.core.validators.MaxValueValidator(1024),
+                ],
             ),
         ),
         migrations.AlterModelOptions(
             name='cabletermination',
-            options={'ordering': ('cable', 'cable_end', 'connector', 'pk')},  # connector may be null
+            options={'ordering': ('cable', 'cable_end', 'position', 'pk')},
         ),
         migrations.AddConstraint(
             model_name='cabletermination',
             constraint=models.UniqueConstraint(
-                fields=('cable', 'cable_end', 'connector'),
-                name='dcim_cabletermination_unique_connector'
+                fields=('cable', 'cable_end', 'position'),
+                name='dcim_cabletermination_unique_position'
             ),
         ),
     ]

netbox/dcim/migrations/0221_cable_connector_positions.py

@@ -1,228 +0,0 @@
-import django.contrib.postgres.fields
-import django.core.validators
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-    dependencies = [
-        ('dcim', '0220_cable_profile'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='consoleport',
-            name='cable_connector',
-            field=models.PositiveSmallIntegerField(
-                blank=True,
-                null=True,
-                validators=[
-                    django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(256)
-                ],
-            ),
-        ),
-        migrations.AddField(
-            model_name='consoleport',
-            name='cable_positions',
-            field=django.contrib.postgres.fields.ArrayField(
-                base_field=models.PositiveSmallIntegerField(
-                    validators=[
-                        django.core.validators.MinValueValidator(1),
-                        django.core.validators.MaxValueValidator(1024),
-                    ]
-                ),
-                blank=True,
-                null=True,
-                size=None,
-            ),
-        ),
-        migrations.AddField(
-            model_name='consoleserverport',
-            name='cable_connector',
-            field=models.PositiveSmallIntegerField(
-                blank=True,
-                null=True,
-                validators=[
-                    django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(256)
-                ],
-            ),
-        ),
-        migrations.AddField(
-            model_name='consoleserverport',
-            name='cable_positions',
-            field=django.contrib.postgres.fields.ArrayField(
-                base_field=models.PositiveSmallIntegerField(
-                    validators=[
-                        django.core.validators.MinValueValidator(1),
-                        django.core.validators.MaxValueValidator(1024),
-                    ]
-                ),
-                blank=True,
-                null=True,
-                size=None,
-            ),
-        ),
-        migrations.AddField(
-            model_name='frontport',
-            name='cable_connector',
-            field=models.PositiveSmallIntegerField(
-                blank=True,
-                null=True,
-                validators=[
-                    django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(256)
-                ],
-            ),
-        ),
-        migrations.AddField(
-            model_name='frontport',
-            name='cable_positions',
-            field=django.contrib.postgres.fields.ArrayField(
-                base_field=models.PositiveSmallIntegerField(
-                    validators=[
-                        django.core.validators.MinValueValidator(1),
-                        django.core.validators.MaxValueValidator(1024),
-                    ]
-                ),
-                blank=True,
-                null=True,
-                size=None,
-            ),
-        ),
-        migrations.AddField(
-            model_name='interface',
-            name='cable_connector',
-            field=models.PositiveSmallIntegerField(
-                blank=True,
-                null=True,
-                validators=[
-                    django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(256)
-                ],
-            ),
-        ),
-        migrations.AddField(
-            model_name='interface',
-            name='cable_positions',
-            field=django.contrib.postgres.fields.ArrayField(
-                base_field=models.PositiveSmallIntegerField(
-                    validators=[
-                        django.core.validators.MinValueValidator(1),
-                        django.core.validators.MaxValueValidator(1024),
-                    ]
-                ),
-                blank=True,
-                null=True,
-                size=None,
-            ),
-        ),
-        migrations.AddField(
-            model_name='powerfeed',
-            name='cable_connector',
-            field=models.PositiveSmallIntegerField(
-                blank=True,
-                null=True,
-                validators=[
-                    django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(256)
-                ],
-            ),
-        ),
-        migrations.AddField(
-            model_name='powerfeed',
-            name='cable_positions',
-            field=django.contrib.postgres.fields.ArrayField(
-                base_field=models.PositiveSmallIntegerField(
-                    validators=[
-                        django.core.validators.MinValueValidator(1),
-                        django.core.validators.MaxValueValidator(1024),
-                    ]
-                ),
-                blank=True,
-                null=True,
-                size=None,
-            ),
-        ),
-        migrations.AddField(
-            model_name='poweroutlet',
-            name='cable_connector',
-            field=models.PositiveSmallIntegerField(
-                blank=True,
-                null=True,
-                validators=[
-                    django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(256)
-                ],
-            ),
-        ),
-        migrations.AddField(
-            model_name='poweroutlet',
-            name='cable_positions',
-            field=django.contrib.postgres.fields.ArrayField(
-                base_field=models.PositiveSmallIntegerField(
-                    validators=[
-                        django.core.validators.MinValueValidator(1),
-                        django.core.validators.MaxValueValidator(1024),
-                    ]
-                ),
-                blank=True,
-                null=True,
-                size=None,
-            ),
-        ),
-        migrations.AddField(
-            model_name='powerport',
-            name='cable_connector',
-            field=models.PositiveSmallIntegerField(
-                blank=True,
-                null=True,
-                validators=[
-                    django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(256)
-                ],
-            ),
-        ),
-        migrations.AddField(
-            model_name='powerport',
-            name='cable_positions',
-            field=django.contrib.postgres.fields.ArrayField(
-                base_field=models.PositiveSmallIntegerField(
-                    validators=[
-                        django.core.validators.MinValueValidator(1),
-                        django.core.validators.MaxValueValidator(1024),
-                    ]
-                ),
-                blank=True,
-                null=True,
-                size=None,
-            ),
-        ),
-        migrations.AddField(
-            model_name='rearport',
-            name='cable_connector',
-            field=models.PositiveSmallIntegerField(
-                blank=True,
-                null=True,
-                validators=[
-                    django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(256)
-                ],
-            ),
-        ),
-        migrations.AddField(
-            model_name='rearport',
-            name='cable_positions',
-            field=django.contrib.postgres.fields.ArrayField(
-                base_field=models.PositiveSmallIntegerField(
-                    validators=[
-                        django.core.validators.MinValueValidator(1),
-                        django.core.validators.MaxValueValidator(1024),
-                    ]
-                ),
-                blank=True,
-                null=True,
-                size=None,
-            ),
-        ),
-    ]

netbox/dcim/migrations/0221_cable_position.py

@@ -0,0 +1,107 @@
+import django.core.validators
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ('dcim', '0220_cable_profile'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='consoleport',
+            name='cable_position',
+            field=models.PositiveIntegerField(
+                blank=True,
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(1),
+                    django.core.validators.MaxValueValidator(1024),
+                ],
+            ),
+        ),
+        migrations.AddField(
+            model_name='consoleserverport',
+            name='cable_position',
+            field=models.PositiveIntegerField(
+                blank=True,
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(1),
+                    django.core.validators.MaxValueValidator(1024),
+                ],
+            ),
+        ),
+        migrations.AddField(
+            model_name='frontport',
+            name='cable_position',
+            field=models.PositiveIntegerField(
+                blank=True,
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(1),
+                    django.core.validators.MaxValueValidator(1024),
+                ],
+            ),
+        ),
+        migrations.AddField(
+            model_name='interface',
+            name='cable_position',
+            field=models.PositiveIntegerField(
+                blank=True,
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(1),
+                    django.core.validators.MaxValueValidator(1024),
+                ],
+            ),
+        ),
+        migrations.AddField(
+            model_name='powerfeed',
+            name='cable_position',
+            field=models.PositiveIntegerField(
+                blank=True,
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(1),
+                    django.core.validators.MaxValueValidator(1024),
+                ],
+            ),
+        ),
+        migrations.AddField(
+            model_name='poweroutlet',
+            name='cable_position',
+            field=models.PositiveIntegerField(
+                blank=True,
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(1),
+                    django.core.validators.MaxValueValidator(1024),
+                ],
+            ),
+        ),
+        migrations.AddField(
+            model_name='powerport',
+            name='cable_position',
+            field=models.PositiveIntegerField(
+                blank=True,
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(1),
+                    django.core.validators.MaxValueValidator(1024),
+                ],
+            ),
+        ),
+        migrations.AddField(
+            model_name='rearport',
+            name='cable_position',
+            field=models.PositiveIntegerField(
+                blank=True,
+                null=True,
+                validators=[
+                    django.core.validators.MinValueValidator(1),
+                    django.core.validators.MaxValueValidator(1024),
+                ],
+            ),
+        ),
+    ]

netbox/dcim/migrations/0222_port_mappings.py

@@ -1,219 +0,0 @@
-import django.core.validators
-import django.db.models.deletion
-from django.db import migrations
-from django.db import models
-from itertools import islice
-
-
-def chunked(iterable, size):
-    """
-    Yield successive chunks of a given size from an iterator.
-    """
-    iterator = iter(iterable)
-    while chunk := list(islice(iterator, size)):
-        yield chunk
-
-
-def populate_port_template_mappings(apps, schema_editor):
-    FrontPortTemplate = apps.get_model('dcim', 'FrontPortTemplate')
-    PortTemplateMapping = apps.get_model('dcim', 'PortTemplateMapping')
-
-    front_ports = FrontPortTemplate.objects.iterator(chunk_size=1000)
-
-    def generate_copies():
-        for front_port in front_ports:
-            yield PortTemplateMapping(
-                device_type_id=front_port.device_type_id,
-                module_type_id=front_port.module_type_id,
-                front_port_id=front_port.pk,
-                front_port_position=1,
-                rear_port_id=front_port.rear_port_id,
-                rear_port_position=front_port.rear_port_position,
-            )
-
-    # Bulk insert in streaming batches
-    for chunk in chunked(generate_copies(), 1000):
-        PortTemplateMapping.objects.bulk_create(chunk, batch_size=1000)
-
-
-def populate_port_mappings(apps, schema_editor):
-    FrontPort = apps.get_model('dcim', 'FrontPort')
-    PortMapping = apps.get_model('dcim', 'PortMapping')
-
-    front_ports = FrontPort.objects.iterator(chunk_size=1000)
-
-    def generate_copies():
-        for front_port in front_ports:
-            yield PortMapping(
-                device_id=front_port.device_id,
-                front_port_id=front_port.pk,
-                front_port_position=1,
-                rear_port_id=front_port.rear_port_id,
-                rear_port_position=front_port.rear_port_position,
-            )
-
-    # Bulk insert in streaming batches
-    for chunk in chunked(generate_copies(), 1000):
-        PortMapping.objects.bulk_create(chunk, batch_size=1000)
-
-
-class Migration(migrations.Migration):
-    dependencies = [
-        ('dcim', '0221_cable_connector_positions'),
-    ]
-
-    operations = [
-        # Create PortTemplateMapping model (for DeviceTypes)
-        migrations.CreateModel(
-            name='PortTemplateMapping',
-            fields=[
-                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False)),
-                (
-                    'front_port_position',
-                    models.PositiveSmallIntegerField(
-                        default=1,
-                        validators=[
-                            django.core.validators.MinValueValidator(1),
-                            django.core.validators.MaxValueValidator(1024)
-                        ]
-                    )
-                ),
-                (
-                    'rear_port_position',
-                    models.PositiveSmallIntegerField(
-                        default=1,
-                        validators=[
-                            django.core.validators.MinValueValidator(1),
-                            django.core.validators.MaxValueValidator(1024)
-                        ]
-                    )
-                ),
-                (
-                    'device_type',
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        to='dcim.devicetype',
-                        related_name='port_mappings',
-                        blank=True,
-                        null=True
-                    )
-                ),
-                (
-                    'module_type',
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        to='dcim.moduletype',
-                        related_name='port_mappings',
-                        blank=True,
-                        null=True
-                    )
-                ),
-                (
-                    'front_port',
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        to='dcim.frontporttemplate',
-                        related_name='mappings'
-                    )
-                ),
-                (
-                    'rear_port',
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        to='dcim.rearporttemplate',
-                        related_name='mappings'
-                    )
-                ),
-            ],
-        ),
-        migrations.AddConstraint(
-            model_name='porttemplatemapping',
-            constraint=models.UniqueConstraint(
-                fields=('front_port', 'front_port_position'),
-                name='dcim_porttemplatemapping_unique_front_port_position'
-            ),
-        ),
-        migrations.AddConstraint(
-            model_name='porttemplatemapping',
-            constraint=models.UniqueConstraint(
-                fields=('rear_port', 'rear_port_position'),
-                name='dcim_porttemplatemapping_unique_rear_port_position'
-            ),
-        ),
-
-        # Create PortMapping model (for Devices)
-        migrations.CreateModel(
-            name='PortMapping',
-            fields=[
-                ('id', models.BigAutoField(auto_created=True, primary_key=True, serialize=False)),
-                (
-                    'front_port_position',
-                    models.PositiveSmallIntegerField(
-                        default=1,
-                        validators=[
-                            django.core.validators.MinValueValidator(1),
-                            django.core.validators.MaxValueValidator(1024)
-                        ]
-                    ),
-                ),
-                (
-                    'rear_port_position',
-                    models.PositiveSmallIntegerField(
-                        default=1,
-                        validators=[
-                            django.core.validators.MinValueValidator(1),
-                            django.core.validators.MaxValueValidator(1024),
-                        ]
-                    ),
-                ),
-                (
-                    'device',
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        to='dcim.device',
-                        related_name='port_mappings'
-                    )
-                ),
-                (
-                    'front_port',
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        to='dcim.frontport',
-                        related_name='mappings'
-                    )
-                ),
-                (
-                    'rear_port',
-                    models.ForeignKey(
-                        on_delete=django.db.models.deletion.CASCADE,
-                        to='dcim.rearport',
-                        related_name='mappings'
-                    )
-                ),
-            ],
-        ),
-        migrations.AddConstraint(
-            model_name='portmapping',
-            constraint=models.UniqueConstraint(
-                fields=('front_port', 'front_port_position'),
-                name='dcim_portmapping_unique_front_port_position'
-            ),
-        ),
-        migrations.AddConstraint(
-            model_name='portmapping',
-            constraint=models.UniqueConstraint(
-                fields=('rear_port', 'rear_port_position'),
-                name='dcim_portmapping_unique_rear_port_position'
-            ),
-        ),
-
-        # Data migration
-        migrations.RunPython(
-            code=populate_port_template_mappings,
-            reverse_code=migrations.RunPython.noop
-        ),
-        migrations.RunPython(
-            code=populate_port_mappings,
-            reverse_code=migrations.RunPython.noop
-        ),
-    ]

netbox/dcim/migrations/0223_frontport_positions.py

@@ -1,65 +0,0 @@
-import django.core.validators
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('dcim', '0222_port_mappings'),
-    ]
-
-    operations = [
-        # Remove rear_port & rear_port_position from FrontPortTemplate
-        migrations.RemoveConstraint(
-            model_name='frontporttemplate',
-            name='dcim_frontporttemplate_unique_rear_port_position',
-        ),
-        migrations.RemoveField(
-            model_name='frontporttemplate',
-            name='rear_port',
-        ),
-        migrations.RemoveField(
-            model_name='frontporttemplate',
-            name='rear_port_position',
-        ),
-
-        # Add positions on FrontPortTemplate
-        migrations.AddField(
-            model_name='frontporttemplate',
-            name='positions',
-            field=models.PositiveSmallIntegerField(
-                default=1,
-                validators=[
-                    django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(1024)
-                ]
-            ),
-        ),
-
-        # Remove rear_port & rear_port_position from FrontPort
-        migrations.RemoveConstraint(
-            model_name='frontport',
-            name='dcim_frontport_unique_rear_port_position',
-        ),
-        migrations.RemoveField(
-            model_name='frontport',
-            name='rear_port',
-        ),
-        migrations.RemoveField(
-            model_name='frontport',
-            name='rear_port_position',
-        ),
-
-        # Add positions on FrontPort
-        migrations.AddField(
-            model_name='frontport',
-            name='positions',
-            field=models.PositiveSmallIntegerField(
-                default=1,
-                validators=[
-                    django.core.validators.MinValueValidator(1),
-                    django.core.validators.MaxValueValidator(1024)
-                ]
-            ),
-        ),
-    ]

netbox/dcim/migrations/0224_add_comments_to_organizationalmodel.py

@@ -1,28 +0,0 @@
-# Generated by Django 5.2.8 on 2025-12-08 17:38
-
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-
-    dependencies = [
-        ('dcim', '0223_frontport_positions'),
-    ]
-
-    operations = [
-        migrations.AddField(
-            model_name='inventoryitemrole',
-            name='comments',
-            field=models.TextField(blank=True),
-        ),
-        migrations.AddField(
-            model_name='manufacturer',
-            name='comments',
-            field=models.TextField(blank=True),
-        ),
-        migrations.AddField(
-            model_name='rackrole',
-            name='comments',
-            field=models.TextField(blank=True),
-        ),
-    ]

netbox/dcim/migrations/0225_gfk_indexes.py

@@ -1,19 +0,0 @@
-from django.db import migrations, models
-
-
-class Migration(migrations.Migration):
-    dependencies = [
-        ('contenttypes', '0002_remove_content_type_name'),
-        ('dcim', '0224_add_comments_to_organizationalmodel'),
-        ('extras', '0134_owner'),
-        ('users', '0015_owner'),
-    ]
-
-    operations = [
-        migrations.AddIndex(
-            model_name='macaddress',
-            index=models.Index(
-                fields=['assigned_object_type', 'assigned_object_id'], name='dcim_macadd_assigne_54115d_idx'
-            ),
-        ),
-    ]

netbox/dcim/models/base.py

@@ -1,61 +0,0 @@
-from django.core.exceptions import ValidationError
-from django.core.validators import MaxValueValidator, MinValueValidator
-from django.db import models
-from django.utils.translation import gettext_lazy as _
-
-from dcim.constants import PORT_POSITION_MAX, PORT_POSITION_MIN
-
-__all__ = (
-    'PortMappingBase',
-)
-
-
-class PortMappingBase(models.Model):
-    """
-    Base class for PortMapping and PortTemplateMapping
-    """
-    front_port_position = models.PositiveSmallIntegerField(
-        default=1,
-        validators=(
-            MinValueValidator(PORT_POSITION_MIN),
-            MaxValueValidator(PORT_POSITION_MAX),
-        ),
-    )
-    rear_port_position = models.PositiveSmallIntegerField(
-        default=1,
-        validators=(
-            MinValueValidator(PORT_POSITION_MIN),
-            MaxValueValidator(PORT_POSITION_MAX),
-        ),
-    )
-
-    _netbox_private = True
-
-    class Meta:
-        abstract = True
-        constraints = (
-            models.UniqueConstraint(
-                fields=('front_port', 'front_port_position'),
-                name='%(app_label)s_%(class)s_unique_front_port_position'
-            ),
-            models.UniqueConstraint(
-                fields=('rear_port', 'rear_port_position'),
-                name='%(app_label)s_%(class)s_unique_rear_port_position'
-            ),
-        )
-
-    def clean(self):
-        super().clean()
-
-        # Validate rear port position
-        if self.rear_port_position > self.rear_port.positions:
-            raise ValidationError({
-                "rear_port_position": _(
-                    "Invalid rear port position ({rear_port_position}): Rear port {name} has only {positions} "
-                    "positions."
-                ).format(
-                    rear_port_position=self.rear_port_position,
-                    name=self.rear_port.name,
-                    positions=self.rear_port.positions
-                )
-            })

netbox/dcim/models/cables.py

@@ -1,9 +1,7 @@
 import itertools
-import logging
 
 from django.contrib.contenttypes.fields import GenericForeignKey
 from django.contrib.contenttypes.models import ContentType
-from django.contrib.postgres.fields import ArrayField
 from django.core.exceptions import ValidationError
 from django.core.validators import MaxValueValidator, MinValueValidator
 from django.db import models
@@ -24,7 +22,7 @@ from utilities.fields import ColorField, GenericArrayForeignKey
 from utilities.querysets import RestrictedQuerySet
 from utilities.serialization import deserialize_object, serialize_object
 from wireless.models import WirelessLink
-from .device_components import FrontPort, PathEndpoint, PortMapping, RearPort
+from .device_components import FrontPort, PathEndpoint, RearPort
 
 __all__ = (
     'Cable',
@@ -32,8 +30,6 @@ __all__ = (
     'CableTermination',
 )
 
-logger = logging.getLogger(f'netbox.{__name__}')
-
 trace_paths = Signal()
 
 
@@ -115,9 +111,8 @@ class Cable(PrimaryModel):
         # A copy of the PK to be used by __str__ in case the object is deleted
         self._pk = self.__dict__.get('id')
 
-        # Cache the original profile & status so we can check later whether either has been changed
+        # Cache the original status so we can check later if it's been changed
         self._orig_status = self.__dict__.get('status')
-        self._orig_profile = self.__dict__.get('profile')
 
         self._terminations_modified = False
 
@@ -138,30 +133,10 @@ class Cable(PrimaryModel):
     def profile_class(self):
         from dcim import cable_profiles
         return {
-            CableProfileChoices.SINGLE_1C1P: cable_profiles.Single1C1PCableProfile,
-            CableProfileChoices.SINGLE_1C2P: cable_profiles.Single1C2PCableProfile,
-            CableProfileChoices.SINGLE_1C4P: cable_profiles.Single1C4PCableProfile,
-            CableProfileChoices.SINGLE_1C6P: cable_profiles.Single1C6PCableProfile,
-            CableProfileChoices.SINGLE_1C8P: cable_profiles.Single1C8PCableProfile,
-            CableProfileChoices.SINGLE_1C12P: cable_profiles.Single1C12PCableProfile,
-            CableProfileChoices.SINGLE_1C16P: cable_profiles.Single1C16PCableProfile,
-            CableProfileChoices.TRUNK_2C1P: cable_profiles.Trunk2C1PCableProfile,
-            CableProfileChoices.TRUNK_2C2P: cable_profiles.Trunk2C2PCableProfile,
-            CableProfileChoices.TRUNK_2C4P: cable_profiles.Trunk2C4PCableProfile,
-            CableProfileChoices.TRUNK_2C4P_SHUFFLE: cable_profiles.Trunk2C4PShuffleCableProfile,
-            CableProfileChoices.TRUNK_2C6P: cable_profiles.Trunk2C6PCableProfile,
-            CableProfileChoices.TRUNK_2C8P: cable_profiles.Trunk2C8PCableProfile,
-            CableProfileChoices.TRUNK_2C12P: cable_profiles.Trunk2C12PCableProfile,
-            CableProfileChoices.TRUNK_4C1P: cable_profiles.Trunk4C1PCableProfile,
-            CableProfileChoices.TRUNK_4C2P: cable_profiles.Trunk4C2PCableProfile,
-            CableProfileChoices.TRUNK_4C4P: cable_profiles.Trunk4C4PCableProfile,
-            CableProfileChoices.TRUNK_4C4P_SHUFFLE: cable_profiles.Trunk4C4PShuffleCableProfile,
-            CableProfileChoices.TRUNK_4C6P: cable_profiles.Trunk4C6PCableProfile,
-            CableProfileChoices.TRUNK_4C8P: cable_profiles.Trunk4C8PCableProfile,
-            CableProfileChoices.TRUNK_8C4P: cable_profiles.Trunk8C4PCableProfile,
-            CableProfileChoices.BREAKOUT_1C4P_4C1P: cable_profiles.Breakout1C4Px4C1PCableProfile,
-            CableProfileChoices.BREAKOUT_1C6P_6C1P: cable_profiles.Breakout1C6Px6C1PCableProfile,
-            CableProfileChoices.BREAKOUT_2C4P_8C1P_SHUFFLE: cable_profiles.Breakout2C4Px8C1PShuffleCableProfile,
+            CableProfileChoices.STRAIGHT_SINGLE: cable_profiles.StraightSingleCableProfile,
+            CableProfileChoices.STRAIGHT_MULTI: cable_profiles.StraightMultiCableProfile,
+            CableProfileChoices.SHUFFLE_2X2_MPO8: cable_profiles.Shuffle2x2MPO8CableProfile,
+            CableProfileChoices.SHUFFLE_4X4_MPO8: cable_profiles.Shuffle4x4MPO8CableProfile,
         }.get(self.profile)
 
     def _get_x_terminations(self, side):
@@ -291,10 +266,7 @@ class Cable(PrimaryModel):
             # Update the private PK used in __str__()
             self._pk = self.pk
 
-        if self._orig_profile != self.profile:
-            print(f'profile changed from {self._orig_profile} to {self.profile}')
-            self.update_terminations(force=True)
-        elif self._terminations_modified:
+        if self._terminations_modified:
             self.update_terminations()
 
         super().save(*args, force_update=True, using=using, update_fields=update_fields)
@@ -348,52 +320,29 @@ class Cable(PrimaryModel):
 
         return a_terminations, b_terminations
 
-    def update_terminations(self, force=False):
+    def update_terminations(self):
         """
         Create/delete CableTerminations for this Cable to reflect its current state.
-
-        Args:
-            force: Force the recreation of all CableTerminations, even if no changes have been made. Needed e.g. when
-                altering a Cable's assigned profile.
         """
         a_terminations, b_terminations = self.get_terminations()
 
         # Delete any stale CableTerminations
         for termination, ct in a_terminations.items():
-            if force or (termination.pk and termination not in self.a_terminations):
+            if termination.pk and termination not in self.a_terminations:
                 ct.delete()
         for termination, ct in b_terminations.items():
-            if force or (termination.pk and termination not in self.b_terminations):
+            if termination.pk and termination not in self.b_terminations:
                 ct.delete()
 
         # Save any new CableTerminations
-        profile = self.profile_class() if self.profile else None
         for i, termination in enumerate(self.a_terminations, start=1):
-            if force or not termination.pk or termination not in a_terminations:
-                connector = positions = None
-                if profile:
-                    connector = i
-                    positions = profile.get_position_list(profile.a_connectors[i])
-                CableTermination(
-                    cable=self,
-                    cable_end=CableEndChoices.SIDE_A,
-                    connector=connector,
-                    positions=positions,
-                    termination=termination
-                ).save()
+            if not termination.pk or termination not in a_terminations:
+                position = i if self.profile and isinstance(termination, PathEndpoint) else None
+                CableTermination(cable=self, cable_end='A', position=position, termination=termination).save()
         for i, termination in enumerate(self.b_terminations, start=1):
-            if force or not termination.pk or termination not in b_terminations:
-                connector = positions = None
-                if profile:
-                    connector = i
-                    positions = profile.get_position_list(profile.b_connectors[i])
-                CableTermination(
-                    cable=self,
-                    cable_end=CableEndChoices.SIDE_B,
-                    connector=connector,
-                    positions=positions,
-                    termination=termination
-                ).save()
+            if not termination.pk or termination not in b_terminations:
+                position = i if self.profile and isinstance(termination, PathEndpoint) else None
+                CableTermination(cable=self, cable_end='B', position=position, termination=termination).save()
 
 
 class CableTermination(ChangeLoggedModel):
@@ -420,23 +369,13 @@ class CableTermination(ChangeLoggedModel):
         ct_field='termination_type',
         fk_field='termination_id'
     )
-    connector = models.PositiveSmallIntegerField(
+    position = models.PositiveIntegerField(
         blank=True,
         null=True,
         validators=(
-            MinValueValidator(CABLE_CONNECTOR_MIN),
-            MaxValueValidator(CABLE_CONNECTOR_MAX)
-        ),
-    )
-    positions = ArrayField(
-        base_field=models.PositiveSmallIntegerField(
-            validators=(
-                MinValueValidator(CABLE_POSITION_MIN),
-                MaxValueValidator(CABLE_POSITION_MAX)
-            )
-        ),
-        blank=True,
-        null=True,
+            MinValueValidator(CABLE_POSITION_MIN),
+            MaxValueValidator(CABLE_POSITION_MAX)
+        )
     )
 
     # Cached associations to enable efficient filtering
@@ -468,15 +407,15 @@ class CableTermination(ChangeLoggedModel):
     objects = RestrictedQuerySet.as_manager()
 
     class Meta:
-        ordering = ('cable', 'cable_end', 'connector', 'pk')
+        ordering = ('cable', 'cable_end', 'position', 'pk')
         constraints = (
             models.UniqueConstraint(
                 fields=('termination_type', 'termination_id'),
                 name='%(app_label)s_%(class)s_unique_termination'
             ),
             models.UniqueConstraint(
-                fields=('cable', 'cable_end', 'connector'),
-                name='%(app_label)s_%(class)s_unique_connector'
+                fields=('cable', 'cable_end', 'position'),
+                name='%(app_label)s_%(class)s_unique_position'
             ),
         )
         verbose_name = _('cable termination')
@@ -539,7 +478,9 @@ class CableTermination(ChangeLoggedModel):
         # Set the cable on the terminating object
         termination = self.termination._meta.model.objects.get(pk=self.termination_id)
         termination.snapshot()
-        termination.set_cable_termination(self)
+        termination.cable = self.cable
+        termination.cable_end = self.cable_end
+        termination.cable_position = self.position
         termination.save()
 
     def delete(self, *args, **kwargs):
@@ -547,7 +488,9 @@ class CableTermination(ChangeLoggedModel):
         # Delete the cable association on the terminating object
         termination = self.termination._meta.model.objects.get(pk=self.termination_id)
         termination.snapshot()
-        termination.clear_cable_termination(self)
+        termination.cable = None
+        termination.cable_end = None
+        termination.cable_position = None
         termination.save()
 
         super().delete(*args, **kwargs)
@@ -723,13 +666,7 @@ class CablePath(models.Model):
         is_active = True
         is_split = False
 
-        logger.debug(f'Tracing cable path from {terminations}...')
-
-        segment = 0
         while terminations:
-            segment += 1
-            logger.debug(f'[Path segment #{segment}] Position stack: {position_stack}')
-            logger.debug(f'[Path segment #{segment}] Local terminations: {terminations}')
 
             # Terminations must all be of the same type
             if not all(isinstance(t, type(terminations[0])) for t in terminations[1:]):
@@ -755,15 +692,12 @@ class CablePath(models.Model):
             path.append([
                 object_to_path_node(t) for t in terminations
             ])
-            # If not null, push cable position onto the stack
-            if isinstance(terminations[0], PathEndpoint) and terminations[0].cable_positions:
-                position_stack.append([terminations[0].cable_positions[0]])
+            # If not null, push cable_position onto the stack
+            if terminations[0].cable_position is not None:
+                position_stack.append([terminations[0].cable_position])
 
             # Step 2: Determine the attached links (Cable or WirelessLink), if any
-            links = list(dict.fromkeys(
-                termination.link for termination in terminations if termination.link is not None
-            ))
-            logger.debug(f'[Path segment #{segment}] Links: {links}')
+            links = [termination.link for termination in terminations if termination.link is not None]
             if len(links) == 0:
                 if len(path) == 1:
                     # If this is the start of the path and no link exists, return None
@@ -798,10 +732,8 @@ class CablePath(models.Model):
                 # Profile-based tracing
                 if links[0].profile:
                     cable_profile = links[0].profile_class()
-                    position = position_stack.pop()[0] if position_stack else None
-                    term, position = cable_profile.get_peer_termination(terminations[0], position)
-                    remote_terminations = [term]
-                    position_stack.append([position])
+                    peer_cable_terminations = cable_profile.get_peer_terminations(terminations, position_stack)
+                    remote_terminations = [ct.termination for ct in peer_cable_terminations]
 
                 # Legacy (positionless) behavior
                 else:
@@ -828,13 +760,10 @@ class CablePath(models.Model):
                     link.interface_b if link.interface_a is terminations[0] else link.interface_a for link in links
                 ]
 
-            logger.debug(f'[Path segment #{segment}] Remote terminations: {remote_terminations}')
-
             # Remote Terminations must all be of the same type, otherwise return a split path
             if not all(isinstance(t, type(remote_terminations[0])) for t in remote_terminations[1:]):
                 is_complete = False
                 is_split = True
-                logger.debug('Remote termination types differ; aborting trace.')
                 break
 
             # Step 7: Record the far-end termination object(s)
@@ -848,53 +777,58 @@ class CablePath(models.Model):
 
             if isinstance(remote_terminations[0], FrontPort):
                 # Follow FrontPorts to their corresponding RearPorts
-                if remote_terminations[0].positions > 1 and position_stack:
-                    positions = position_stack.pop()
-                    q_filter = Q()
-                    for rt in remote_terminations:
-                        q_filter |= Q(front_port=rt, front_port_position__in=positions)
-                    port_mappings = PortMapping.objects.filter(q_filter)
-                elif remote_terminations[0].positions > 1:
-                    is_split = True
-                    logger.debug(
-                        'Encountered front port mapped to multiple rear ports but position stack is empty; aborting '
-                        'trace.'
-                    )
-                    break
-                else:
-                    port_mappings = PortMapping.objects.filter(front_port__in=remote_terminations)
-                if not port_mappings:
-                    break
+                rear_ports = RearPort.objects.filter(
+                    pk__in=[t.rear_port_id for t in remote_terminations]
+                )
+                if len(rear_ports) > 1 or rear_ports[0].positions > 1:
+                    position_stack.append([fp.rear_port_position for fp in remote_terminations])
 
-                # Compile the list of RearPorts without duplication or altering their ordering
-                terminations = list(dict.fromkeys(mapping.rear_port for mapping in port_mappings))
-                if any(t.positions > 1 for t in terminations):
-                    position_stack.append([mapping.rear_port_position for mapping in port_mappings])
+                terminations = rear_ports
 
             elif isinstance(remote_terminations[0], RearPort):
-                # Follow RearPorts to their corresponding FrontPorts
-                if remote_terminations[0].positions > 1 and position_stack:
+                if len(remote_terminations) == 1 and remote_terminations[0].positions == 1:
+                    front_ports = FrontPort.objects.filter(
+                        rear_port_id__in=[rp.pk for rp in remote_terminations],
+                        rear_port_position=1
+                    )
+                # Obtain the individual front ports based on the termination and all positions
+                elif len(remote_terminations) > 1 and position_stack:
                     positions = position_stack.pop()
+
+                    # Ensure we have a number of positions equal to the amount of remote terminations
+                    if len(remote_terminations) != len(positions):
+                        raise UnsupportedCablePath(
+                            _("All positions counts within the path on opposite ends of links must match")
+                        )
+
+                    # Get our front ports
                     q_filter = Q()
                     for rt in remote_terminations:
-                        q_filter |= Q(rear_port=rt, rear_port_position__in=positions)
-                    port_mappings = PortMapping.objects.filter(q_filter)
-                elif remote_terminations[0].positions > 1:
-                    is_split = True
-                    logger.debug(
-                        'Encountered rear port mapped to multiple front ports but position stack is empty; aborting '
-                        'trace.'
+                        position = positions.pop()
+                        q_filter |= Q(rear_port_id=rt.pk, rear_port_position=position)
+                    if q_filter is Q():
+                        raise UnsupportedCablePath(_("Remote termination position filter is missing"))
+                    front_ports = FrontPort.objects.filter(q_filter)
+                # Obtain the individual front ports based on the termination and position
+                elif position_stack:
+                    front_ports = FrontPort.objects.filter(
+                        rear_port_id=remote_terminations[0].pk,
+                        rear_port_position__in=position_stack.pop()
                     )
-                    break
+                # If all rear ports have a single position, we can just get the front ports
+                elif all([rp.positions == 1 for rp in remote_terminations]):
+                    front_ports = FrontPort.objects.filter(rear_port_id__in=[rp.pk for rp in remote_terminations])
+
+                    if len(front_ports) != len(remote_terminations):
+                        # Some rear ports does not have a front port
+                        is_split = True
+                        break
                 else:
-                    port_mappings = PortMapping.objects.filter(rear_port__in=remote_terminations)
-                if not port_mappings:
+                    # No position indicated: path has split, so we stop at the RearPorts
+                    is_split = True
                     break
 
-                # Compile the list of FrontPorts without duplication or altering their ordering
-                terminations = list(dict.fromkeys(mapping.front_port for mapping in port_mappings))
-                if any(t.positions > 1 for t in terminations):
-                    position_stack.append([mapping.front_port_position for mapping in port_mappings])
+                terminations = front_ports
 
             elif isinstance(remote_terminations[0], CircuitTermination):
                 # Follow a CircuitTermination to its corresponding CircuitTermination (A to Z or vice versa)
@@ -942,7 +876,6 @@ class CablePath(models.Model):
                     # Unsupported topology, mark as split and exit
                     is_complete = False
                     is_split = True
-                    logger.warning('Encountered an unsupported topology; aborting trace.')
                 break
 
         return cls(
@@ -1021,23 +954,16 @@ class CablePath(models.Model):
 
         # RearPort splitting to multiple FrontPorts with no stack position
         if type(nodes[0]) is RearPort:
-            return [
-                mapping.front_port for mapping in
-                PortMapping.objects.filter(rear_port__in=nodes).prefetch_related('front_port')
-            ]
+            return FrontPort.objects.filter(rear_port__in=nodes)
         # Cable terminating to multiple FrontPorts mapped to different
         # RearPorts connected to different cables
-        if type(nodes[0]) is FrontPort:
-            return [
-                mapping.rear_port for mapping in
-                PortMapping.objects.filter(front_port__in=nodes).prefetch_related('rear_port')
-            ]
+        elif type(nodes[0]) is FrontPort:
+            return RearPort.objects.filter(pk__in=[fp.rear_port_id for fp in nodes])
         # Cable terminating to multiple CircuitTerminations
-        if type(nodes[0]) is CircuitTermination:
+        elif type(nodes[0]) is CircuitTermination:
             return [
                 ct.get_peer_termination() for ct in nodes
             ]
-        return []
 
     def get_asymmetric_nodes(self):
         """

netbox/dcim/models/device_component_templates.py

@@ -7,7 +7,6 @@ from mptt.models import MPTTModel, TreeForeignKey
 
 from dcim.choices import *
 from dcim.constants import *
-from dcim.models.base import PortMappingBase
 from dcim.models.mixins import InterfaceValidationMixin
 from netbox.models import ChangeLoggedModel
 from utilities.fields import ColorField, NaturalOrderingField
@@ -29,7 +28,6 @@ __all__ = (
     'InterfaceTemplate',
     'InventoryItemTemplate',
     'ModuleBayTemplate',
-    'PortTemplateMapping',
     'PowerOutletTemplate',
     'PowerPortTemplate',
     'RearPortTemplate',
@@ -520,53 +518,6 @@ class InterfaceTemplate(InterfaceValidationMixin, ModularComponentTemplateModel)
         }
 
 
-class PortTemplateMapping(PortMappingBase):
-    """
-    Maps a FrontPortTemplate & position to a RearPortTemplate & position.
-    """
-    device_type = models.ForeignKey(
-        to='dcim.DeviceType',
-        on_delete=models.CASCADE,
-        related_name='port_mappings',
-        blank=True,
-        null=True,
-    )
-    module_type = models.ForeignKey(
-        to='dcim.ModuleType',
-        on_delete=models.CASCADE,
-        related_name='port_mappings',
-        blank=True,
-        null=True,
-    )
-    front_port = models.ForeignKey(
-        to='dcim.FrontPortTemplate',
-        on_delete=models.CASCADE,
-        related_name='mappings',
-    )
-    rear_port = models.ForeignKey(
-        to='dcim.RearPortTemplate',
-        on_delete=models.CASCADE,
-        related_name='mappings',
-    )
-
-    def clean(self):
-        super().clean()
-
-        # Validate rear port assignment
-        if self.front_port.device_type_id != self.rear_port.device_type_id:
-            raise ValidationError({
-                "rear_port": _("Rear port ({rear_port}) must belong to the same device type").format(
-                    rear_port=self.rear_port
-                )
-            })
-
-    def save(self, *args, **kwargs):
-        # Associate the mapping with the parent DeviceType/ModuleType
-        self.device_type = self.front_port.device_type
-        self.module_type = self.front_port.module_type
-        super().save(*args, **kwargs)
-
-
 class FrontPortTemplate(ModularComponentTemplateModel):
     """
     Template for a pass-through port on the front of a new Device.
@@ -580,13 +531,18 @@ class FrontPortTemplate(ModularComponentTemplateModel):
         verbose_name=_('color'),
         blank=True
     )
-    positions = models.PositiveSmallIntegerField(
-        verbose_name=_('positions'),
+    rear_port = models.ForeignKey(
+        to='dcim.RearPortTemplate',
+        on_delete=models.CASCADE,
+        related_name='frontport_templates'
+    )
+    rear_port_position = models.PositiveSmallIntegerField(
+        verbose_name=_('rear port position'),
         default=1,
         validators=[
-            MinValueValidator(PORT_POSITION_MIN),
-            MaxValueValidator(PORT_POSITION_MAX)
-        ],
+            MinValueValidator(REARPORT_POSITIONS_MIN),
+            MaxValueValidator(REARPORT_POSITIONS_MAX)
+        ]
     )
 
     component_model = FrontPort
@@ -601,6 +557,10 @@ class FrontPortTemplate(ModularComponentTemplateModel):
                 fields=('module_type', 'name'),
                 name='%(app_label)s_%(class)s_unique_module_type_name'
             ),
+            models.UniqueConstraint(
+                fields=('rear_port', 'rear_port_position'),
+                name='%(app_label)s_%(class)s_unique_rear_port_position'
+            ),
         )
         verbose_name = _('front port template')
         verbose_name_plural = _('front port templates')
@@ -608,23 +568,40 @@ class FrontPortTemplate(ModularComponentTemplateModel):
     def clean(self):
         super().clean()
 
-        # Check that positions is greater than or equal to the number of associated RearPortTemplates
-        if not self._state.adding:
-            mapping_count = self.mappings.count()
-            if self.positions < mapping_count:
-                raise ValidationError({
-                    "positions": _(
-                        "The number of positions cannot be less than the number of mapped rear port templates ({count})"
-                    ).format(count=mapping_count)
-                })
+        try:
+
+            # Validate rear port assignment
+            if self.rear_port.device_type != self.device_type:
+                raise ValidationError(
+                    _("Rear port ({name}) must belong to the same device type").format(name=self.rear_port)
+                )
+
+            # Validate rear port position assignment
+            if self.rear_port_position > self.rear_port.positions:
+                raise ValidationError(
+                    _("Invalid rear port position ({position}); rear port {name} has only {count} positions").format(
+                        position=self.rear_port_position,
+                        name=self.rear_port.name,
+                        count=self.rear_port.positions
+                    )
+                )
+
+        except RearPortTemplate.DoesNotExist:
+            pass
 
     def instantiate(self, **kwargs):
+        if self.rear_port:
+            rear_port_name = self.rear_port.resolve_name(kwargs.get('module'))
+            rear_port = RearPort.objects.get(name=rear_port_name, **kwargs)
+        else:
+            rear_port = None
         return self.component_model(
             name=self.resolve_name(kwargs.get('module')),
             label=self.resolve_label(kwargs.get('module')),
             type=self.type,
             color=self.color,
-            positions=self.positions,
+            rear_port=rear_port,
+            rear_port_position=self.rear_port_position,
             **kwargs
         )
     instantiate.do_not_call_in_templates = True
@@ -634,7 +611,8 @@ class FrontPortTemplate(ModularComponentTemplateModel):
             'name': self.name,
             'type': self.type,
             'color': self.color,
-            'positions': self.positions,
+            'rear_port': self.rear_port.name,
+            'rear_port_position': self.rear_port_position,
             'label': self.label,
             'description': self.description,
         }
@@ -657,9 +635,9 @@ class RearPortTemplate(ModularComponentTemplateModel):
         verbose_name=_('positions'),
         default=1,
         validators=[
-            MinValueValidator(PORT_POSITION_MIN),
-            MaxValueValidator(PORT_POSITION_MAX)
-        ],
+            MinValueValidator(REARPORT_POSITIONS_MIN),
+            MaxValueValidator(REARPORT_POSITIONS_MAX)
+        ]
     )
 
     component_model = RearPort
@@ -668,20 +646,6 @@ class RearPortTemplate(ModularComponentTemplateModel):
         verbose_name = _('rear port template')
         verbose_name_plural = _('rear port templates')
 
-    def clean(self):
-        super().clean()
-
-        # Check that positions is greater than or equal to the number of associated FrontPortTemplates
-        if not self._state.adding:
-            mapping_count = self.mappings.count()
-            if self.positions < mapping_count:
-                raise ValidationError({
-                    "positions": _(
-                        "The number of positions cannot be less than the number of mapped front port templates "
-                        "({count})"
-                    ).format(count=mapping_count)
-                })
-
     def instantiate(self, **kwargs):
         return self.component_model(
             name=self.resolve_name(kwargs.get('module')),
@@ -723,8 +687,8 @@ class ModuleBayTemplate(ModularComponentTemplateModel):
 
     def instantiate(self, **kwargs):
         return self.component_model(
-            name=self.resolve_name(kwargs.get('module')),
-            label=self.resolve_label(kwargs.get('module')),
+            name=self.name,
+            label=self.label,
             position=self.position,
             **kwargs
         )

netbox/dcim/models/device_components.py

@@ -1,7 +1,6 @@
 from functools import cached_property
 
 from django.contrib.contenttypes.fields import GenericForeignKey, GenericRelation
-from django.contrib.postgres.fields import ArrayField
 from django.core.exceptions import ValidationError
 from django.core.validators import MaxValueValidator, MinValueValidator
 from django.db import models
@@ -12,7 +11,6 @@ from mptt.models import MPTTModel, TreeForeignKey
 from dcim.choices import *
 from dcim.constants import *
 from dcim.fields import WWNField
-from dcim.models.base import PortMappingBase
 from dcim.models.mixins import InterfaceValidationMixin
 from netbox.choices import ColorChoices
 from netbox.models import OrganizationalModel, NetBoxModel
@@ -37,7 +35,6 @@ __all__ = (
     'InventoryItemRole',
     'ModuleBay',
     'PathEndpoint',
-    'PortMapping',
     'PowerOutlet',
     'PowerPort',
     'RearPort',
@@ -178,24 +175,15 @@ class CabledObjectModel(models.Model):
         blank=True,
         null=True
     )
-    cable_connector = models.PositiveSmallIntegerField(
+    cable_position = models.PositiveIntegerField(
+        verbose_name=_('cable position'),
         blank=True,
         null=True,
         validators=(
-            MinValueValidator(CABLE_CONNECTOR_MIN),
-            MaxValueValidator(CABLE_CONNECTOR_MAX)
+            MinValueValidator(CABLE_POSITION_MIN),
+            MaxValueValidator(CABLE_POSITION_MAX)
         ),
     )
-    cable_positions = ArrayField(
-        base_field=models.PositiveSmallIntegerField(
-            validators=(
-                MinValueValidator(CABLE_POSITION_MIN),
-                MaxValueValidator(CABLE_POSITION_MAX)
-            )
-        ),
-        blank=True,
-        null=True,
-    )
     mark_connected = models.BooleanField(
         verbose_name=_('mark connected'),
         default=False,
@@ -220,31 +208,22 @@ class CabledObjectModel(models.Model):
                 raise ValidationError({
                     "cable_end": _("Must specify cable end (A or B) when attaching a cable.")
                 })
-            if self.cable_connector and not self.cable_positions:
+            if not self.cable_position:
                 raise ValidationError({
-                    "cable_positions": _("Must specify position(s) when specifying a cable connector.")
-                })
-            if self.cable_positions and not self.cable_connector:
-                raise ValidationError({
-                    "cable_positions": _("Cable positions cannot be set without a cable connector.")
-                })
-            if self.mark_connected:
-                raise ValidationError({
-                    "mark_connected": _("Cannot mark as connected with a cable attached.")
-                })
-        else:
-            if self.cable_end:
-                raise ValidationError({
-                    "cable_end": _("Cable end must not be set without a cable.")
-                })
-            if self.cable_connector:
-                raise ValidationError({
-                    "cable_connector": _("Cable connector must not be set without a cable.")
-                })
-            if self.cable_positions:
-                raise ValidationError({
-                    "cable_positions": _("Cable termination positions must not be set without a cable.")
+                    "cable_position": _("Must specify cable termination position when attaching a cable.")
                 })
+        if self.cable_end and not self.cable:
+            raise ValidationError({
+                "cable_end": _("Cable end must not be set without a cable.")
+            })
+        if self.cable_position and not self.cable:
+            raise ValidationError({
+                "cable_position": _("Cable termination position must not be set without a cable.")
+            })
+        if self.mark_connected and self.cable:
+            raise ValidationError({
+                "mark_connected": _("Cannot mark as connected with a cable attached.")
+            })
 
     @property
     def link(self):
@@ -279,22 +258,6 @@ class CabledObjectModel(models.Model):
             return None
         return CableEndChoices.SIDE_A if self.cable_end == CableEndChoices.SIDE_B else CableEndChoices.SIDE_B
 
-    def set_cable_termination(self, termination):
-        """Save attributes from the given CableTermination on the terminating object."""
-        self.cable = termination.cable
-        self.cable_end = termination.cable_end
-        self.cable_connector = termination.connector
-        self.cable_positions = termination.positions
-    set_cable_termination.alters_data = True
-
-    def clear_cable_termination(self, termination):
-        """Clear all cable termination attributes from the terminating object."""
-        self.cable = None
-        self.cable_end = None
-        self.cable_connector = None
-        self.cable_positions = None
-    clear_cable_termination.alters_data = True
-
 
 class PathEndpoint(models.Model):
     """
@@ -1106,43 +1069,6 @@ class Interface(
 # Pass-through ports
 #
 
-class PortMapping(PortMappingBase):
-    """
-    Maps a FrontPort & position to a RearPort & position.
-    """
-    device = models.ForeignKey(
-        to='dcim.Device',
-        on_delete=models.CASCADE,
-        related_name='port_mappings',
-    )
-    front_port = models.ForeignKey(
-        to='dcim.FrontPort',
-        on_delete=models.CASCADE,
-        related_name='mappings',
-    )
-    rear_port = models.ForeignKey(
-        to='dcim.RearPort',
-        on_delete=models.CASCADE,
-        related_name='mappings',
-    )
-
-    def clean(self):
-        super().clean()
-
-        # Both ports must belong to the same device
-        if self.front_port.device_id != self.rear_port.device_id:
-            raise ValidationError({
-                "rear_port": _("Rear port ({rear_port}) must belong to the same device").format(
-                    rear_port=self.rear_port
-                )
-            })
-
-    def save(self, *args, **kwargs):
-        # Associate the mapping with the parent Device
-        self.device = self.front_port.device
-        super().save(*args, **kwargs)
-
-
 class FrontPort(ModularComponentModel, CabledObjectModel, TrackingModelMixin):
     """
     A pass-through port on the front of a Device.
@@ -1156,16 +1082,22 @@ class FrontPort(ModularComponentModel, CabledObjectModel, TrackingModelMixin):
         verbose_name=_('color'),
         blank=True
     )
-    positions = models.PositiveSmallIntegerField(
-        verbose_name=_('positions'),
+    rear_port = models.ForeignKey(
+        to='dcim.RearPort',
+        on_delete=models.CASCADE,
+        related_name='frontports'
+    )
+    rear_port_position = models.PositiveSmallIntegerField(
+        verbose_name=_('rear port position'),
         default=1,
         validators=[
-            MinValueValidator(PORT_POSITION_MIN),
-            MaxValueValidator(PORT_POSITION_MAX)
+            MinValueValidator(REARPORT_POSITIONS_MIN),
+            MaxValueValidator(REARPORT_POSITIONS_MAX)
         ],
+        help_text=_('Mapped position on corresponding rear port')
     )
 
-    clone_fields = ('device', 'type', 'color', 'positions')
+    clone_fields = ('device', 'type', 'color')
 
     class Meta(ModularComponentModel.Meta):
         constraints = (
@@ -1173,6 +1105,10 @@ class FrontPort(ModularComponentModel, CabledObjectModel, TrackingModelMixin):
                 fields=('device', 'name'),
                 name='%(app_label)s_%(class)s_unique_device_name'
             ),
+            models.UniqueConstraint(
+                fields=('rear_port', 'rear_port_position'),
+                name='%(app_label)s_%(class)s_unique_rear_port_position'
+            ),
         )
         verbose_name = _('front port')
         verbose_name_plural = _('front ports')
@@ -1180,14 +1116,27 @@ class FrontPort(ModularComponentModel, CabledObjectModel, TrackingModelMixin):
     def clean(self):
         super().clean()
 
-        # Check that positions is greater than or equal to the number of associated RearPorts
-        if not self._state.adding:
-            mapping_count = self.mappings.count()
-            if self.positions < mapping_count:
+        if hasattr(self, 'rear_port'):
+
+            # Validate rear port assignment
+            if self.rear_port.device != self.device:
                 raise ValidationError({
-                    "positions": _(
-                        "The number of positions cannot be less than the number of mapped rear ports ({count})"
-                    ).format(count=mapping_count)
+                    "rear_port": _(
+                        "Rear port ({rear_port}) must belong to the same device"
+                    ).format(rear_port=self.rear_port)
+                })
+
+            # Validate rear port position assignment
+            if self.rear_port_position > self.rear_port.positions:
+                raise ValidationError({
+                    "rear_port_position": _(
+                        "Invalid rear port position ({rear_port_position}): Rear port {name} has only {positions} "
+                        "positions."
+                    ).format(
+                        rear_port_position=self.rear_port_position,
+                        name=self.rear_port.name,
+                        positions=self.rear_port.positions
+                    )
                 })
 
 
@@ -1208,11 +1157,11 @@ class RearPort(ModularComponentModel, CabledObjectModel, TrackingModelMixin):
         verbose_name=_('positions'),
         default=1,
         validators=[
-            MinValueValidator(PORT_POSITION_MIN),
-            MaxValueValidator(PORT_POSITION_MAX)
+            MinValueValidator(REARPORT_POSITIONS_MIN),
+            MaxValueValidator(REARPORT_POSITIONS_MAX)
         ],
+        help_text=_('Number of front ports which may be mapped')
     )
-
     clone_fields = ('device', 'type', 'color', 'positions')
 
     class Meta(ModularComponentModel.Meta):
@@ -1224,13 +1173,13 @@ class RearPort(ModularComponentModel, CabledObjectModel, TrackingModelMixin):
 
         # Check that positions count is greater than or equal to the number of associated FrontPorts
         if not self._state.adding:
-            mapping_count = self.mappings.count()
-            if self.positions < mapping_count:
+            frontport_count = self.frontports.count()
+            if self.positions < frontport_count:
                 raise ValidationError({
                     "positions": _(
                         "The number of positions cannot be less than the number of mapped front ports "
-                        "({count})"
-                    ).format(count=mapping_count)
+                        "({frontport_count})"
+                    ).format(frontport_count=frontport_count)
                 })
 
 
@@ -1292,8 +1241,6 @@ class ModuleBay(ModularComponentModel, TrackingModelMixin, MPTTModel):
     def save(self, *args, **kwargs):
         if self.module:
             self.parent = self.module.module_bay
-        else:
-            self.parent = None
         super().save(*args, **kwargs)
 
 

netbox/dcim/models/devices.py

@@ -1,7 +1,8 @@
 import decimal
+import yaml
+
 from functools import cached_property
 
-import yaml
 from django.contrib.contenttypes.fields import GenericForeignKey, GenericRelation
 from django.contrib.contenttypes.models import ContentType
 from django.core.exceptions import ValidationError
@@ -18,14 +19,14 @@ from django.utils.translation import gettext_lazy as _
 from dcim.choices import *
 from dcim.constants import *
 from dcim.fields import MACAddressField
-from dcim.utils import create_port_mappings, update_interface_bridges
+from dcim.utils import update_interface_bridges
 from extras.models import ConfigContextModel, CustomField
 from extras.querysets import ConfigContextModelQuerySet
 from netbox.choices import ColorChoices
 from netbox.config import ConfigItem
 from netbox.models import NestedGroupModel, OrganizationalModel, PrimaryModel
-from netbox.models.features import ContactsMixin, ImageAttachmentsMixin
 from netbox.models.mixins import WeightMixin
+from netbox.models.features import ContactsMixin, ImageAttachmentsMixin
 from utilities.fields import ColorField, CounterCacheField
 from utilities.prefetch import get_prefetchable_fields
 from utilities.tracking import TrackingModelMixin
@@ -33,6 +34,7 @@ from .device_components import *
 from .mixins import RenderConfigMixin
 from .modules import Module
 
+
 __all__ = (
     'Device',
     'DeviceRole',
@@ -648,10 +650,7 @@ class Device(
         decimal_places=6,
         blank=True,
         null=True,
-        validators=[
-            MinValueValidator(decimal.Decimal('-90.0')),
-            MaxValueValidator(decimal.Decimal('90.0'))
-        ],
+        validators=[MinValueValidator(-90.0), MaxValueValidator(90.0)],
         help_text=_("GPS coordinate in decimal format (xx.yyyyyy)")
     )
     longitude = models.DecimalField(
@@ -660,10 +659,7 @@ class Device(
         decimal_places=6,
         blank=True,
         null=True,
-        validators=[
-            MinValueValidator(decimal.Decimal('-180.0')),
-            MaxValueValidator(decimal.Decimal('180.0'))
-        ],
+        validators=[MinValueValidator(-180.0), MaxValueValidator(180.0)],
         help_text=_("GPS coordinate in decimal format (xx.yyyyyy)")
     )
     services = GenericRelation(
@@ -959,11 +955,6 @@ class Device(
             if cf_defaults := CustomField.objects.get_defaults_for_model(model):
                 for component in components:
                     component.custom_field_data = cf_defaults
-            # Set denormalized references
-            for component in components:
-                component._site = self.site
-                component._location = self.location
-                component._rack = self.rack
             components = model.objects.bulk_create(components)
             # Prefetch related objects to minimize queries needed during post_save
             prefetch_fields = get_prefetchable_fields(model)
@@ -1012,8 +1003,6 @@ class Device(
             self._instantiate_components(self.device_type.interfacetemplates.all())
             self._instantiate_components(self.device_type.rearporttemplates.all())
             self._instantiate_components(self.device_type.frontporttemplates.all())
-            # Replicate any front/rear port mappings from the DeviceType
-            create_port_mappings(self, self.device_type)
             # Disable bulk_create to accommodate MPTT
             self._instantiate_components(self.device_type.modulebaytemplates.all(), bulk_create=False)
             self._instantiate_components(self.device_type.devicebaytemplates.all())
@@ -1323,10 +1312,7 @@ class MACAddress(PrimaryModel):
     )
 
     class Meta:
-        ordering = ('mac_address', 'pk')
-        indexes = (
-            models.Index(fields=('assigned_object_type', 'assigned_object_id')),
-        )
+        ordering = ('mac_address', 'pk',)
         verbose_name = _('MAC address')
         verbose_name_plural = _('MAC addresses')
 

netbox/dcim/models/modules.py

@@ -7,6 +7,7 @@ from django.utils.translation import gettext_lazy as _
 from jsonschema.exceptions import ValidationError as JSONValidationError
 
 from dcim.choices import *
+from dcim.constants import MODULE_TOKEN
 from dcim.utils import update_interface_bridges
 from extras.models import ConfigContextModel, CustomField
 from netbox.models import PrimaryModel
@@ -259,13 +260,11 @@ class Module(TrackingModelMixin, PrimaryModel, ConfigContextModel):
         module_bays = []
         modules = []
         while module:
-            module_module_bay = getattr(module, "module_bay", None)
-            if module.pk in modules or (module_module_bay and module_module_bay.pk in module_bays):
+            if module.pk in modules or module.module_bay.pk in module_bays:
                 raise ValidationError(_("A module bay cannot belong to a module installed within it."))
             modules.append(module.pk)
-            if module_module_bay:
-                module_bays.append(module_module_bay.pk)
-            module = module_module_bay.module if module_module_bay else None
+            module_bays.append(module.module_bay.pk)
+            module = module.module_bay.module if module.module_bay else None
 
     def save(self, *args, **kwargs):
         is_new = self.pk is None
@@ -323,12 +322,6 @@ class Module(TrackingModelMixin, PrimaryModel, ConfigContextModel):
                 for component in create_instances:
                     component.custom_field_data = cf_defaults
 
-            # Set denormalized references
-            for component in create_instances:
-                component._site = self.device.site
-                component._location = self.device.location
-                component._rack = self.device.rack
-
             if component_model is not ModuleBay:
                 component_model.objects.bulk_create(create_instances)
                 # Emit the post_save signal for each newly created object
@@ -344,6 +337,7 @@ class Module(TrackingModelMixin, PrimaryModel, ConfigContextModel):
             else:
                 # ModuleBays must be saved individually for MPTT
                 for instance in create_instances:
+                    instance.name = instance.name.replace(MODULE_TOKEN, str(self.module_bay.position))
                     instance.save()
 
             update_fields = ['module']

netbox/dcim/models/sites.py

@@ -1,5 +1,3 @@
-import decimal
-
 from django.contrib.contenttypes.fields import GenericRelation
 from django.core.exceptions import ValidationError
 from django.core.validators import MaxValueValidator, MinValueValidator
@@ -213,10 +211,7 @@ class Site(ContactsMixin, ImageAttachmentsMixin, PrimaryModel):
         decimal_places=6,
         blank=True,
         null=True,
-        validators=[
-            MinValueValidator(decimal.Decimal('-90.0')),
-            MaxValueValidator(decimal.Decimal('90.0'))
-        ],
+        validators=[MinValueValidator(-90.0), MaxValueValidator(90.0)],
         help_text=_('GPS coordinate in decimal format (xx.yyyyyy)')
     )
     longitude = models.DecimalField(
@@ -225,10 +220,7 @@ class Site(ContactsMixin, ImageAttachmentsMixin, PrimaryModel):
         decimal_places=6,
         blank=True,
         null=True,
-        validators=[
-            MinValueValidator(decimal.Decimal('-180.0')),
-            MaxValueValidator(decimal.Decimal('180.0'))
-        ],
+        validators=[MinValueValidator(-180.0), MaxValueValidator(180.0)],
         help_text=_('GPS coordinate in decimal format (xx.yyyyyy)')
     )
 

netbox/dcim/search.py

@@ -137,18 +137,6 @@ class InventoryItemIndex(SearchIndex):
     display_attrs = ('device', 'manufacturer', 'parent', 'part_id', 'serial', 'asset_tag', 'description')
 
 
-@register_search
-class InventoryItemRoleIndex(SearchIndex):
-    model = models.InventoryItemRole
-    fields = (
-        ('name', 100),
-        ('slug', 110),
-        ('description', 500),
-        ('comments', 5000),
-    )
-    display_attrs = ('description',)
-
-
 @register_search
 class LocationIndex(SearchIndex):
     model = models.Location
@@ -169,7 +157,6 @@ class ManufacturerIndex(SearchIndex):
         ('name', 100),
         ('slug', 110),
         ('description', 500),
-        ('comments', 5000),
     )
     display_attrs = ('description',)
 
@@ -321,7 +308,6 @@ class RackRoleIndex(SearchIndex):
         ('name', 100),
         ('slug', 110),
         ('description', 500),
-        ('comments', 5000),
     )
     display_attrs = ('description',)
 

netbox/dcim/signals.py

@@ -1,17 +1,14 @@
 import logging
 
-from django.db.models import Q
-from django.db.models.signals import post_delete, post_save
+from django.db.models.signals import post_save, post_delete
 from django.dispatch import receiver
 
 from dcim.choices import CableEndChoices, LinkStatusChoices
-from ipam.models import Prefix
-from virtualization.models import Cluster, VMInterface
-from wireless.models import WirelessLAN
+from virtualization.models import VMInterface
 from .models import (
     Cable, CablePath, CableTermination, ConsolePort, ConsoleServerPort, Device, DeviceBay, FrontPort, Interface,
-    InventoryItem, Location, ModuleBay, PathEndpoint, PortMapping, PowerOutlet, PowerPanel, PowerPort, Rack, RearPort,
-    Site, VirtualChassis,
+    InventoryItem, ModuleBay, PathEndpoint, PowerOutlet, PowerPanel, PowerPort, Rack, RearPort, Location,
+    VirtualChassis,
 )
 from .models.cables import trace_paths
 from .utils import create_cablepaths, rebuild_paths
@@ -47,9 +44,6 @@ def handle_location_site_change(instance, created, **kwargs):
         Device.objects.filter(location__in=locations).update(site=instance.site)
         PowerPanel.objects.filter(location__in=locations).update(site=instance.site)
         CableTermination.objects.filter(_location__in=locations).update(_site=instance.site)
-        # Update component models for devices in these locations
-        for model in COMPONENT_MODELS:
-            model.objects.filter(device__location__in=locations).update(_site=instance.site)
 
 
 @receiver(post_save, sender=Rack)
@@ -59,12 +53,6 @@ def handle_rack_site_change(instance, created, **kwargs):
     """
     if not created:
         Device.objects.filter(rack=instance).update(site=instance.site, location=instance.location)
-        # Update component models for devices in this rack
-        for model in COMPONENT_MODELS:
-            model.objects.filter(device__rack=instance).update(
-                _site=instance.site,
-                _location=instance.location,
-            )
 
 
 @receiver(post_save, sender=Device)
@@ -147,17 +135,6 @@ def retrace_cable_paths(instance, **kwargs):
         cablepath.retrace()
 
 
-@receiver((post_delete, post_save), sender=PortMapping)
-def update_passthrough_port_paths(instance, **kwargs):
-    """
-    When a PortMapping is created or deleted, retrace any CablePaths which traverse its front and/or rear ports.
-    """
-    for cablepath in CablePath.objects.filter(
-        Q(_nodes__contains=instance.front_port) | Q(_nodes__contains=instance.rear_port)
-    ):
-        cablepath.retrace()
-
-
 @receiver(post_delete, sender=CableTermination)
 def nullify_connected_endpoints(instance, **kwargs):
     """
@@ -173,6 +150,17 @@ def nullify_connected_endpoints(instance, **kwargs):
         cablepath.retrace()
 
 
+@receiver(post_save, sender=FrontPort)
+def extend_rearport_cable_paths(instance, created, raw, **kwargs):
+    """
+    When a new FrontPort is created, add it to any CablePaths which end at its corresponding RearPort.
+    """
+    if created and not raw:
+        rearport = instance.rear_port
+        for cablepath in CablePath.objects.filter(_nodes__contains=rearport):
+            cablepath.retrace()
+
+
 @receiver(post_save, sender=Interface)
 @receiver(post_save, sender=VMInterface)
 def update_mac_address_interface(instance, created, raw, **kwargs):
@@ -183,44 +171,3 @@ def update_mac_address_interface(instance, created, raw, **kwargs):
     if created and not raw and instance.primary_mac_address:
         instance.primary_mac_address.assigned_object = instance
         instance.primary_mac_address.save()
-
-
-@receiver(post_save, sender=Location)
-@receiver(post_save, sender=Site)
-def sync_cached_scope_fields(instance, created, **kwargs):
-    """
-    Rebuild cached scope fields for all CachedScopeMixin-based models
-    affected by a change in a Region, SiteGroup, Site, or Location.
-
-    This method is safe to run for objects created in the past and does
-    not rely on incremental updates. Cached fields are recomputed from
-    authoritative relationships.
-    """
-    if created:
-        return
-
-    if isinstance(instance, Location):
-        filters = {'_location': instance}
-    elif isinstance(instance, Site):
-        filters = {'_site': instance}
-    else:
-        return
-
-    # These models are explicitly listed because they all subclass CachedScopeMixin
-    # and therefore require their cached scope fields to be recomputed.
-    for model in (Prefix, Cluster, WirelessLAN):
-        qs = model.objects.filter(**filters)
-
-        # Bulk update cached fields to avoid O(N) performance issues with large datasets.
-        # This does not trigger post_save signals, avoiding spurious change log entries.
-        objects_to_update = []
-        for obj in qs:
-            # Recompute cache using the same logic as save()
-            obj.cache_related_objects()
-            objects_to_update.append(obj)
-
-        if objects_to_update:
-            model.objects.bulk_update(
-                objects_to_update,
-                ['_location', '_site', '_site_group', '_region']
-            )

netbox/dcim/tables/devices.py

@@ -749,9 +749,12 @@ class FrontPortTable(ModularDeviceComponentTable, CableTerminationTable):
     color = columns.ColorColumn(
         verbose_name=_('Color'),
     )
-    mappings = columns.ManyToManyColumn(
-        verbose_name=_('Mappings'),
-        transform=lambda obj: f'{obj.rear_port}:{obj.rear_port_position}'
+    rear_port_position = tables.Column(
+        verbose_name=_('Position')
+    )
+    rear_port = tables.Column(
+        verbose_name=_('Rear Port'),
+        linkify=True
     )
     tags = columns.TagColumn(
         url_name='dcim:frontport_list'
@@ -760,12 +763,12 @@ class FrontPortTable(ModularDeviceComponentTable, CableTerminationTable):
     class Meta(DeviceComponentTable.Meta):
         model = models.FrontPort
         fields = (
-            'pk', 'id', 'name', 'device', 'module_bay', 'module', 'label', 'type', 'color', 'positions', 'mappings',
-            'description', 'mark_connected', 'cable', 'cable_color', 'link_peer', 'inventory_items', 'tags', 'created',
-            'last_updated',
+            'pk', 'id', 'name', 'device', 'module_bay', 'module', 'label', 'type', 'color', 'rear_port',
+            'rear_port_position', 'description', 'mark_connected', 'cable', 'cable_color', 'link_peer',
+            'inventory_items', 'tags', 'created', 'last_updated',
         )
         default_columns = (
-            'pk', 'name', 'device', 'label', 'type', 'color', 'positions', 'mappings', 'description',
+            'pk', 'name', 'device', 'label', 'type', 'color', 'rear_port', 'rear_port_position', 'description',
         )
 
 
@@ -783,11 +786,11 @@ class DeviceFrontPortTable(FrontPortTable):
     class Meta(CableTerminationTable.Meta, DeviceComponentTable.Meta):
         model = models.FrontPort
         fields = (
-            'pk', 'id', 'name', 'module_bay', 'module', 'label', 'type', 'color', 'positions', 'mappings',
+            'pk', 'id', 'name', 'module_bay', 'module', 'label', 'type', 'rear_port', 'rear_port_position',
             'description', 'mark_connected', 'cable', 'cable_color', 'link_peer', 'tags', 'actions',
         )
         default_columns = (
-            'pk', 'name', 'label', 'type', 'color', 'positions', 'mappings', 'description', 'cable', 'link_peer',
+            'pk', 'name', 'label', 'type', 'rear_port', 'rear_port_position', 'description', 'cable', 'link_peer',
         )
 
 
@@ -802,10 +805,6 @@ class RearPortTable(ModularDeviceComponentTable, CableTerminationTable):
     color = columns.ColorColumn(
         verbose_name=_('Color'),
     )
-    mappings = columns.ManyToManyColumn(
-        verbose_name=_('Mappings'),
-        transform=lambda obj: f'{obj.front_port}:{obj.front_port_position}'
-    )
     tags = columns.TagColumn(
         url_name='dcim:rearport_list'
     )
@@ -813,13 +812,10 @@ class RearPortTable(ModularDeviceComponentTable, CableTerminationTable):
     class Meta(DeviceComponentTable.Meta):
         model = models.RearPort
         fields = (
-            'pk', 'id', 'name', 'device', 'module_bay', 'module', 'label', 'type', 'color', 'positions', 'mappings',
-            'description', 'mark_connected', 'cable', 'cable_color', 'link_peer', 'inventory_items', 'tags', 'created',
-            'last_updated',
-        )
-        default_columns = (
-            'pk', 'name', 'device', 'label', 'type', 'color', 'positions', 'mappings', 'description',
+            'pk', 'id', 'name', 'device', 'module_bay', 'module', 'label', 'type', 'color', 'positions', 'description',
+            'mark_connected', 'cable', 'cable_color', 'link_peer', 'inventory_items', 'tags', 'created', 'last_updated',
         )
+        default_columns = ('pk', 'name', 'device', 'label', 'type', 'color', 'description')
 
 
 class DeviceRearPortTable(RearPortTable):
@@ -836,11 +832,11 @@ class DeviceRearPortTable(RearPortTable):
     class Meta(CableTerminationTable.Meta, DeviceComponentTable.Meta):
         model = models.RearPort
         fields = (
-            'pk', 'id', 'name', 'module_bay', 'module', 'label', 'type', 'color', 'positions', 'mappings',
-            'description', 'mark_connected', 'cable', 'cable_color', 'link_peer', 'tags', 'actions',
+            'pk', 'id', 'name', 'module_bay', 'module', 'label', 'type', 'positions', 'description', 'mark_connected',
+            'cable', 'cable_color', 'link_peer', 'tags', 'actions',
         )
         default_columns = (
-            'pk', 'name', 'label', 'type', 'color', 'positions', 'mappings', 'description', 'cable', 'link_peer',
+            'pk', 'name', 'label', 'type', 'positions', 'description', 'cable', 'link_peer',
         )
 
 
@@ -1057,7 +1053,7 @@ class InventoryItemRoleTable(OrganizationalModelTable):
     class Meta(OrganizationalModelTable.Meta):
         model = models.InventoryItemRole
         fields = (
-            'pk', 'id', 'name', 'inventoryitem_count', 'color', 'description', 'slug', 'comments', 'tags', 'actions',
+            'pk', 'id', 'name', 'inventoryitem_count', 'color', 'description', 'slug', 'tags', 'actions',
         )
         default_columns = ('pk', 'name', 'inventoryitem_count', 'color', 'description')
 

netbox/dcim/tables/devicetypes.py

@@ -64,8 +64,7 @@ class ManufacturerTable(ContactsColumnMixin, OrganizationalModelTable):
         model = models.Manufacturer
         fields = (
             'pk', 'id', 'name', 'racktype_count', 'devicetype_count', 'moduletype_count', 'inventoryitem_count',
-            'platform_count', 'description', 'slug', 'comments', 'tags', 'contacts', 'actions', 'created',
-            'last_updated',
+            'platform_count', 'description', 'slug', 'tags', 'contacts', 'actions', 'created', 'last_updated',
         )
         default_columns = (
             'pk', 'name', 'racktype_count', 'devicetype_count', 'moduletype_count', 'inventoryitem_count',
@@ -250,13 +249,12 @@ class InterfaceTemplateTable(ComponentTemplateTable):
 
 
 class FrontPortTemplateTable(ComponentTemplateTable):
+    rear_port_position = tables.Column(
+        verbose_name=_('Position')
+    )
     color = columns.ColorColumn(
         verbose_name=_('Color'),
     )
-    mappings = columns.ManyToManyColumn(
-        verbose_name=_('Mappings'),
-        transform=lambda obj: f'{obj.rear_port}:{obj.rear_port_position}'
-    )
     actions = columns.ActionsColumn(
         actions=('edit', 'delete'),
         extra_buttons=MODULAR_COMPONENT_TEMPLATE_BUTTONS
@@ -264,7 +262,7 @@ class FrontPortTemplateTable(ComponentTemplateTable):
 
     class Meta(ComponentTemplateTable.Meta):
         model = models.FrontPortTemplate
-        fields = ('pk', 'name', 'label', 'type', 'color', 'positions', 'mappings', 'description', 'actions')
+        fields = ('pk', 'name', 'label', 'type', 'color', 'rear_port', 'rear_port_position', 'description', 'actions')
         empty_text = "None"
 
 
@@ -272,10 +270,6 @@ class RearPortTemplateTable(ComponentTemplateTable):
     color = columns.ColorColumn(
         verbose_name=_('Color'),
     )
-    mappings = columns.ManyToManyColumn(
-        verbose_name=_('Mappings'),
-        transform=lambda obj: f'{obj.front_port}:{obj.front_port_position}'
-    )
     actions = columns.ActionsColumn(
         actions=('edit', 'delete'),
         extra_buttons=MODULAR_COMPONENT_TEMPLATE_BUTTONS
@@ -283,7 +277,7 @@ class RearPortTemplateTable(ComponentTemplateTable):
 
     class Meta(ComponentTemplateTable.Meta):
         model = models.RearPortTemplate
-        fields = ('pk', 'name', 'label', 'type', 'color', 'positions', 'mappings', 'description', 'actions')
+        fields = ('pk', 'name', 'label', 'type', 'color', 'positions', 'description', 'actions')
         empty_text = "None"
 
 

netbox/dcim/tables/racks.py

@@ -35,7 +35,7 @@ class RackRoleTable(OrganizationalModelTable):
     class Meta(OrganizationalModelTable.Meta):
         model = RackRole
         fields = (
-            'pk', 'id', 'name', 'rack_count', 'color', 'description', 'slug', 'comments', 'tags', 'actions', 'created',
+            'pk', 'id', 'name', 'rack_count', 'color', 'description', 'slug', 'tags', 'actions', 'created',
             'last_updated',
         )
         default_columns = ('pk', 'name', 'rack_count', 'color', 'description')

netbox/dcim/tests/test_api.py

@@ -532,7 +532,7 @@ class RackReservationTest(APIViewTestCases.APIViewTestCase):
 
 class ManufacturerTest(APIViewTestCases.APIViewTestCase):
     model = Manufacturer
-    brief_fields = ['description', 'display', 'id', 'name', 'slug', 'url']
+    brief_fields = ['description', 'devicetype_count', 'display', 'id', 'name', 'slug', 'url']
     create_data = [
         {
             'name': 'Manufacturer 4',
@@ -973,99 +973,72 @@ class FrontPortTemplateTest(APIViewTestCases.APIViewTestCase):
             RearPortTemplate(device_type=devicetype, name='Rear Port Template 2', type=PortTypeChoices.TYPE_8P8C),
             RearPortTemplate(device_type=devicetype, name='Rear Port Template 3', type=PortTypeChoices.TYPE_8P8C),
             RearPortTemplate(device_type=devicetype, name='Rear Port Template 4', type=PortTypeChoices.TYPE_8P8C),
-            RearPortTemplate(device_type=devicetype, name='Rear Port Template 5', type=PortTypeChoices.TYPE_8P8C),
-            RearPortTemplate(device_type=devicetype, name='Rear Port Template 6', type=PortTypeChoices.TYPE_8P8C),
+            RearPortTemplate(module_type=moduletype, name='Rear Port Template 5', type=PortTypeChoices.TYPE_8P8C),
+            RearPortTemplate(module_type=moduletype, name='Rear Port Template 6', type=PortTypeChoices.TYPE_8P8C),
+            RearPortTemplate(module_type=moduletype, name='Rear Port Template 7', type=PortTypeChoices.TYPE_8P8C),
+            RearPortTemplate(module_type=moduletype, name='Rear Port Template 8', type=PortTypeChoices.TYPE_8P8C),
         )
         RearPortTemplate.objects.bulk_create(rear_port_templates)
+
         front_port_templates = (
-            FrontPortTemplate(device_type=devicetype, name='Front Port Template 1', type=PortTypeChoices.TYPE_8P8C),
-            FrontPortTemplate(device_type=devicetype, name='Front Port Template 2', type=PortTypeChoices.TYPE_8P8C),
-            FrontPortTemplate(module_type=moduletype, name='Front Port Template 3', type=PortTypeChoices.TYPE_8P8C),
+            FrontPortTemplate(
+                device_type=devicetype,
+                name='Front Port Template 1',
+                type=PortTypeChoices.TYPE_8P8C,
+                rear_port=rear_port_templates[0]
+            ),
+            FrontPortTemplate(
+                device_type=devicetype,
+                name='Front Port Template 2',
+                type=PortTypeChoices.TYPE_8P8C,
+                rear_port=rear_port_templates[1]
+            ),
+            FrontPortTemplate(
+                module_type=moduletype,
+                name='Front Port Template 5',
+                type=PortTypeChoices.TYPE_8P8C,
+                rear_port=rear_port_templates[4]
+            ),
+            FrontPortTemplate(
+                module_type=moduletype,
+                name='Front Port Template 6',
+                type=PortTypeChoices.TYPE_8P8C,
+                rear_port=rear_port_templates[5]
+            ),
         )
         FrontPortTemplate.objects.bulk_create(front_port_templates)
-        PortTemplateMapping.objects.bulk_create([
-            PortTemplateMapping(
-                device_type=devicetype,
-                front_port=front_port_templates[0],
-                rear_port=rear_port_templates[0],
-            ),
-            PortTemplateMapping(
-                device_type=devicetype,
-                front_port=front_port_templates[1],
-                rear_port=rear_port_templates[1],
-            ),
-            PortTemplateMapping(
-                module_type=moduletype,
-                front_port=front_port_templates[2],
-                rear_port=rear_port_templates[2],
-            ),
-        ])
 
         cls.create_data = [
             {
                 'device_type': devicetype.pk,
                 'name': 'Front Port Template 3',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'rear_ports': [
-                    {
-                        'position': 1,
-                        'rear_port': rear_port_templates[3].pk,
-                        'rear_port_position': 1,
-                    },
-                ],
+                'rear_port': rear_port_templates[2].pk,
+                'rear_port_position': 1,
             },
             {
                 'device_type': devicetype.pk,
                 'name': 'Front Port Template 4',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'rear_ports': [
-                    {
-                        'position': 1,
-                        'rear_port': rear_port_templates[4].pk,
-                        'rear_port_position': 1,
-                    },
-                ],
+                'rear_port': rear_port_templates[3].pk,
+                'rear_port_position': 1,
             },
             {
                 'module_type': moduletype.pk,
                 'name': 'Front Port Template 7',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'rear_ports': [
-                    {
-                        'position': 1,
-                        'rear_port': rear_port_templates[5].pk,
-                        'rear_port_position': 1,
-                    },
-                ],
+                'rear_port': rear_port_templates[6].pk,
+                'rear_port_position': 1,
+            },
+            {
+                'module_type': moduletype.pk,
+                'name': 'Front Port Template 8',
+                'type': PortTypeChoices.TYPE_8P8C,
+                'rear_port': rear_port_templates[7].pk,
+                'rear_port_position': 1,
             },
         ]
 
-        cls.update_data = {
-            'type': PortTypeChoices.TYPE_LC,
-            'rear_ports': [
-                {
-                    'position': 1,
-                    'rear_port': rear_port_templates[3].pk,
-                    'rear_port_position': 1,
-                },
-            ],
-        }
-
-    def test_update_object(self):
-        super().test_update_object()
-
-        # Check that the port mapping was updated after modifying the front port template
-        front_port_template = FrontPortTemplate.objects.get(name='Front Port Template 1')
-        rear_port_template = RearPortTemplate.objects.get(name='Rear Port Template 4')
-        self.assertTrue(
-            PortTemplateMapping.objects.filter(
-                front_port=front_port_template,
-                front_port_position=1,
-                rear_port=rear_port_template,
-                rear_port_position=1,
-            ).exists()
-        )
-
 
 class RearPortTemplateTest(APIViewTestCases.APIViewTestCase):
     model = RearPortTemplate
@@ -1084,104 +1057,36 @@ class RearPortTemplateTest(APIViewTestCases.APIViewTestCase):
             manufacturer=manufacturer, model='Module Type 1'
         )
 
-        front_port_templates = (
-            FrontPortTemplate(device_type=devicetype, name='Front Port Template 1', type=PortTypeChoices.TYPE_8P8C),
-            FrontPortTemplate(device_type=devicetype, name='Front Port Template 2', type=PortTypeChoices.TYPE_8P8C),
-            FrontPortTemplate(module_type=moduletype, name='Front Port Template 3', type=PortTypeChoices.TYPE_8P8C),
-            FrontPortTemplate(module_type=moduletype, name='Front Port Template 4', type=PortTypeChoices.TYPE_8P8C),
-            FrontPortTemplate(module_type=moduletype, name='Front Port Template 5', type=PortTypeChoices.TYPE_8P8C),
-            FrontPortTemplate(module_type=moduletype, name='Front Port Template 6', type=PortTypeChoices.TYPE_8P8C),
-        )
-        FrontPortTemplate.objects.bulk_create(front_port_templates)
         rear_port_templates = (
             RearPortTemplate(device_type=devicetype, name='Rear Port Template 1', type=PortTypeChoices.TYPE_8P8C),
             RearPortTemplate(device_type=devicetype, name='Rear Port Template 2', type=PortTypeChoices.TYPE_8P8C),
             RearPortTemplate(device_type=devicetype, name='Rear Port Template 3', type=PortTypeChoices.TYPE_8P8C),
         )
         RearPortTemplate.objects.bulk_create(rear_port_templates)
-        PortTemplateMapping.objects.bulk_create([
-            PortTemplateMapping(
-                device_type=devicetype,
-                front_port=front_port_templates[0],
-                rear_port=rear_port_templates[0],
-            ),
-            PortTemplateMapping(
-                device_type=devicetype,
-                front_port=front_port_templates[1],
-                rear_port=rear_port_templates[1],
-            ),
-            PortTemplateMapping(
-                module_type=moduletype,
-                front_port=front_port_templates[2],
-                rear_port=rear_port_templates[2],
-            ),
-        ])
 
         cls.create_data = [
             {
                 'device_type': devicetype.pk,
                 'name': 'Rear Port Template 4',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'front_ports': [
-                    {
-                        'position': 1,
-                        'front_port': front_port_templates[3].pk,
-                        'front_port_position': 1,
-                    },
-                ],
             },
             {
                 'device_type': devicetype.pk,
                 'name': 'Rear Port Template 5',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'front_ports': [
-                    {
-                        'position': 1,
-                        'front_port': front_port_templates[4].pk,
-                        'front_port_position': 1,
-                    },
-                ],
             },
             {
                 'module_type': moduletype.pk,
                 'name': 'Rear Port Template 6',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'front_ports': [
-                    {
-                        'position': 1,
-                        'front_port': front_port_templates[5].pk,
-                        'front_port_position': 1,
-                    },
-                ],
+            },
+            {
+                'module_type': moduletype.pk,
+                'name': 'Rear Port Template 7',
+                'type': PortTypeChoices.TYPE_8P8C,
             },
         ]
 
-        cls.update_data = {
-            'type': PortTypeChoices.TYPE_LC,
-            'front_ports': [
-                {
-                    'position': 1,
-                    'front_port': front_port_templates[3].pk,
-                    'front_port_position': 1,
-                },
-            ],
-        }
-
-    def test_update_object(self):
-        super().test_update_object()
-
-        # Check that the port mapping was updated after modifying the rear port template
-        front_port_template = FrontPortTemplate.objects.get(name='Front Port Template 4')
-        rear_port_template = RearPortTemplate.objects.get(name='Rear Port Template 1')
-        self.assertTrue(
-            PortTemplateMapping.objects.filter(
-                front_port=front_port_template,
-                front_port_position=1,
-                rear_port=rear_port_template,
-                rear_port_position=1,
-            ).exists()
-        )
-
 
 class ModuleBayTemplateTest(APIViewTestCases.APIViewTestCase):
     model = ModuleBayTemplate
@@ -2110,90 +2015,51 @@ class FrontPortTest(APIViewTestCases.APIViewTestCase):
             RearPort(device=device, name='Rear Port 6', type=PortTypeChoices.TYPE_8P8C),
         )
         RearPort.objects.bulk_create(rear_ports)
+
         front_ports = (
-            FrontPort(device=device, name='Front Port 1', type=PortTypeChoices.TYPE_8P8C),
-            FrontPort(device=device, name='Front Port 2', type=PortTypeChoices.TYPE_8P8C),
-            FrontPort(device=device, name='Front Port 3', type=PortTypeChoices.TYPE_8P8C),
+            FrontPort(device=device, name='Front Port 1', type=PortTypeChoices.TYPE_8P8C, rear_port=rear_ports[0]),
+            FrontPort(device=device, name='Front Port 2', type=PortTypeChoices.TYPE_8P8C, rear_port=rear_ports[1]),
+            FrontPort(device=device, name='Front Port 3', type=PortTypeChoices.TYPE_8P8C, rear_port=rear_ports[2]),
         )
         FrontPort.objects.bulk_create(front_ports)
-        PortMapping.objects.bulk_create([
-            PortMapping(device=device, front_port=front_ports[0], rear_port=rear_ports[0]),
-            PortMapping(device=device, front_port=front_ports[1], rear_port=rear_ports[1]),
-            PortMapping(device=device, front_port=front_ports[2], rear_port=rear_ports[2]),
-        ])
 
         cls.create_data = [
             {
                 'device': device.pk,
                 'name': 'Front Port 4',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'rear_ports': [
-                    {
-                        'position': 1,
-                        'rear_port': rear_ports[3].pk,
-                        'rear_port_position': 1,
-                    },
-                ],
+                'rear_port': rear_ports[3].pk,
+                'rear_port_position': 1,
             },
             {
                 'device': device.pk,
                 'name': 'Front Port 5',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'rear_ports': [
-                    {
-                        'position': 1,
-                        'rear_port': rear_ports[4].pk,
-                        'rear_port_position': 1,
-                    },
-                ],
+                'rear_port': rear_ports[4].pk,
+                'rear_port_position': 1,
             },
             {
                 'device': device.pk,
                 'name': 'Front Port 6',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'rear_ports': [
-                    {
-                        'position': 1,
-                        'rear_port': rear_ports[5].pk,
-                        'rear_port_position': 1,
-                    },
-                ],
+                'rear_port': rear_ports[5].pk,
+                'rear_port_position': 1,
             },
         ]
 
-        cls.update_data = {
-            'type': PortTypeChoices.TYPE_LC,
-            'rear_ports': [
-                {
-                    'position': 1,
-                    'rear_port': rear_ports[3].pk,
-                    'rear_port_position': 1,
-                },
-            ],
-        }
-
-    def test_update_object(self):
-        super().test_update_object()
-
-        # Check that the port mapping was updated after modifying the front port
-        front_port = FrontPort.objects.get(name='Front Port 1')
-        rear_port = RearPort.objects.get(name='Rear Port 4')
-        self.assertTrue(
-            PortMapping.objects.filter(
-                front_port=front_port,
-                front_port_position=1,
-                rear_port=rear_port,
-                rear_port_position=1,
-            ).exists()
-        )
-
     @tag('regression')  # Issue #18991
     def test_front_port_paths(self):
         device = Device.objects.first()
+        rear_port = RearPort.objects.create(
+            device=device, name='Rear Port 10', type=PortTypeChoices.TYPE_8P8C
+        )
         interface1 = Interface.objects.create(device=device, name='Interface 1')
-        rear_port = RearPort.objects.create(device=device, name='Rear Port 10', type=PortTypeChoices.TYPE_8P8C)
-        front_port = FrontPort.objects.create(device=device, name='Front Port 10', type=PortTypeChoices.TYPE_8P8C)
-        PortMapping.objects.create(device=device, front_port=front_port, rear_port=rear_port)
+        front_port = FrontPort.objects.create(
+            device=device,
+            name='Rear Port 10',
+            type=PortTypeChoices.TYPE_8P8C,
+            rear_port=rear_port,
+        )
         Cable.objects.create(a_terminations=[interface1], b_terminations=[front_port])
 
         self.add_permissions(f'dcim.view_{self.model._meta.model_name}')
@@ -2220,15 +2086,6 @@ class RearPortTest(APIViewTestCases.APIViewTestCase):
         role = DeviceRole.objects.create(name='Test Device Role 1', slug='test-device-role-1', color='ff0000')
         device = Device.objects.create(device_type=devicetype, role=role, name='Device 1', site=site)
 
-        front_ports = (
-            FrontPort(device=device, name='Front Port 1', type=PortTypeChoices.TYPE_8P8C),
-            FrontPort(device=device, name='Front Port 2', type=PortTypeChoices.TYPE_8P8C),
-            FrontPort(device=device, name='Front Port 3', type=PortTypeChoices.TYPE_8P8C),
-            FrontPort(device=device, name='Front Port 4', type=PortTypeChoices.TYPE_8P8C),
-            FrontPort(device=device, name='Front Port 5', type=PortTypeChoices.TYPE_8P8C),
-            FrontPort(device=device, name='Front Port 6', type=PortTypeChoices.TYPE_8P8C),
-        )
-        FrontPort.objects.bulk_create(front_ports)
         rear_ports = (
             RearPort(device=device, name='Rear Port 1', type=PortTypeChoices.TYPE_8P8C),
             RearPort(device=device, name='Rear Port 2', type=PortTypeChoices.TYPE_8P8C),
@@ -2241,66 +2098,19 @@ class RearPortTest(APIViewTestCases.APIViewTestCase):
                 'device': device.pk,
                 'name': 'Rear Port 4',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'front_ports': [
-                    {
-                        'position': 1,
-                        'front_port': front_ports[3].pk,
-                        'front_port_position': 1,
-                    },
-                ],
             },
             {
                 'device': device.pk,
                 'name': 'Rear Port 5',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'front_ports': [
-                    {
-                        'position': 1,
-                        'front_port': front_ports[4].pk,
-                        'front_port_position': 1,
-                    },
-                ],
             },
             {
                 'device': device.pk,
                 'name': 'Rear Port 6',
                 'type': PortTypeChoices.TYPE_8P8C,
-                'front_ports': [
-                    {
-                        'position': 1,
-                        'front_port': front_ports[5].pk,
-                        'front_port_position': 1,
-                    },
-                ],
             },
         ]
 
-        cls.update_data = {
-            'type': PortTypeChoices.TYPE_LC,
-            'front_ports': [
-                {
-                    'position': 1,
-                    'front_port': front_ports[3].pk,
-                    'front_port_position': 1,
-                },
-            ],
-        }
-
-    def test_update_object(self):
-        super().test_update_object()
-
-        # Check that the port mapping was updated after modifying the rear port
-        front_port = FrontPort.objects.get(name='Front Port 4')
-        rear_port = RearPort.objects.get(name='Rear Port 1')
-        self.assertTrue(
-            PortMapping.objects.filter(
-                front_port=front_port,
-                front_port_position=1,
-                rear_port=rear_port,
-                rear_port_position=1,
-            ).exists()
-        )
-
     @tag('regression')  # Issue #18991
     def test_rear_port_paths(self):
         device = Device.objects.first()
@@ -2586,7 +2396,7 @@ class CableTest(APIViewTestCases.APIViewTestCase):
                     'object_id': interfaces[14].pk,
                 }],
                 'label': 'Cable 4',
-                'profile': CableProfileChoices.SINGLE_1C1P,
+                'profile': CableProfileChoices.STRAIGHT_SINGLE,
             },
             {
                 'a_terminations': [{
@@ -2598,7 +2408,7 @@ class CableTest(APIViewTestCases.APIViewTestCase):
                     'object_id': interfaces[15].pk,
                 }],
                 'label': 'Cable 5',
-                'profile': CableProfileChoices.SINGLE_1C1P,
+                'profile': CableProfileChoices.STRAIGHT_SINGLE,
             },
             {
                 'a_terminations': [{
@@ -2620,9 +2430,7 @@ class CableTerminationTest(
     APIViewTestCases.ListObjectsViewTestCase,
 ):
     model = CableTermination
-    brief_fields = [
-        'cable', 'cable_end', 'connector', 'display', 'id', 'positions', 'termination_id', 'termination_type', 'url',
-    ]
+    brief_fields = ['cable', 'cable_end', 'display', 'id', 'position', 'termination_id', 'termination_type', 'url']
 
     @classmethod
     def setUpTestData(cls):