From 64d653bc618fd53d038f3ffdc3b3707f3b94c961 Mon Sep 17 00:00:00 2001 From: Akita Noek Date: Fri, 22 Jul 2016 09:42:00 -0400 Subject: [PATCH] Equate '' to None for foreign key references when PUTing to a job template The UI will sometimes send an empty string instead of null, we already handle this in our serializer and treat it as null, so this change brings our access system in line with the expectation that those two things should be equal. Possibly related to #3077 --- awx/main/access.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/awx/main/access.py b/awx/main/access.py index 19b4e34f70..592359a031 100644 --- a/awx/main/access.py +++ b/awx/main/access.py @@ -978,7 +978,8 @@ class JobTemplateAccess(BaseAccess): for k, v in data.items(): if hasattr(obj, k) and getattr(obj, k) != v: - if k not in field_whitelist and v != getattr(obj, '%s_id' % k, None): + if k not in field_whitelist and v != getattr(obj, '%s_id' % k, None) \ + and not (hasattr(obj, '%s_id' % k) and getattr(obj, '%s_id' % k) is None and v == ''): # Equate '' to None in the case of foreign keys return False return True