Glassvision/IfcOpenShell
1
0
Fork 0
mirror of https://github.com/IfcOpenShell/IfcOpenShell.git synced 2026-09-11 22:31:55 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix incorrect handling of prohibited specifications

Browse Source
This commit is contained in:
Dion Moult
2024-03-30 00:13:02 +11:00
parent 1ab24e8362
commit 0065c5fba8
2 changed files with 27 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifctester/ifctester/ids.py
+19 -19
View File
@@ -259,31 +259,31 @@ class Specification:
self.applicable_entities.append(element)
for facet in self.requirements:
result = facet(element)
if not bool(result):
self.failed_entities.add(element)
facet.failures.append({"element": element, "reason": str(result)})
for facet in self.requirements:
if facet.cardinality == "required":
facet.status = not bool(facet.failures)
elif facet.cardinality == "optional":
facet.status = True
elif facet.cardinality == "prohibited":
facet.status = bool(facet.failures)
is_pass = bool(result)
if facet.cardinality == "prohibited":
is_pass = not is_pass
if self.maxOccurs != 0: # This is a required or optional specification
if not is_pass:
self.failed_entities.add(element)
facet.failures.append({"element": element, "reason": str(result)})
else: # This is a prohibited specification
if is_pass:
self.failed_entities.add(element)
facet.failures.append({"element": element, "reason": str(result)})
self.status = True
if self.minOccurs != 0:
for facet in self.requirements:
facet.status = not bool(facet.failures)
if not facet.status:
self.status = False
if self.minOccurs != 0: # Required specification
if not self.applicable_entities:
self.status = False
for facet in self.requirements:
facet.status = False
elif self.failed_entities:
self.status = False
elif self.minOccurs == 0 and self.maxOccurs != 0:
if self.failed_entities:
self.status = False
elif self.maxOccurs == 0:
if (len(self.applicable_entities)) > 0:
elif self.maxOccurs == 0: # Prohibited specification
if self.applicable_entities and not self.requirements:
self.status = False
def get_usage(self):