From e18fb9898e0ef746f92c08376f7a53f96c23716a Mon Sep 17 00:00:00 2001
From: Yorik van Havre <yorik@uncreated.net>
Date: Fri, 18 Oct 2013 18:20:13 -0300
Subject: [PATCH] 0000893: Draft upgrade

---
 src/Mod/Draft/Draft.py | 20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

diff --git a/src/Mod/Draft/Draft.py b/src/Mod/Draft/Draft.py
index 30587ae6a..2ceadc2fa 100644
--- a/src/Mod/Draft/Draft.py
+++ b/src/Mod/Draft/Draft.py
@@ -2127,15 +2127,17 @@ def upgrade(objects,delete=False,force=None):
         result = False
         for o in objectslist:
             for w in o.Shape.Wires:
-                if w.isClosed() and DraftGeomUtils.isPlanar(w):
+                try:
                     f = Part.Face(w)
-                    if f:
-                        newobj = FreeCAD.ActiveDocument.addObject("Part::Feature","Face")
-                        newobj.Shape = f
-                        addList.append(newobj)
-                        result = True
-                        if not o in deleteList:
-                            deleteList.append(o)
+                except:
+                    pass
+                else:
+                    newobj = FreeCAD.ActiveDocument.addObject("Part::Feature","Face")
+                    newobj.Shape = f
+                    addList.append(newobj)
+                    result = True
+                    if not o in deleteList:
+                        deleteList.append(o)
         return result
 
     def makeWires(objectslist):
@@ -2287,7 +2289,7 @@ def upgrade(objects,delete=False,force=None):
             
         # no result has been obtained
         if not result:
-            msg(translate("draft", "Unable to upgrade these objects\n"))
+            msg(translate("draft", "Unable to upgrade these objects.\n"))
             
     if delete:
         names = []