Skip to content

Commit 30c72cf

Browse files
committed
[section/property] Ensure name cannot be set None
1 parent ec16774 commit 30c72cf

File tree

2 files changed

+10
-0
lines changed

2 files changed

+10
-0
lines changed

odml/property.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -213,6 +213,11 @@ def name(self, new_name):
213213
if self.name == new_name:
214214
return
215215

216+
# Make sure name cannot be set to None or empty
217+
if not new_name:
218+
self._name = self._id
219+
return
220+
216221
curr_parent = self.parent
217222
if hasattr(curr_parent, "properties") and new_name in curr_parent.properties:
218223

odml/section.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -171,6 +171,11 @@ def name(self, new_value):
171171
if self.name == new_value:
172172
return
173173

174+
# Make sure name cannot be set to None or empty
175+
if not new_value:
176+
self._name = self._id
177+
return
178+
174179
curr_parent = self.parent
175180
if hasattr(curr_parent, "sections") and new_value in curr_parent.sections:
176181
raise KeyError("Object with the same name already exists!")

0 commit comments

Comments
 (0)