Fix problem when using FormData

santigarcor · santigarcor · commit 280e244b4c45 · 2017-10-11T18:57:19.000-05:00
Related to laravel/framework#13457
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "laravel-form-js",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "A javascript representation of a form",
   "main": "dist/Form.js",
   "scripts": {
diff --git a/src/Form.js b/src/Form.js
@@ -56,6 +56,10 @@ class Form {
       container.set(field, this[field]);
     }
 
+    if (this.method != 'post') {
+      container.set('_method', this.method);
+    }
+
     return container.data;
   }
 
@@ -94,7 +98,7 @@ class Form {
     }
 
     return new Promise((resolve, reject) => {
-      this.axios[this.method](url, this.data(dataType))
+      this.axios.post(url, this.data(dataType))
         .then(response => {
           this.formSubmitSucceded(response);
           resolve(response);
diff --git a/tests/form.test.js b/tests/form.test.js
@@ -45,19 +45,18 @@ test('can retrieve the errors for a specific field', () => {
 
 
 test('it retrieves the form data as a json', () => {
-  let form = Form.makeFrom({name: 'test', email: 'test@mail.com'});
+  let form = Form.makeFrom({name: 'test', email: 'test@mail.com'}, 'put');
 
-  expect(form.data('json')).toEqual({name: 'test', email: 'test@mail.com'});
+  expect(form.data('json')).toEqual({name: 'test', email: 'test@mail.com', _method: 'put'});
 });
 
 test('it retrieves the form data as a FormData', () => {
-  let form = Form.makeFrom({name: 'test', email: 'test@mail.com'});
+  let form = Form.makeFrom({name: 'test', email: 'test@mail.com'}, 'patch');
 
-  let data = new FormData();
-  data.append('name', 'test');
-  data.append('email', 'test@mail.com');
-
-  expect(form.data('form-data')).toEqual(data);
+  let data = form.data('form-data');
+  expect(data.get('name') == form.name).toBeTruthy();
+  expect(data.get('email') == form.email).toBeTruthy();
+  expect(data.get('_method') == form.method).toBeTruthy();
 });
 
 test('can submit the form and use promises to receive the results', () => {

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "laravel-form-js",`
`3`		`- "version": "1.0.1",`
	`3`	`+ "version": "1.0.2",`
`4`	`4`	`"description": "A javascript representation of a form",`
`5`	`5`	`"main": "dist/Form.js",`
`6`	`6`	`"scripts": {`
Original file line number	Diff line number	Diff line change
`@@ -56,6 +56,10 @@ class Form {`
`56`	`56`	`container.set(field, this[field]);`
`57`	`57`	`}`
`58`	`58`
	`59`	`+ if (this.method != 'post') {`
	`60`	`+ container.set('_method', this.method);`
	`61`	`+ }`
	`62`	`+`
`59`	`63`	`return container.data;`
`60`	`64`	`}`
`61`	`65`
`@@ -94,7 +98,7 @@ class Form {`
`94`	`98`	`}`
`95`	`99`
`96`	`100`	`return new Promise((resolve, reject) => {`
`97`		`- this.axios[this.method](url, this.data(dataType))`
	`101`	`+ this.axios.post(url, this.data(dataType))`
`98`	`102`	`.then(response => {`
`99`	`103`	`this.formSubmitSucceded(response);`
`100`	`104`	`resolve(response);`