fix error on command line input data from active command

Author: mboscolo

Oldstuff/OldGeneric/arc.py

@@ -346,13 +346,13 @@ def throughAngle(self, angle):
         return _val
 
     def getEndpoints(self):
-        """Return where the two endpoints for the arc-segment lie.
+        """Return where the two end points for the arc-segment lie.
 
-getEndpoints(self)
-
-This function returns two tuples, each containing the x-y coordinates
-of the arc endpoints. The first tuple corresponds to the endpoint at
-the start_angle, the second to the endpoint at the end_angle.
+            getEndpoints(self)
+            
+            This function returns two tuples, each containing the x-y coordinates
+            of the arc end points. The first tuple corresponds to the end point at
+            the start_angle, the second to the end point at the end_angle.
         """
         _cx, _cy = self.__center.getCoords()
         _r = self.__radius

PythonCAD/Generic/Kernel/GeoEntity/arc.py

@@ -254,6 +254,7 @@ def getProjection(self,fromPoint):
             return pj1 # ######################## adding return value for pj2
         else:
             return None
+        
     def GetTangentPoint(self,x,y,outx,outy):
         """
             Get the tangent from an axternal point
@@ -454,7 +455,7 @@ def __str__(self):
             str(self.center), str(self.radius), str(self.startAngle), str(self.endAngle))
         return msg
 
-    def test_angle(s, e, a):
+    def test_angle(self,s, e, a):
         """
             Returns if an angle lies between the start and end angle of an arc.
             s: arc start angle

PythonCAD/Generic/Kernel/GeoEntity/geometricalentity.py

@@ -32,7 +32,7 @@ class GeometricalEntity(dict):
     """
     def __init__(self, kw, argNameType):
         """
-            argv name must be created befor init the base class
+            argv name must be created before init the base class
         """
         if kw is None and argNameType is None:
             return

PythonCAD/Interface/Command/icommand.py

@@ -164,25 +164,26 @@ def addTextEvent(self, value):
         """
             compute imput from text
         """
-        if value=="":
+        if str(value)=="":
             self.kernelCommand.applyDefault()
             self.applyCommand()
             return
-        elif value.upper()=="UNDO":
+        elif str(value).upper()=="UNDO":
             #TODO: perform a back operation to the command
             return
-        elif value.upper()=="REDO":
+        elif str(value).upper()=="REDO":
             #TODO: perform a forward operation to the command
             return
         else:
             try:
-                tValue=self.decodeText(value)
+                tValue=self.decodeText(str(value))
                 self.addMauseEvent(tValue[0], tValue[1], tValue[2], tValue[3], tValue[4], correct=None)
             except PyCadWrongImputData, msg:
                 print "Problem on ICommand.addTextEvent"
                 self.updateInput(msg)
                 self.updateInput(self.kernelCommand.activeMessage)
                 return
+            
     def applyDefault(self):
         """
             apply the default value command
@@ -297,18 +298,24 @@ def decodeText(self, value):
         entitys=None
         text=None
         angle=None
+        value=str(value)
+        
+        def niceReturn():
+            return (point,entitys, distance,angle, text)
         try:
             try:
                 raise self.kernelCommand.activeException()(None)
             except ExcPoint:
                 if value.find(',')>-1:              # ABSOLUTE CARTESIAN INPUT
                     x, y=value.split(',')
                     point=Point(convertLengh(x), convertLengh(y))
+                    return niceReturn()
                 elif value.find(';')>-1:            # RELATIVE CARTESIAN INPUT
                     x, y=value.split(';')
                     x=self.scene.fromPoint.getx()+convertLengh(x)
                     y=self.scene.fromPoint.gety()+convertLengh(y)
                     point=Point(x, y)
+                    return niceReturn()
                 elif value.find('>')>-1:
                     ang, distance=value.split('>')
                     ang=convertAngle(ang)
@@ -318,6 +325,7 @@ def decodeText(self, value):
                     x=self.scene.fromPoint.getx()+float(x)
                     y=self.scene.fromPoint.gety()+float(y)
                     point=Point(x, y)
+                    return niceReturn()
                 else: # DISTANCE+ANGLE FROM SCENE set coordinate based on distance input and angle from mouse position on the scene
                     d=float(value)
                     pX=self.scene.mouseOnSceneX
@@ -333,24 +341,29 @@ def decodeText(self, value):
                     x=self.scene.fromPoint.getx()+d*math.cos(a)
                     y=self.scene.fromPoint.gety()+d*math.sin(a)
                     point=Point(x, y)
+                    return niceReturn()
             except (ExcEntity,ExcMultiEntity):
                 entitys=self.getIdsString(value)
+                return niceReturn()
             except ExcEntityPoint:
                 #(4@10,20)
                 id, p=value.split('@')
                 x, y=p.split(',')
                 point=Point(float(x), float(y))
                 entitys=self.getIdsString(id)
-                return
+                return niceReturn()
             except (ExcLenght, ExcInt, ExcBool):
                 distance=value
+                return niceReturn()
             except(ExcAngle):
                 angle=value
+                return niceReturn()
             except(ExcText):
                 text=value
+                return niceReturn()
         except:
             raise PyCadWrongImputData("BaseCommand : Wrong imput parameter for the command")
-        return (point,entitys, distance,angle, text)
+        return niceReturn()
 
     def getIdsString(self, value):
         """