From 098d250552f9ab7317f62e71ee52b27191923944 Mon Sep 17 00:00:00 2001 From: Arthur Hanson Date: Wed, 4 Sep 2024 13:00:31 -0700 Subject: [PATCH] 17354 change to use kwargs --- netbox/extras/models/customfields.py | 46 ++++++++++++---------------- 1 file changed, 20 insertions(+), 26 deletions(-) diff --git a/netbox/extras/models/customfields.py b/netbox/extras/models/customfields.py index 09fdf2872..7a0a08614 100644 --- a/netbox/extras/models/customfields.py +++ b/netbox/extras/models/customfields.py @@ -524,36 +524,30 @@ class CustomField(CloningMixin, ExportTemplatesMixin, ChangeLoggedModel): # Object elif self.type == CustomFieldTypeChoices.TYPE_OBJECT: model = self.related_object_type.model_class() - if for_csv_import: - field = CSVModelChoiceField( - queryset=model.objects.all(), - required=required, - initial=initial, - ) - else: - field = DynamicModelChoiceField( - queryset=model.objects.all(), - required=required, - initial=initial, - query_params=self.related_object_filter - ) + field_class = CSVModelChoiceField if for_csv_import else DynamicModelChoiceField + kwargs = { + 'queryset': model.objects.all(), + 'required': required, + 'initial': initial, + } + if not for_csv_import: + kwargs['query_params'] = self.related_object_filter + + field = field_class(**kwargs) # Multiple objects elif self.type == CustomFieldTypeChoices.TYPE_MULTIOBJECT: model = self.related_object_type.model_class() - if for_csv_import: - field = CSVModelMultipleChoiceField( - queryset=model.objects.all(), - required=required, - initial=initial, - ) - else: - field = DynamicModelMultipleChoiceField( - queryset=model.objects.all(), - required=required, - initial=initial, - query_params=self.related_object_filter - ) + field_class = CSVModelMultipleChoiceField if for_csv_import else DynamicModelMultipleChoiceField + kwargs = { + 'queryset': model.objects.all(), + 'required': required, + 'initial': initial, + } + if not for_csv_import: + kwargs['query_params'] = self.related_object_filter + + field = field_class(**kwargs) # Text else: