0032583: Changing a customer in the Web POS can lead to unexpected changes in the business partner properties

diff --git a/web/org.openbravo.retail.posterminal/js/data/datacustomersave.js b/web/org.openbravo.retail.posterminal/js/data/datacustomersave.js
--- a/web/org.openbravo.retail.posterminal/js/data/datacustomersave.js
+++ b/web/org.openbravo.retail.posterminal/js/data/datacustomersave.js
@@ -1,6 +1,6 @@
 /*
  ************************************************************************************
- * Copyright (C) 2012-2015 Openbravo S.L.U.
+ * Copyright (C) 2012-2016 Openbravo S.L.U.
  * Licensed under the Openbravo Commercial License version 1.0
  * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html
  * or in the legal folder of this module distribution.
@@ -31,7 +31,7 @@
       bpToSave.set('createdBy', OB.MobileApp.model.get('orgUserId'));
       if (customerId) {
         this.customer.set('posTerminal', OB.MobileApp.model.get('terminal').id);
-        bpToSave.set('json', JSON.stringify(this.customer.serializeToJSON()));
+        bpToSave.set('json', JSON.stringify(this.customer.serializeEditedToJSON()));
         bpToSave.set('c_bpartner_id', this.customer.get('id'));
       } else {
         isNew = true;
diff --git a/web/org.openbravo.retail.posterminal/js/model/businesspartner.js b/web/org.openbravo.retail.posterminal/js/model/businesspartner.js
--- a/web/org.openbravo.retail.posterminal/js/model/businesspartner.js
+++ b/web/org.openbravo.retail.posterminal/js/model/businesspartner.js
@@ -117,6 +117,33 @@
         }
       });
     },
+    adjustNames: function () {
+      var firstName = this.get('firstName'),
+          lastName = this.get('lastName');
+      if (firstName) {
+        firstName = firstName.trim();
+      }
+      if (lastName) {
+        lastName = lastName.trim();
+      }
+      this.set('firstName', firstName);
+      this.set('lastName', lastName);
+      this.set('name', firstName + (lastName ? ' ' + lastName : ''));
+    },
+    serializeEditedToJSON: function () {
+      var me = this,
+          editedBp = new OB.Model.BusinessPartner();
+      //Set entities ids: BusinessPartner, Location and User
+      editedBp.set('id', this.get('id'));
+      editedBp.set('locId', this.get('locId'));
+      editedBp.set('contactId', this.get('contactId'));
+      //Set only form attributes
+      _.each(OB.OBPOSPointOfSale.UI.customers.edit_createcustomers_impl.prototype.newAttributes, function (model) {
+        editedBp.set(model.modelProperty, me.get(model.modelProperty));
+      });
+      editedBp.adjustNames();
+      return JSON.parse(JSON.stringify(editedBp.toJSON()));
+    },
     serializeToJSON: function () {
       return JSON.parse(JSON.stringify(this.toJSON()));
     }
diff --git a/web/org.openbravo.retail.posterminal/js/pointofsale/view/subwindows/customers/components/sharedcomponents.js b/web/org.openbravo.retail.posterminal/js/pointofsale/view/subwindows/customers/components/sharedcomponents.js
--- a/web/org.openbravo.retail.posterminal/js/pointofsale/view/subwindows/customers/components/sharedcomponents.js
+++ b/web/org.openbravo.retail.posterminal/js/pointofsale/view/subwindows/customers/components/sharedcomponents.js
@@ -1,6 +1,6 @@
 /*
  ************************************************************************************
- * Copyright (C) 2012-2015 Openbravo S.L.U.
+ * Copyright (C) 2012-2016 Openbravo S.L.U.
  * Licensed under the Openbravo Commercial License version 1.0
  * You may obtain a copy of the License at http://www.openbravo.com/legal/obcl.html
  * or in the legal folder of this module distribution.
@@ -236,7 +236,7 @@
       this.waterfall('onSaveChange', {
         customer: this.model.get('customer')
       });
-      this.adjustNames(this.model.get('customer'));
+      this.model.get('customer').adjustNames();
       OB.UTIL.HookManager.executeHooks('OBPOS_BeforeCustomerSave', {
         customer: this.model.get('customer'),
         isNew: true
@@ -257,7 +257,7 @@
         getCustomerValues({
           customer: customer
         });
-        that.adjustNames(customer);
+        customer.adjustNames();
         OB.UTIL.HookManager.executeHooks('OBPOS_BeforeCustomerSave', {
           customer: customer,
           isNew: false
@@ -275,19 +275,6 @@
       });
     }
   },
-  adjustNames: function (customer) {
-    var firstName = customer.get('firstName'),
-        lastName = customer.get('lastName');
-    if (firstName) {
-      firstName = firstName.trim();
-    }
-    if (lastName) {
-      lastName = lastName.trim();
-    }
-    customer.set('firstName', firstName);
-    customer.set('lastName', lastName);
-    customer.set('name', firstName + (lastName ? ' ' + lastName : ''));
-  },
   initComponents: function () {
     this.inherited(arguments);
     this.$.bodyheader.createComponent({