TemplatePyMod: fix DocumentObject for newer FreeCAD versions (FreeCAD#27662)

* TemplatePyMod: fix DocumentObject for newer FreeCAD versions

original bug report:
https://forum.freecad.org/viewtopic.php?p=873546

* TemplatePyMod: fix FeaturePython example for newer FreeCAD versions

Author: bdieterm

src/Mod/TemplatePyMod/DocumentObject.py

@@ -28,9 +28,14 @@ def propertyChanged(self,prop):
 
     def __getattr__(self, attr):
         if attr !="__object__" and hasattr(self.__object__,attr):
-            return getattr(self.__object__,attr)
-        else:
-            return object.__getattribute__(self,attr)
+            # Methods like "getSubObject" are called from the C++ code if they exist in
+            # this class (DocumentObject).
+            # Our __object__ also has a method with the same name ("getSubObject"),
+            # but it does not require the extra first argument "obj" which is the same
+            # as self.__object__. So we cannot map these methods 1:1.
+            if attr not in ("getSubObject", "getSubObjects", "getLinkedObject"):
+                return getattr(self.__object__,attr)
+        return object.__getattribute__(self,attr)
     def __setattr__(self, attr, value):
         if attr !="__object__" and hasattr(self.__object__,attr):
             setattr(self.__object__,attr,value)
@@ -91,12 +96,11 @@ def touch(self):
     def purgeTouched(self):
         "removes the to-be-recomputed flag of this object"
         return self.__object__.purgeTouched()
-    def __setstate__(self,value):
-        """allows saving custom attributes of this object as strings, so
-        they can be saved when saving the FreeCAD document"""
+    def loads(self,value):
+        """Called during document restore."""
         return None
-    def __getstate__(self):
-        """reads values previously saved with __setstate__()"""
+    def dumps(self):
+        """Called during document saving."""
         return None
     @property
     def PropertiesList(self):
@@ -224,12 +228,11 @@ def getGroupOfProperty(self,attr):
     def getDocumentationOfProperty(self,attr):
         "returns the documentation string of a given property"
         return self.__vobject__.getDocumentationOfProperty(attr)
-    def __setstate__(self,value):
-        """allows saving custom attributes of this object as strings, so
-        they can be saved when saving the FreeCAD document"""
+    def loads(self,value):
+        """Called during document restore."""
         return None
-    def __getstate__(self):
-        """reads values previously saved with __setstate__()"""
+    def dumps(self):
+        """Called during document saving."""
         return None
     @property
     def Annotation(self):

src/Mod/TemplatePyMod/FeaturePython.py

@@ -96,14 +96,14 @@ def getIcon(self):
             "   #######      "};
             """
 
-    def __getstate__(self):
+    def dumps(self):
         ''' When saving the document this object gets stored using Python's cPickle module.
         Since we have some un-pickable here -- the Coin stuff -- we must define this method
         to return a tuple of all pickable objects or None.
         '''
         return None
 
-    def __setstate__(self,state):
+    def loads(self,state):
         ''' When restoring the pickled object from document we have the chance to set some
         internals here. Since no data were pickled nothing needs to be done here.
         '''
@@ -314,10 +314,10 @@ def getIcon(self):
             "   #######      "};
             """
 
-    def __getstate__(self):
+    def dumps(self):
         return None
 
-    def __setstate__(self,state):
+    def loads(self,state):
         return None
 
 def makeOctahedron():
@@ -405,14 +405,14 @@ def getIcon(self):
             "   #######      "};
             """
 
-    def __getstate__(self):
+    def dumps(self):
         ''' When saving the document this object gets stored using Python's cPickle module.
         Since we have some un-pickable here -- the Coin stuff -- we must define this method
         to return a tuple of all pickable objects or None.
         '''
         return None
 
-    def __setstate__(self,state):
+    def loads(self,state):
         ''' When restoring the pickled object from document we have the chance to set some
         internals here. Since no data were pickled nothing needs to be done here.
         '''
@@ -498,14 +498,14 @@ def getIcon(self):
             "   #######      "};
             """
 
-    def __getstate__(self):
+    def dumps(self):
         ''' When saving the document this object gets stored using Python's cPickle module.
         Since we have some un-pickable here -- the Coin stuff -- we must define this method
         to return a tuple of all pickable objects or None.
         '''
         return None
 
-    def __setstate__(self,state):
+    def loads(self,state):
         ''' When restoring the pickled object from document we have the chance to set some
         internals here. Since no data were pickled nothing needs to be done here.
         '''
@@ -562,10 +562,10 @@ def updateData(self, fp, prop):
             p = fp.getPropertyByName("p2")
             self.trl2.translation=(p.x,p.y,p.z)
 
-    def __getstate__(self):
+    def dumps(self):
         return None
 
-    def __setstate__(self,state):
+    def loads(self,state):
         return None
 
 def makeMolecule():
@@ -612,10 +612,10 @@ def updateData(self, fp, prop):
                 self.coords.point.setValues(pts)
                 self.lines.numVertices.setValues(ver)
 
-    def __getstate__(self):
+    def dumps(self):
         return None
 
-    def __setstate__(self,state):
+    def loads(self,state):
         return None
 
 def makeCircleSet():
@@ -659,10 +659,10 @@ def __init__(self, obj):
     def updateData(self, fp, prop):
         print("prop updated:",prop)
 
-    def __getstate__(self):
+    def dumps(self):
         return None
 
-    def __setstate__(self,state):
+    def loads(self,state):
         return None
 
 def makeEnumTest():
@@ -723,3 +723,4 @@ def makeDistanceBolt():
     DistanceBolt(bolt)
     bolt.ViewObject.Proxy=0
     doc.recompute()
+