# HG changeset patch
# User Laurent Bessard
# Date 1372110837 -7200
# Node ID d63541200d8e42b5919fcb7b9ba70af37edf78c1
# Parent  6bbe937999560e5b755d4d215b3675f41c33ca91
Added support for set flag for optional variables activation in axis public struct

diff -r 6bbe93799956 -r d63541200d8e etherlab/EthercatCIA402Slave.py
--- a/etherlab/EthercatCIA402Slave.py	Mon Jun 24 21:03:22 2013 +0200
+++ b/etherlab/EthercatCIA402Slave.py	Mon Jun 24 23:53:57 2013 +0200
@@ -133,23 +133,20 @@
     def GetVariableLocationTree(self):
         axis_name = self.CTNName()
         current_location = self.GetCurrentLocation()
-        children = [{"name": "%s Network Position" % (axis_name),
+        children = [{"name": name_frmt % (axis_name),
                      "type": LOCATION_VAR_INPUT,
                      "size": "W",
-                     "IEC_type": "UINT",
-                     "var_name": axis_name,
-                     "location": "%%IW%s" % (".".join(map(str, current_location))),
+                     "IEC_type": iec_type,
+                     "var_name": var_name_frmt % axis_name,
+                     "location": location_frmt % (
+                            ".".join(map(str, current_location))),
                      "description": "",
-                     "children": []},
-                    {"name": "%s Axis Ref" % (axis_name),
-                     "type": LOCATION_VAR_INPUT,
-                     "size": "W",
-                     "IEC_type": "AXIS_REF",
-                     "var_name": axis_name,
-                     "location": "%%IW%s.402" % (".".join(map(str, current_location))),
-                     "description": "",
-                     "children": []}]
-        children.extend(self.CTNParent.GetDeviceLocationTree(self.GetSlavePos(), current_location, axis_name))
+                     "children": []}
+                    for name_frmt, iec_type, var_name_frmt, location_frmt in
+                        [("%s Network Position", "UINT", "%s_pos", "%%IW%s"),
+                         ("%s Axis Ref", "AXIS_REF", "%s", "%%IW%s.402")]]
+        children.extend(self.CTNParent.GetDeviceLocationTree(
+                            self.GetSlavePos(), current_location, axis_name))
         return  {"name": axis_name,
                  "type": LOCATION_CONFNODE,
                  "location": self.GetFullIEC_Channel(),
@@ -253,6 +250,7 @@
             # Param is optional variables section enable flag
             extra_node_variable_infos = EXTRA_NODE_VARIABLES_DICT.get(param_name)
             if extra_node_variable_infos is not None:
+                param_name += "Enabled"
                 
                 if not param["value"]:
                     continue
@@ -278,7 +276,7 @@
                             extra_variables_publish.append(template % locals())
             
             # Set AxisRef public struct member value if defined
-            elif param["value"] is not None:
+            if param["value"] is not None:
                 param_value = ({True: "1", False: "0"}[param["value"]]
                                if param["type"] == "boolean"
                                else str(param["value"]))