From 39faea6ef22668446ca87bdbb44d32a8a7ad676a Mon Sep 17 00:00:00 2001 From: Arthur Date: Tue, 31 Mar 2026 10:34:34 -0700 Subject: [PATCH] cleanup --- netbox/extras/api/serializers_/scripts.py | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/netbox/extras/api/serializers_/scripts.py b/netbox/extras/api/serializers_/scripts.py index f52086409..5a6a7bc8e 100644 --- a/netbox/extras/api/serializers_/scripts.py +++ b/netbox/extras/api/serializers_/scripts.py @@ -42,16 +42,10 @@ class ScriptModuleSerializer(ValidatedModelSerializer): # For multipart requests, nested serializer fields (data_source, data_file) are # silently dropped by DRF's HTML parser, so also check initial_data for raw values. - # These checks must run before super().validate() calls full_clean(), which would - # otherwise surface confusing unique-constraint errors for empty file_path values. has_data_file = data.get('data_file') or self.initial_data.get('data_file') has_data_source = data.get('data_source') or self.initial_data.get('data_source') - if upload_file and has_data_file: - raise serializers.ValidationError( - _("Cannot upload a file and sync from an existing file.") - ) - if upload_file and has_data_source: + if upload_file and (has_data_file or has_data_source): raise serializers.ValidationError( _("Cannot upload a file and sync from a data source.") )