From 1497e8eaa4adcae90b3ea424483e1a878120e454 Mon Sep 17 00:00:00 2001
From: Yorik van Havre <yorik@uncreated.net>
Date: Tue, 24 Sep 2013 22:38:03 -0300
Subject: [PATCH] Arch: Fixes in IFC importer (spaces didn't import correctly)

---
 src/Mod/Arch/ArchSpace.py  | 5 +++--
 src/Mod/Arch/ArchWindow.py | 1 -
 src/Mod/Arch/importIFC.py  | 3 ++-
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/Mod/Arch/ArchSpace.py b/src/Mod/Arch/ArchSpace.py
index cbd20b315..638750135 100644
--- a/src/Mod/Arch/ArchSpace.py
+++ b/src/Mod/Arch/ArchSpace.py
@@ -29,11 +29,11 @@ import FreeCAD,FreeCADGui,ArchComponent,ArchCommands,math,Draft
 from DraftTools import translate
 from PyQt4 import QtCore
 
-def makeSpace(objects=None):
+def makeSpace(objects=None,name="Space"):
     """makeSpace([objects]): Creates a space object from the given objects. Objects can be one
     document object, in which case it becomes the base shape of the space object, or a list of
     selection objects as got from getSelectionEx(), or a list of tuples (object, subobjectname)"""
-    obj = FreeCAD.ActiveDocument.addObject("Part::FeaturePython","Space")
+    obj = FreeCAD.ActiveDocument.addObject("Part::FeaturePython",name)
     _Space(obj)
     _ViewProviderSpace(obj.ViewObject)
     if objects:
@@ -44,6 +44,7 @@ def makeSpace(objects=None):
             objects[0].ViewObject.hide()
         else:
             obj.Proxy.addSubobjects(obj,objects)
+    return obj
         
 def addSpaceBoundaries(space,subobjects):
     """addSpaceBoundaries(space,subobjects): adds the given subobjects to the given space"""
diff --git a/src/Mod/Arch/ArchWindow.py b/src/Mod/Arch/ArchWindow.py
index 3915b895e..194201c43 100644
--- a/src/Mod/Arch/ArchWindow.py
+++ b/src/Mod/Arch/ArchWindow.py
@@ -124,7 +124,6 @@ class _Window(ArchComponent.Component):
         self.createGeometry(obj)
         
     def onChanged(self,obj,prop):
-        print prop
         self.hideSubobjects(obj,prop)
         if prop in ["Base","WindowParts"]:
             self.createGeometry(obj)
diff --git a/src/Mod/Arch/importIFC.py b/src/Mod/Arch/importIFC.py
index 0556b1136..e5cb1531d 100644
--- a/src/Mod/Arch/importIFC.py
+++ b/src/Mod/Arch/importIFC.py
@@ -464,8 +464,9 @@ def makeSpace(entity,shape=None,name="Space"):
                 body = FreeCAD.ActiveDocument.addObject("Part::Feature",name+"_body")
                 body.Shape = shape
                 space.Base = body
+                body.ViewObject.hide()
                 if DEBUG: print "made space object  ",entity,":",space
-                return site
+                return space
     except:
         return None