From 5986f95c8dd1c9b9172ffa121564cfbee869c532 Mon Sep 17 00:00:00 2001 From: Lanrik Date: Mon, 20 Feb 2017 13:48:19 +0400 Subject: [PATCH] Rearrange code (#4908) Minor refactor. Move assignment to after `None` check. --- rest_framework/fields.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/rest_framework/fields.py b/rest_framework/fields.py index f46edef1e..2efe89610 100644 --- a/rest_framework/fields.py +++ b/rest_framework/fields.py @@ -1446,11 +1446,11 @@ class FileField(Field): return data def to_representation(self, value): - use_url = getattr(self, 'use_url', api_settings.UPLOADED_FILES_USE_URL) - if not value: return None + use_url = getattr(self, 'use_url', api_settings.UPLOADED_FILES_USE_URL) + if use_url: if not getattr(value, 'url', None): # If the file has not been saved it may not have a URL.