0013873: Autosave integration with multi-tab interface should be done

diff --git a/src-wad/src/org/openbravo/wad/javasource.javaxml b/src-wad/src/org/openbravo/wad/javasource.javaxml
--- a/src-wad/src/org/openbravo/wad/javasource.javaxml
+++ b/src-wad/src/org/openbravo/wad/javasource.javaxml
@@ -29,7 +29,7 @@
 import org.openbravo.erpCommon.ad_actionButton.*;
 </PARAMETER_TMP>
 
-
+import org.codehaus.jettison.json.JSONObject;
 import org.openbravo.erpCommon.utility.*;
 import org.openbravo.data.FieldProvider;
 import org.openbravo.utils.FormatUtilities;
@@ -613,6 +613,37 @@
         if (log4j.isDebugEnabled() &amp;&amp; myError!=null) log4j.debug(myError.getMessage());
         printPageClosePopUp(response, vars);
 </FIELDS_TMP>
+    } else if (vars.commandIn("SAVE_XHR")) {
+      <PARAMETER_TMP id="parent">String strP<PARENT_TMP id="keyParent">Almacen</PARENT_TMP> = vars.getGlobalVariable("inp<PARENT_TMP id="keyParentINP">Almacen</PARENT_TMP>", windowId + "|<PARENT_TMP id="keyParent">Almacen</PARENT_TMP>");</PARAMETER_TMP>
+      OBError myError = new OBError();
+      JSONObject result = new JSONObject();
+      String commandType = vars.getStringParameter("inpCommandType");
+      char saveType = "NEW".equals(commandType) ? 'I' : 'U';
+      try {
+        int total = saveRecord(vars, myError, saveType<PARAMETER_TMP id="parent">, strP<PARENT_TMP id="keyParent">Almacen</PARENT_TMP></PARAMETER_TMP>);
+        if (myError.isEmpty()) {
+          myError = Utility.translateError(this, vars, vars.getLanguage(), "@CODE=RowsUpdated");
+          myError.setMessage(total + " " + myError.getMessage());
+          myError.setType("Success");
+        }
+        result.put("oberror", myError.toHashMap());
+        result.put("tabid", vars.getStringParameter("tabID"));
+        result.put("redirect", strDireccion + request.getServletPath() + "?Command=" + commandType);
+      } catch (Exception e) {
+        log4j.error("Error saving record (XHR request): " + e.getMessage(), e);
+        try {
+          myError.setType("Error");
+          myError.setMessage(e.getMessage());
+        } catch (Exception e2) {
+          // Ignored
+        }
+      }
+
+      response.setContentType("application/json");
+      PrintWriter out = response.getWriter();
+      out.print(result.toString());
+      out.flush();
+      out.close();
     } else if (vars.getCommand().toUpperCase().startsWith("BUTTON") || vars.getCommand().toUpperCase().startsWith("SAVE_BUTTON")) {
       pageErrorPopUp(response);
     } else pageError(response);
diff --git a/src/org/openbravo/erpCommon/utility/OBError.java b/src/org/openbravo/erpCommon/utility/OBError.java
--- a/src/org/openbravo/erpCommon/utility/OBError.java
+++ b/src/org/openbravo/erpCommon/utility/OBError.java
@@ -18,6 +18,8 @@
  */
 package org.openbravo.erpCommon.utility;
 
+import java.util.HashMap;
+
 public class OBError {
   private String type = "";
   private String title = "";
@@ -75,4 +77,12 @@
   public boolean isConnectionAvailable() {
     return this.connectionAvailable;
   }
+
+  public HashMap<String, String> toHashMap() {
+    HashMap<String, String> o = new HashMap<String, String>();
+    o.put("title", title);
+    o.put("message", message);
+    o.put("type", type);
+    return o;
+  }
 }

diff --git a/src/org/openbravo/client/application/templates/layout.js.ftl b/src/org/openbravo/client/application/templates/layout.js.ftl
--- a/src/org/openbravo/client/application/templates/layout.js.ftl
+++ b/src/org/openbravo/client/application/templates/layout.js.ftl
@@ -212,6 +212,13 @@
   },
   
   removeTabs: function(tabs, destroyPanes) {
+    var i, tab, appFrame, tabsLength,
+        tabSet = OB.MainView.TabSet;
+
+    if(!tabs) {
+      return;
+    }
+
     if (!isc.isAn.Array(tabs)) { 
         tabs = [tabs];
     }
@@ -220,14 +227,29 @@
     // We can pass this to tabBar.removeTabs()
     tabs = this.map("getTab", tabs);
 
-    var tabsLength = tabs.length, i;
-    
+    tabsLength = tabs.length;
+
     for (i = 0; i < tabsLength; i++) {
-        OB.Layout.ViewManager.views.removeTab(tabs[i].ID);        
+      tab = tabSet.getTab(tabs[i].ID);
+      if(tab.pane.Class === "ClassicOBWindow") {
+
+        appFrame = tab.pane.appFrameWindow ||
+                   tab.pane.getAppFrameWindow();
+
+        if(appFrame && appFrame.isUserChanges) {
+          if(appFrame.validate && !appFrame.validate()) {
+            return;
+          }
+          tab.pane.saveRecord(tabs[i].ID);
+        } else {
+          OB.Layout.ViewManager.views.removeTab(tabs[i].ID);
+          this.Super("removeTabs", arguments);
+        }
+      } else {
+        OB.Layout.ViewManager.views.removeTab(tabs[i].ID);
+        this.Super("removeTabs", arguments);
+      }
     }
-  
-    this.Super("removeTabs", arguments);
-    
     OB.Layout.HistoryManager.updateHistory();
   }
 });
diff --git a/web/org.openbravo.client.application/js/classic-ob-window.js b/web/org.openbravo.client.application/js/classic-ob-window.js
--- a/web/org.openbravo.client.application/js/classic-ob-window.js
+++ b/web/org.openbravo.client.application/js/classic-ob-window.js
@@ -237,6 +237,41 @@
       return {viewId: 'ClassicOBHelp', windowId: null, tabTitle: this.tabTitle + ' - ' + OB.I18N.getLabel('UINAVBA_Help'), windowType: "X", windowName: this.formId};
     }
     return null;
+  },
+
+  saveRecord: function(/*String*/ tabID, /*Function*/ callback) {
+    var postData, reqObj,
+        appFrame = this.appFrameWindow || this.getAppFrameWindow(),
+        saveCallback = callback || this.ID + ".saveCallback(rpcResponse, data, rpcRequest)",
+        tabid = tabID || "";
+
+    postData = {};
+    OB.Utilities.addFormInputsToCriteria(postData, appFrame);
+
+    postData.Command = "SAVE_XHR";
+    postData.tabID = tabid;
+
+    reqObj = {
+      params: postData,
+      callback: saveCallback,
+      evalResult: false,
+      httpMethod : "POST",
+      useSimpleHttp : true,
+      actionURL: OB.Application.contextUrl + postData.mappingName
+    };
+    isc.RPCManager.sendRequest(reqObj);
+  },
+
+  saveCallback: function(rpcResponse, data, rpcRequest) {
+    var result = eval("("+data+")"),
+        appFrame = this.appFrameWindow || this.getAppFrameWindow();
+    if(result && result.oberror) {
+      if(result.oberror.type === "Success") {
+        appFrame.isUserChanges = false;
+        OB.MainView.TabSet.removeTab(OB.MainView.TabSet.getTab(result.tabid));
+      } else {
+        appFrame.location.href = result.redirect;
+      }
+    }
   }
-  
 });

diff --git a/web/org.openbravo.userinterface.smartclient/js/utilities.js b/web/org.openbravo.userinterface.smartclient/js/utilities.js
--- a/web/org.openbravo.userinterface.smartclient/js/utilities.js
+++ b/web/org.openbravo.userinterface.smartclient/js/utilities.js
@@ -98,16 +98,72 @@
   return str1 === str2;
 };
 
+OB.Utilities.processLogoutQueue = function() {
+  var q = OB.Utilities.logoutWorkQueue,
+          qElement, result, tab, tabID;
+
+  if(q && q.length === 0) {
+    return;
+  }
+
+  if(typeof arguments[1] === "string") {
+    // The 2nd parameter in a sendRequest callback is the 'data' parameter
+	// http://www.smartclient.com/docs/7.0rc2/a/b/c/go.html#type..RPCCallback
+    result = eval("("+arguments[1]+")");
+    if(result && result.oberror) {
+      if(result.oberror.type === "Error") {
+        tab = OB.MainView.TabSet.getTab(arguments[2].params.tabID);
+        if(tab) {
+          tab.pane.getAppFrameWindow().location.href = result.redirect;
+        }
+        q = [];
+        return;
+      }
+    }
+  }
+
+  // Process one element at the time,
+  // the save callbacks will empty the queue
+  qElement = q.pop();
+  if(qElement.func && qElement.args !== undefined) {
+    qElement.func.apply(qElement.self, qElement.args);
+  }
+};
 
 // ** {{{ OB.Utilities.logout }}} **
 // Logout from the application, removes server side session info and redirects
 // the client to the
 // Login page.
 OB.Utilities.logout = function() {
-  var redirectCallback = function() {
-    window.location.replace(OB.Application.loginPage);
-  };
-  OB.RemoteCallManager.call('org.openbravo.client.application.LogOutActionHandler', {}, {}, redirectCallback);
+	
+  var q = OB.Utilities.logoutWorkQueue = [],
+      i, tabs = OB.MainView.TabSet.tabs,
+      tabsLength = tabs.length, appFrame;
+
+  // Push the logout process to the 'end' of the queue
+  q.push({func: OB.RemoteCallManager.call,
+	      self: this,
+		  args: ['org.openbravo.client.application.LogOutActionHandler', {}, {}, function() {
+		            window.location.href = OB.Application.loginPage;
+		          }
+                ]});
+
+  for(i = 0; i < tabsLength; i++) {
+    if(tabs[i].pane.Class === "ClassicOBWindow") {
+      appFrame = tabs[i].pane.appFrameWindow || tabs[i].pane.getAppFrameWindow();
+      if(appFrame && appFrame.isUserChanges) {
+        if(appFrame.validate && !appFrame.validate()) {
+          q = [];
+          return;
+        }
+        q.push({func: tabs[i].pane.saveRecord,
+                self: tabs[i].pane,
+                args: [tabs[i].ID, OB.Utilities.processLogoutQueue]});
+      }
+    }
+  }
+
+  OB.Utilities.processLogoutQueue();
 };
 
 
@@ -204,10 +260,14 @@
 // 
 // Parameters:
 // * {{{criteria}}}: the current criteria object.
-OB.Utilities.addFormInputsToCriteria = function (/* Object */criteria) {
-  var elementsLength = document.frmMain.elements.length;
+// * {{{win}}}: (Optional) a reference to the global context (window) where to get the document
+//   and functions are located, if not passed, the current window is used
+OB.Utilities.addFormInputsToCriteria = function (/*Object*/ criteria, /*Window*/ win) {
+  var d = (win && win.document ? win.document : null) || window.document,
+      elementsLength = (d.frmMain ? d.frmMain.elements.length : 0),
+      inputValue = (win && win.inputValue ? win.inputValue : null) || window.inputValue;
   for (var i = 0; i < elementsLength; i++) {
-    var elem = document.frmMain.elements[i];
+    var elem = d.frmMain.elements[i];
     if(elem.name) {
       criteria[elem.name] = inputValue(elem);
     }
@@ -216,8 +276,8 @@
   // the form can have an organization field,
   // in the server it is used to determine the accessible orgs
   // TODO: make this optional or make it possible to set the orgid html id
-  if (document.frmMain.inpadOrgId) {
-    criteria[OB.Constants.ORG_PARAMETER] = inputValue(document.frmMain.inpadOrgId);
+  if (d.frmMain.inpadOrgId) {
+    criteria[OB.Constants.ORG_PARAMETER] = inputValue(d.frmMain.inpadOrgId);
   }
 };