Fix sequence issue by new object for Isostatic nodes

src/App/PropertyStandard.cpp

@@ -667,13 +667,13 @@ PyObject *PropertyIntegerList::getPyObject(void)
 
 void PropertyIntegerList::setPyObject(PyObject *value)
 { 
-    if (PyList_Check(value)) {
+    if (PySequence_Check(value)) {
-        Py_ssize_t nSize = PyList_Size(value);
+        Py_ssize_t nSize = PySequence_Size(value);
         std::vector<long> values;
         values.resize(nSize);
 
         for (Py_ssize_t i=0; i<nSize;++i) {
-            PyObject* item = PyList_GetItem(value, i);
+            PyObject* item =  PySequence_GetItem(value, i);
             if (!PyInt_Check(item)) {
                 std::string error = std::string("type in list must be int, not ");
                 error += item->ob_type->tp_name;
@@ -688,7 +688,7 @@ void PropertyIntegerList::setPyObject(PyObject *value)
         setValue(PyInt_AsLong(value));
     }
     else {
-        std::string error = std::string("type must be int or list of int, not ");
+        std::string error = std::string("type must be int or a sequence of int, not ");
         error += value->ob_type->tp_name;
         throw Base::TypeError(error);
     }

src/Mod/Machining_Distortion/MachDistAnalysis.py

@@ -236,12 +236,13 @@ class _JobControlTaskPanel:
             
         IsoNodeObject = None
         for i in FemGui.getActiveAnalysis().Member:
-            if i.isDerivedFrom("Fem::FemSetNodesObject"):
+            if i.isDerivedFrom("App::FeaturePython"):
-                IsoNodeObject = i
+                if i.Proxy.Type == 'MachDist_IsostaticNodes':
+                    IsoNodeObject = i
         if not IsoNodeObject:
             QtGui.QMessageBox.critical(None, "Missing prerequisit","No Isostatic nodes defined in the Analysis")
             return
-        IsoNodes = IsoNodeObject.Nodes
+        IsoNodes = IsoNodeObject.IsostaticNodes
         
         filename_without_suffix = MeshObject.Name
         #current_file_name

src/Mod/Machining_Distortion/MachDistIsostatic.py

@@ -34,6 +34,8 @@ __author__ = "Juergen Riegel"
 __url__ = "http://free-cad.sourceforge.net"
 
 
+
+
 def getBoundaryCoditions(Mesh):
     BndBox = Mesh.BoundBox
     FirstLength = 10000.0
@@ -63,6 +65,15 @@ def getBoundaryCoditions(Mesh):
     print FirstIndex,SecondIndex,ThirdIndex
     return (FirstIndex,SecondIndex,ThirdIndex)
 
+def makeIsostatic(name):
+    '''makeMaterial(name): makes an Material
+    name there fore is a material name or an file name for a FCMat file'''
+    obj = FreeCAD.ActiveDocument.addObject("App::FeaturePython",name)
+    _IsostaticNodes(obj)
+    _ViewProviderIsostaticNodes(obj.ViewObject)
+    #FreeCAD.ActiveDocument.recompute()
+    return obj
+
 
 class _CommandIsostatic:
     "the MachDist Isostatic command definition"
@@ -77,12 +88,13 @@ class _CommandIsostatic:
         FreeCAD.ActiveDocument.openTransaction("Isostatic")
 
         obj = None
-        FemMesh = None
+        FemMeshObj = None
         if FemGui.getActiveAnalysis():
             for i in FemGui.getActiveAnalysis().Member:
-                if i.isDerivedFrom("Fem::FemSetNodesObject"):
+                if i.isDerivedFrom("App::FeaturePython"):
-                    obj = i
+                    if i.Proxy.Type == 'MachDist_IsostaticNodes':
-                    break
+                        obj = i
+                        break
         else: 
             return 
         
@@ -91,14 +103,14 @@ class _CommandIsostatic:
                 FemMeshObj = i
             
         if not obj:
-            FreeCADGui.doCommand("App.activeDocument().addObject('Fem::FemSetNodesObject','IsostaticNodes')")
+            FreeCADGui.addModule("MachDistIsostatic")
-            FreeCADGui.doCommand("App.activeDocument().ActiveObject.FemMesh = App.activeDocument()."+FemMeshObj.Name)
+            FreeCADGui.doCommand("MachDistIsostatic.makeIsostatic('IsostaticNodes')")
             obj = FreeCAD.activeDocument().ActiveObject
             FreeCADGui.doCommand("FemGui.getActiveAnalysis().Member = FemGui.getActiveAnalysis().Member + [App.activeDocument().ActiveObject]")
         
         #node_numbers = Fem.getBoundary_Conditions(FemMeshObj.FemMesh)
         node_numbers = getBoundaryCoditions(FemMeshObj.FemMesh)
-        obj.Nodes = node_numbers
+        obj.IsostaticNodes = node_numbers
         
         nodes = FemMeshObj.FemMesh.Nodes
         meshObj = None
@@ -135,6 +147,66 @@ class _CommandIsostatic:
         else:
             return False
        
+class _IsostaticNodes:
+    "The IsostaticNodes object"
+    def __init__(self,obj):
+        self.Type = "MachDist_IsostaticNodes"
+        obj.Proxy = self
+        obj.addProperty("App::PropertyIntegerList","IsostaticNodes","Base",
+                        "The isostatic node numbers")
+
+        
+    def execute(self,obj):
+        return
+        
+    def onChanged(self,obj,prop):
+        if prop in ["IsostaticNodes"]:
+            return
+
+    def __getstate__(self):
+        return self.Type
+
+    def __setstate__(self,state):
+        if state:
+            self.Type = state
+        
+class _ViewProviderIsostaticNodes:
+    "A View Provider for the IsostaticNodes object"
+
+    def __init__(self,vobj):
+        #vobj.addProperty("App::PropertyLength","BubbleSize","Base", str(translate("MachDist","The size of the axis bubbles")))
+        vobj.Proxy = self
+       
+    def getIcon(self):
+        import machdist_rc
+        return ":/icons/MachDist_Isostatic.svg"
+
+    def attach(self, vobj):
+        self.ViewObject = vobj
+        self.Object = vobj.Object
+  
+    def setEdit(self,vobj,mode):
+        FemMeshObj = None
+        if FemGui.getActiveAnalysis():
+            for i in FemGui.getActiveAnalysis().Member:
+                if i.isDerivedFrom("Fem::FemMeshObject"):
+                    FemMeshObj = i
+                    break
+        else: 
+            return False
+        
+        for i in FemGui.getActiveAnalysis().Member:
+            if i.isDerivedFrom("Fem::FemMeshObject"):
+                FemMeshObj = i
+        taskd = _IsostaticTaskPanel(self.Object, None,FemMeshObj)
+        taskd.obj = vobj.Object
+        taskd.update()
+        FreeCADGui.Control.showDialog(taskd)
+        return True
+    
+    def unsetEdit(self,vobj,mode):
+        FreeCADGui.Control.closeDialog()
+        return
 
  
 class _IsostaticTaskPanel:
@@ -168,7 +240,7 @@ class _IsostaticTaskPanel:
         'fills the widgets'
         OutStr = 'Isostatic Plane:\n'
         
-        IsoNodes = list(self.obj.Nodes)
+        IsoNodes = list(self.obj.IsostaticNodes)
         
         AllNodes = self.femMeshObj.FemMesh.Nodes
         GridNode1 = AllNodes[IsoNodes[0]]