Skip to content

Commit c574adc

Browse files
authored
Merge pull request #7 from Spredzy/fix_update_hash
update_(safe|archive): Remove None elements
2 parents eb15e8d + 5c0c82f commit c574adc

File tree

1 file changed

+10
-3
lines changed

1 file changed

+10
-3
lines changed

c14/__init__.py

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,12 @@ def update_safe(self, uuid, name=None, description=None):
122122
:param description: Description of the safe.
123123
"""
124124

125+
data = {'name': name,
126+
'description': description}
127+
128+
data = dict((k, v) for k, v in data.iteritems() if v is not None)
125129
try:
126-
res = (self.api.storage.c14.safe(uuid)
127-
.patch({'name': name, 'description': description}))
130+
res = (self.api.storage.c14.safe(uuid).patch(data))
128131
except slumber.exceptions.HttpClientError as e:
129132
res = self.handle_error(e)
130133
except slumber.exceptions.HttpServerError as e:
@@ -217,9 +220,13 @@ def update_archive(self, safe_id, uuid, name=None, description=None):
217220
:param description: Description of the archive.
218221
"""
219222

223+
data = {'name': name,
224+
'description': description}
225+
226+
data = dict((k, v) for k, v in data.iteritems() if v is not None)
220227
try:
221228
res = (self.api.storage.c14.safe(safe_id).archive(uuid)
222-
.patch({'name': name, 'description': description}))
229+
.patch(data))
223230
except slumber.exceptions.HttpClientError as e:
224231
res = self.handle_error(e)
225232
except slumber.exceptions.HttpServerError as e:

0 commit comments

Comments
 (0)