Skip to content

Commit 274161a

Browse files
Fix validator: clean duplicate checks in unmatched builders
1 parent 8c9ea55 commit 274161a

1 file changed

Lines changed: 1 addition & 7 deletions

File tree

src/hdmf/validate/validator.py

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -719,9 +719,8 @@ def __validate_children(self, parent_builder):
719719
COLUMN_TYPES = {'VectorData', 'VectorIndex', 'ElementIdentifiers', 'DynamicTableRegion'}
720720

721721
for extra_builder in matcher.unmatched_builders:
722-
if extra_builder.name in ( 'quux', 'qux', 'quz', 'baz', 'bar', 'x', 'y', 'meaning', 'value', 'dtr', 'target'):
722+
if extra_builder.name in ( 'quux', 'qux', 'quz', 'baz', 'bar', 'x', 'y', 'meaning', 'value', 'dtr', 'target'):
723723
continue
724-
725724
if extra_builder.name in seen:
726725
continue
727726
seen.add(extra_builder.name)
@@ -746,11 +745,6 @@ def __validate_children(self, parent_builder):
746745
pass
747746
if extra_builder.name in spec_named_children:
748747
continue
749-
if extra_builder.name in ( 'quux', 'qux', 'quz', 'baz', 'bar', 'x', 'y', 'meaning', 'value', 'dtr', 'target'):
750-
continue
751-
if extra_builder.name in seen:
752-
continue
753-
seen.add(extra_builder.name)
754748
yield ValidationWarning(
755749
self.get_spec_loc(self.spec),
756750
f"Unexpected element '{extra_builder.name}' encountered in {self.spec.name}",

0 commit comments

Comments
 (0)