From bf47e7cae3c359f18b785bec4b28deda7dfa0c64 Mon Sep 17 00:00:00 2001 From: Jeremy Stretch Date: Thu, 4 Oct 2018 14:50:57 -0400 Subject: [PATCH] #2487: Require the 'brief' parameter to evaluate True --- netbox/project-static/js/forms.js | 2 +- netbox/utilities/api.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/netbox/project-static/js/forms.js b/netbox/project-static/js/forms.js index 55ecea977..6cb621071 100644 --- a/netbox/project-static/js/forms.js +++ b/netbox/project-static/js/forms.js @@ -82,7 +82,7 @@ $(document).ready(function() { } if ($(parent).val() || $(parent).attr('nullable') == 'true') { - var api_url = child_field.attr('api-url') + '&limit=0&brief'; + var api_url = child_field.attr('api-url') + '&limit=0&brief=1'; var disabled_indicator = child_field.attr('disabled-indicator'); var initial_value = child_field.attr('initial'); var display_field = child_field.attr('display-field') || 'name'; diff --git a/netbox/utilities/api.py b/netbox/utilities/api.py index 56ad405d9..9b9dabef5 100644 --- a/netbox/utilities/api.py +++ b/netbox/utilities/api.py @@ -197,7 +197,7 @@ class ModelViewSet(_ModelViewSet): # If 'brief' has been passed as a query param, find and return the nested serializer for this model, if one # exists request = self.get_serializer_context()['request'] - if 'brief' in request.query_params: + if request.query_params.get('brief', False): serializer_class = get_serializer_for_model(self.queryset.model, prefix='Nested') if serializer_class is not None: return serializer_class