mirror of
https://github.com/netbox-community/netbox.git
synced 2024-05-10 07:54:54 +00:00
Merge branch 'develop' into develop-2.9
This commit is contained in:
@ -1,6 +1,6 @@
|
||||
# NetBox v2.8
|
||||
|
||||
## v2.8.7 (FUTURE)
|
||||
## v2.8.8 (FUTURE)
|
||||
|
||||
### Bug Fixes
|
||||
|
||||
|
@ -376,7 +376,7 @@ class DeviceViewSet(CustomFieldModelViewSet):
|
||||
if device.platform is None:
|
||||
raise ServiceUnavailable("No platform is configured for this device.")
|
||||
if not device.platform.napalm_driver:
|
||||
raise ServiceUnavailable("No NAPALM driver is configured for this device's platform ().".format(
|
||||
raise ServiceUnavailable("No NAPALM driver is configured for this device's platform {}.".format(
|
||||
device.platform
|
||||
))
|
||||
|
||||
|
@ -611,16 +611,6 @@ class DynamicModelChoiceMixin:
|
||||
filter = django_filters.ModelChoiceFilter
|
||||
widget = APISelect
|
||||
|
||||
def filter_queryset(self, data):
|
||||
field_name = getattr(self, 'to_field_name') or 'pk'
|
||||
# If multiple values have been provided, use only the last.
|
||||
if type(data) in (list, tuple):
|
||||
data = data[-1]
|
||||
filter = self.filter(
|
||||
field_name=field_name
|
||||
)
|
||||
return filter.filter(self.queryset, data)
|
||||
|
||||
def get_bound_field(self, form, field_name):
|
||||
bound_field = BoundField(form, self, field_name)
|
||||
|
||||
@ -628,7 +618,13 @@ class DynamicModelChoiceMixin:
|
||||
# will be populated on-demand via the APISelect widget.
|
||||
data = bound_field.value()
|
||||
if data:
|
||||
self.queryset = self.filter_queryset(data)
|
||||
field_name = getattr(self, 'to_field_name') or 'pk'
|
||||
filter = self.filter(field_name=field_name)
|
||||
try:
|
||||
self.queryset = filter.filter(self.queryset, data)
|
||||
except TypeError:
|
||||
# Catch any error caused by invalid initial data passed from the user
|
||||
self.queryset = self.queryset.none()
|
||||
else:
|
||||
self.queryset = self.queryset.none()
|
||||
|
||||
@ -658,17 +654,6 @@ class DynamicModelMultipleChoiceField(DynamicModelChoiceMixin, forms.ModelMultip
|
||||
filter = django_filters.ModelMultipleChoiceFilter
|
||||
widget = APISelectMultiple
|
||||
|
||||
def filter_queryset(self, data):
|
||||
field_name = getattr(self, 'to_field_name') or 'pk'
|
||||
# Normalize data to a list
|
||||
if type(data) not in (list, tuple):
|
||||
data = [data]
|
||||
filter = self.filter(
|
||||
field_name=field_name,
|
||||
lookup_expr='in'
|
||||
)
|
||||
return filter.filter(self.queryset, data)
|
||||
|
||||
|
||||
class LaxURLField(forms.URLField):
|
||||
"""
|
||||
|
Reference in New Issue
Block a user