upgrade parseAction to cleaned up version

js/erp5_loader.js

@@ -6945,29 +6945,26 @@
   // TODO: id ... is crap
   // TODO: passing empty state is not good, refactor, when adding state 
   app.generateActionObject = function (e) {
-    var element, pop, id, gadget;
-    switch (e.type) {
-    case "popupbeforeposition":
-      element = undefined;
-      id = e.target.id;
-      gadget = e.target;
-      break;
-    default:
-      element = e.target || e;
-      pop = element.getAttribute("data-rel") === null;
-      id = pop ?
-          (element.getAttribute("data-reference") || util.getPage().getAttribute("data-url")) :
-              (element.href === undefined ? util.getPage().getAttribute("data-url") :
-                  element.href.split("#")[1]);
-      gadget = document.getElementById(id);
-      break;
+    var element, id, gadget;
+
+    if (e.type === "popupbeforeposition") {
+      return {
+        "id": e.target.id,
+        "gadget": e.target
+      };
     }
 
+    element = e.target || e;
+    id = element.getAttribute("data-reference") ||
+          (element.href || "").split("#")[1] ||
+              util.getPage().getAttribute("data-url");
+    gadget = document.getElementById(id);
+
     return {
       "element": element,
       "id": id,
-      "gadget": document.getElementById(id),
-      "state": gadget ? gadget.state : {}
+      "gadget": document.getElementById(id) || gadget.getElementsByTagName("form")[0],
+      "state": (gadget || {}).state
     };
   };