Fix deep clone for forms with nested objects

Author: santigarcor

package.json

@@ -1,6 +1,6 @@
 {
   "name": "laravel-form-js",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "A javascript representation of a form",
   "main": "dist/Form.js",
   "scripts": {

src/Form.js

@@ -17,7 +17,7 @@ class Form {
    */
   constructor(data, method = 'post', clearAfterResponse = false) {
     this.originalData = data;
-    Object.assign(this, data);
+    Object.assign(this, JSON.parse(JSON.stringify(data)));
     this.method = method;
     this.clearAfterResponse = clearAfterResponse;
     this.errors = new FormErrors();
@@ -39,7 +39,7 @@ class Form {
    * @return {void}
    */
   reset() {
-    Object.assign(this, this.originalData);
+    Object.assign(this, JSON.parse(JSON.stringify(this.originalData)));
     this.errors.clear();
   }
 

tests/form.test.js

@@ -28,6 +28,15 @@ test('can reset the form data to its original data state', () => {
   expect(form.name).toBe('test');
 });
 
+test('if the original data has an object it doesnt change the original data', () => {
+  let form = Form.makeFrom({name: {text: 'name'}});
+  form.name.text = 'new name';
+  expect(form.name).toEqual({text: 'new name'});
+  expect(form.originalData.name).toEqual({text: 'name'});
+  form.reset();
+  expect(form.name).toEqual({text: 'name'});
+});
+
 test('can change the form http method', () => {
   let form = Form.makeFrom({name: 'test'});
   form.name = 'new name';