Skip to content

Commit 5b56d9f

Browse files
committed
[property/section] Refactor init validation msg
Changing the print format of the validation error messages in Section and Property init to make them more readable.
1 parent 4927833 commit 5b56d9f

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

odml/property.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,9 @@ def __init__(self, name=None, values=None, parent=None, unit=None,
140140

141141
for err in validation.Validation(self).errors:
142142
if err.is_error:
143-
msg = "\n\t- %s %s: %s" % (err.obj, err.rank, err.msg)
143+
use_name = err.obj.name if err.obj.id != err.obj.name else None
144+
prop_formatted = "Property[id=%s|%s]" % (err.obj.id, use_name)
145+
msg = "%s\n Validation[%s]: %s" % (prop_formatted, err.rank, err.msg)
144146
print(msg)
145147

146148
def __len__(self):

odml/section.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,9 @@ def __init__(self, name=None, type=None, parent=None,
8484

8585
for err in validation.Validation(self).errors:
8686
if err.is_error:
87-
msg = "\n\t- %s %s: %s" % (err.obj, err.rank, err.msg)
87+
use_name = err.obj.name if err.obj.id != err.obj.name else None
88+
sec_formatted = "Section[id=%s|%s/%s]" % (err.obj.id, use_name, err.obj.type)
89+
msg = "%s\n Validation[%s]: %s" % (sec_formatted, err.rank, err.msg)
8890
print(msg)
8991

9092
def __repr__(self):

0 commit comments

Comments
 (0)