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

Fix bug where copying elements didn't ensure that material usages were unique

Browse Source
This commit is contained in:
Dion Moult
2021-10-22 12:09:57 +11:00
parent a8c99cebaf
commit a5b8728aec
2 changed files with 12 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ifcopenshell-python/ifcopenshell/api/root/copy_class.py
+4
View File
@@ -33,6 +33,10 @@ class Usecase:
continue continue
elif inverse.is_a("IfcRelFillsElement"): elif inverse.is_a("IfcRelFillsElement"):
continue continue
elif inverse.is_a("IfcRelAssociatesMaterial") and "Usage" in inverse.RelatingMaterial.is_a():
inverse = ifcopenshell.util.element.copy(self.file, inverse)
inverse.RelatingMaterial = ifcopenshell.util.element.copy(self.file, inverse.RelatingMaterial)
inverse.RelatedObjects = [to_element]
else: else:
for i, value in enumerate(inverse): for i, value in enumerate(inverse):
if value == from_element: if value == from_element:
src/ifcopenshell-python/test/api/root/test_copy_class.py
+8
View File
@@ -108,3 +108,11 @@ class TestCopyClass(test.bootstrap.IFC4):
ifcopenshell.api.run("void.add_filling", self.file, opening=opening, element=door) ifcopenshell.api.run("void.add_filling", self.file, opening=opening, element=door)
new = ifcopenshell.api.run("root.copy_class", self.file, product=door) new = ifcopenshell.api.run("root.copy_class", self.file, product=door)
assert not new.FillsVoids assert not new.FillsVoids
def test_copying_material_set_usages(self):
element = ifcopenshell.api.run("root.create_entity", self.file, ifc_class="IfcWall")
material = self.file.createIfcMaterialLayerSetUsage()
self.file.createIfcRelAssociatesMaterial(RelatedObjects=[element], RelatingMaterial=material)
new = ifcopenshell.api.run("root.copy_class", self.file, product=element)
assert new.HasAssociations[0].RelatingMaterial != element.HasAssociations[0].RelatingMaterial
assert new.HasAssociations[0].RelatingMaterial.is_a("IfcMaterialLayerSetUsage")