From 65019e261180a456ae30b0217a2ac2c034ed4984 Mon Sep 17 00:00:00 2001 From: Keith Tsui Date: Thu, 17 Mar 2016 19:01:43 -0400 Subject: [PATCH] fix exclude_if_none to actually be a None check and allow for boolean properties with false to pass through. Also set default to True because it doesn't make sense to have nulls all over the place as default --- jsonobject/base_properties.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/jsonobject/base_properties.py b/jsonobject/base_properties.py index cceb5fb..8098d3f 100644 --- a/jsonobject/base_properties.py +++ b/jsonobject/base_properties.py @@ -18,7 +18,7 @@ class JsonProperty(object): type_config = None def __init__(self, default=Ellipsis, name=None, choices=None, - required=False, exclude_if_none=False, validators=None, + required=False, exclude_if_none=True, validators=None, verbose_name=None, type_config=None): validators = validators or () self.name = name @@ -93,7 +93,7 @@ def doc_type(self): return self def exclude(self, value): - return self.exclude_if_none and not value + return self.exclude_if_none and value is None def empty(self, value): return value is None