From 5b74eadbb41459509bb2f25bf9172da64bbc0bc3 Mon Sep 17 00:00:00 2001
From: hansi90 <grob.hans90@gmail.com>
Date: Wed, 11 Oct 2017 15:02:11 +0200
Subject: [PATCH 1/7] New field: file upload (fieldFile).

---
 src/fields/core/fieldFile.vue  | 45 +++++++++++++++++++++++++++
 src/fields/core/fieldInput.vue | 57 +++++++++++++++++-----------------
 2 files changed, 73 insertions(+), 29 deletions(-)
 create mode 100644 src/fields/core/fieldFile.vue

diff --git a/src/fields/core/fieldFile.vue b/src/fields/core/fieldFile.vue
new file mode 100644
index 00000000..1cae4969
--- /dev/null
+++ b/src/fields/core/fieldFile.vue
@@ -0,0 +1,45 @@
+<template lang="pug">
+.wrapper
+	input.form-control(
+		id="ts01",
+		type="file",
+		:name="schema.inputName",
+		@change="schema.onChange || null",	
+		:accept="schema.accept",
+		:multiple="schema.multiple",
+		:placeholder="schema.placeholder",
+		:readonly="schema.readonly",
+		:required="schema.required",
+		:disabled="disabled",)
+</template>
+
+<script>
+import abstractField from "../abstractField";
+import fecha from "fecha";
+
+export default {
+	mixins: [abstractField]
+};
+
+</script>
+
+<style lang="sass">
+	.vue-form-generator .field-input {
+		.wrapper {
+			width: 100%;
+		}
+		input[type="radio"] {
+			width: 100%;
+		}
+		input[type="color"] {
+			width: 60px;
+		}
+		input[type="range"] {
+			padding: 0;
+		}
+
+		.helper {
+			margin: auto 0.5em;
+		}
+	}
+</style>
diff --git a/src/fields/core/fieldInput.vue b/src/fields/core/fieldInput.vue
index 7ba183f9..972efc8f 100644
--- a/src/fields/core/fieldInput.vue
+++ b/src/fields/core/fieldInput.vue
@@ -5,7 +5,7 @@
 		:type="schema.inputType",
 		:value="value",
 		@input="value = $event.target.value",
-		@change="onChange",
+		@change="schema.onChange || null",
 		:disabled="disabled",
 		:accept="schema.accept",
 		:alt="schema.alt",
@@ -37,22 +37,17 @@
 </template>
 
 <script>
-	import abstractField from "../abstractField";
-	import fecha from "fecha";
+import abstractField from "../abstractField";
+import fecha from "fecha";
 
-	export default {
-		mixins: [ abstractField ],
-		methods: {
-			onChange(event){
-				if (this.schema.inputType === "file") {
-					this.value = event.target.files;
-				}
-			},
-			
-			formatValueToField(value) {
-				if (value != null) {
-					let dt;
-					switch(this.schema.inputType){
+export default {
+	mixins: [abstractField],
+	methods: {
+
+		formatValueToField(value) {
+			if (value != null) {
+				let dt;
+				switch (this.schema.inputType) {
 					case "date":
 						dt = this.schema.format ? fecha.parse(value, this.schema.format) : new Date(value);
 						return fecha.format(dt, "YYYY-MM-DD");
@@ -62,16 +57,16 @@
 					case "datetime-local":
 						dt = this.schema.format ? fecha.parse(value, this.schema.format) : new Date(value);
 						return fecha.format(dt, "YYYY-MM-DDTHH:mm:ss");
-					}
 				}
-				
-				return value;
-			},
+			}
 
-			formatValueToModel(value) {
-				if (value != null) {
-					let m;
-					switch (this.schema.inputType){
+			return value;
+		},
+
+		formatValueToModel(value) {
+			if (value != null) {
+				let m;
+				switch (this.schema.inputType) {
 					case "date":
 						m = fecha.parse(value, "YYYY-MM-DD");
 						if (m !== false) {
@@ -100,16 +95,20 @@
 						}
 						break;
 					case "number":
-						return Number(value);					
+						return Number(value);
 					case "range":
 						return Number(value);
-					}
 				}
-
-				return value;
 			}
+
+			return value;
 		}
-	};
+	},
+
+	created () {
+		console.warn("The 'file' type in input field is deprecated. Use 'file' field instead.");	
+	}
+};
 
 </script>
 

From 98bc6b66ed74ecd34509b6f235c07f9ab76b8108 Mon Sep 17 00:00:00 2001
From: hansi90 <grob.hans90@gmail.com>
Date: Wed, 11 Oct 2017 16:06:00 +0200
Subject: [PATCH 2/7] Test fixed.

---
 src/fields/core/fieldFile.vue            |  4 +-
 test/unit/specs/fields/fieldFile.spec.js | 78 ++++++++++++++++++++++++
 2 files changed, 80 insertions(+), 2 deletions(-)
 create mode 100644 test/unit/specs/fields/fieldFile.spec.js

diff --git a/src/fields/core/fieldFile.vue b/src/fields/core/fieldFile.vue
index 1cae4969..70e37dcd 100644
--- a/src/fields/core/fieldFile.vue
+++ b/src/fields/core/fieldFile.vue
@@ -1,10 +1,10 @@
 <template lang="pug">
 .wrapper
 	input.form-control(
-		id="ts01",
+		id="getFieldID(schema)",
 		type="file",
 		:name="schema.inputName",
-		@change="schema.onChange || null",	
+		@change="schema.onChange || null",
 		:accept="schema.accept",
 		:multiple="schema.multiple",
 		:placeholder="schema.placeholder",
diff --git a/test/unit/specs/fields/fieldFile.spec.js b/test/unit/specs/fields/fieldFile.spec.js
new file mode 100644
index 00000000..8b6e5296
--- /dev/null
+++ b/test/unit/specs/fields/fieldFile.spec.js
@@ -0,0 +1,78 @@
+import { expect } from "chai";
+import { createVueField, checkAttribute } from "../util";
+
+import Vue from "vue";
+import fieldFile from "src/fields/core/fieldFile.vue";
+
+Vue.component("fieldFile", fieldFile);
+
+let el, vm, field;
+
+function createField(test, schema = {}, model = null, disabled = false, options) {
+	[el, vm, field] = createVueField(test, "fieldFile", schema, model, disabled, options);
+}
+
+describe("fieldFile.vue", function () {
+
+	describe("check template", () => {
+		let schema = {
+			type: "file",
+			label: "Upload",
+			inputName: "testupload",
+			placeholder: "Field placeholder",
+			readonly: false,
+			required: false,
+			disabled: false,
+			multiple: true,
+			accept: "image/*"
+		};
+		let model = {};
+		let attributes = ["disabled", "placeholder", "readonly"];
+		let input;
+
+		before(() => {
+			createField(this, schema, model, false);
+			input = el.getElementsByTagName("input")[0];
+			field.schema.inputType = "file";
+		});
+
+		it("should contain an input text element", () => {
+			expect(field).to.be.exist;
+			expect(field.$el).to.be.exist;
+
+			expect(input).to.be.defined;
+			expect(input.type).to.be.equal("file");
+			expect(input.classList.contains("form-control")).to.be.true;
+		});
+
+		describe("check optional attribute", () => {
+			attributes.forEach(function (name) {
+				it("should set " + name, function (done) {
+					checkAttribute(name, vm, input, field, schema, done);
+				});
+			});
+
+			it("should set name", () => {
+				expect(input.name).to.be.equal("testupload");
+			});
+
+			it("should change name", () => {
+				expect(input.name).to.be.equal("testupload");
+			});
+
+			it("should set required", () => {
+				expect(input.required).to.be.false;
+			});
+
+			it("should set multiple", () => {
+				expect(input.multiple).to.be.exist;
+			});
+
+			it("should set accept", () => {
+				expect(input.accept).to.be.equal("image/*");
+			});
+
+		});
+
+	});
+});

From 89ee169621b10124481ea83d0ba50bad42198c92 Mon Sep 17 00:00:00 2001
From: hansi90 <grob.hans90@gmail.com>
Date: Wed, 11 Oct 2017 16:39:22 +0200
Subject: [PATCH 3/7] Refinement

---
 src/fields/core/fieldFile.vue            | 1 -
 test/unit/specs/fields/fieldFile.spec.js | 4 ----
 2 files changed, 5 deletions(-)

diff --git a/src/fields/core/fieldFile.vue b/src/fields/core/fieldFile.vue
index 70e37dcd..6903c2f8 100644
--- a/src/fields/core/fieldFile.vue
+++ b/src/fields/core/fieldFile.vue
@@ -15,7 +15,6 @@
 
 <script>
 import abstractField from "../abstractField";
-import fecha from "fecha";
 
 export default {
 	mixins: [abstractField]
diff --git a/test/unit/specs/fields/fieldFile.spec.js b/test/unit/specs/fields/fieldFile.spec.js
index 8b6e5296..5f53e601 100644
--- a/test/unit/specs/fields/fieldFile.spec.js
+++ b/test/unit/specs/fields/fieldFile.spec.js
@@ -56,10 +56,6 @@ describe("fieldFile.vue", function () {
 				expect(input.name).to.be.equal("testupload");
 			});
 
-			it("should change name", () => {
-				expect(input.name).to.be.equal("testupload");
-			});
-
 			it("should set required", () => {
 				expect(input.required).to.be.false;
 			});

From daefe3893ccecb4cb829db02ab7a3c7e3e9e1a7f Mon Sep 17 00:00:00 2001
From: hansi90 <grob.hans90@gmail.com>
Date: Wed, 11 Oct 2017 16:42:40 +0200
Subject: [PATCH 4/7] Refinement

---
 src/fields/core/fieldFile.vue | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/src/fields/core/fieldFile.vue b/src/fields/core/fieldFile.vue
index 6903c2f8..e9ac6d8c 100644
--- a/src/fields/core/fieldFile.vue
+++ b/src/fields/core/fieldFile.vue
@@ -27,16 +27,6 @@ export default {
 		.wrapper {
 			width: 100%;
 		}
-		input[type="radio"] {
-			width: 100%;
-		}
-		input[type="color"] {
-			width: 60px;
-		}
-		input[type="range"] {
-			padding: 0;
-		}
-
 		.helper {
 			margin: auto 0.5em;
 		}

From b0baf00f62136253028f51154d12805ff4f5552a Mon Sep 17 00:00:00 2001
From: hansi90 <grob.hans90@gmail.com>
Date: Thu, 12 Oct 2017 10:42:45 +0200
Subject: [PATCH 5/7] Improve onChange in file field.

---
 src/fields/core/fieldFile.vue | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/src/fields/core/fieldFile.vue b/src/fields/core/fieldFile.vue
index e9ac6d8c..ad4f2058 100644
--- a/src/fields/core/fieldFile.vue
+++ b/src/fields/core/fieldFile.vue
@@ -4,7 +4,7 @@
 		id="getFieldID(schema)",
 		type="file",
 		:name="schema.inputName",
-		@change="schema.onChange || null",
+		@change="onChange",
 		:accept="schema.accept",
 		:multiple="schema.multiple",
 		:placeholder="schema.placeholder",
@@ -15,9 +15,18 @@
 
 <script>
 import abstractField from "../abstractField";
+import { isFunction } from "lodash";
 
 export default {
-	mixins: [abstractField]
+	mixins: [abstractField],
+	methods: {
+		onChange(){
+			if(isFunction(this.schema.onChanged)){
+				// Schema has defined onChange method.
+				this.schema.onChanged.call(this, this.model, this.schema, event, this);
+			}
+		}
+	}
 };
 
 </script>

From c4e4a529b51117555b932afd62b4f339ae973d95 Mon Sep 17 00:00:00 2001
From: hansi90 <grob.hans90@gmail.com>
Date: Thu, 12 Oct 2017 10:51:16 +0200
Subject: [PATCH 6/7] Fix lint.

---
 .eslintrc.js          |  3 ++-
 src/formGenerator.vue | 12 ++++++------
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index 49da56a2..df57cea6 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -21,7 +21,8 @@ module.exports = {
     "rules": {
         "indent": [
             "warn",
-            "tab"
+            "tab",
+            { SwitchCase: 1 }
         ],
         "quotes": [
             "warn",
diff --git a/src/formGenerator.vue b/src/formGenerator.vue
index 47ae7b0b..c6894b1d 100644
--- a/src/formGenerator.vue
+++ b/src/formGenerator.vue
@@ -211,12 +211,12 @@ div.vue-form-generator(v-if='schema != null')
 				}
 
 				switch (relevantType) {
-				case "button":
-				case "submit":
-				case "reset":
-					return false;
-				default:
-					return true;
+					case "button":
+					case "submit":
+					case "reset":
+						return false;
+					default:
+						return true;
 				}
 			},
 

From 9bfbb4efd2602ed5fefc6b02afa6e2386fefd9c1 Mon Sep 17 00:00:00 2001
From: hansi90 <grob.hans90@gmail.com>
Date: Thu, 12 Oct 2017 11:04:54 +0200
Subject: [PATCH 7/7] Rename fieldFile to fieldUpload.

---
 examples/post-form/main.js                         | 14 +++++++++++---
 src/fields/core/{fieldFile.vue => fieldUpload.vue} |  0
 .../{fieldFile.spec.js => fieldUpload.spec.js}     | 10 +++++-----
 3 files changed, 16 insertions(+), 8 deletions(-)
 rename src/fields/core/{fieldFile.vue => fieldUpload.vue} (100%)
 rename test/unit/specs/fields/{fieldFile.spec.js => fieldUpload.spec.js} (85%)

diff --git a/examples/post-form/main.js b/examples/post-form/main.js
index 7cc44221..fd5c4708 100644
--- a/examples/post-form/main.js
+++ b/examples/post-form/main.js
@@ -7,7 +7,7 @@ var vm = new Vue({
 	},
 
 	methods: {
-		prettyJSON: function(json) {
+		prettyJSON: function (json) {
 			if (json) {
 				json = JSON.stringify(json, undefined, 4);
 				json = json.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;");
@@ -74,7 +74,7 @@ var vm = new Vue({
 					required: true,
 					hint: "Minimum 6 characters",
 					validator: VueFormGenerator.validators.string
-				},	
+				},
 				{
 					type: "input",
 					inputType: "email",
@@ -83,7 +83,7 @@ var vm = new Vue({
 					inputName: "email",
 					placeholder: "User's e-mail address",
 					validator: VueFormGenerator.validators.email
-				},					
+				},
 				{
 					type: "select",
 					label: "Skills",
@@ -100,6 +100,14 @@ var vm = new Vue({
 						"VueJS"
 					],
 					validator: VueFormGenerator.validators.string
+				}, {
+					type: "upload",
+					label: "Photo",
+					model: "photo",
+					inputName: "photo",
+					onChanged(model, schema, event) {
+						console.log(model, schema, event);
+					}
 				},
 				{
 					type: "checkbox",
diff --git a/src/fields/core/fieldFile.vue b/src/fields/core/fieldUpload.vue
similarity index 100%
rename from src/fields/core/fieldFile.vue
rename to src/fields/core/fieldUpload.vue
diff --git a/test/unit/specs/fields/fieldFile.spec.js b/test/unit/specs/fields/fieldUpload.spec.js
similarity index 85%
rename from test/unit/specs/fields/fieldFile.spec.js
rename to test/unit/specs/fields/fieldUpload.spec.js
index 5f53e601..a12d984f 100644
--- a/test/unit/specs/fields/fieldFile.spec.js
+++ b/test/unit/specs/fields/fieldUpload.spec.js
@@ -2,21 +2,21 @@ import { expect } from "chai";
 import { createVueField, checkAttribute } from "../util";
 
 import Vue from "vue";
-import fieldFile from "src/fields/core/fieldFile.vue";
+import fieldUpload from "src/fields/core/fieldUpload.vue";
 
-Vue.component("fieldFile", fieldFile);
+Vue.component("fieldUpload", fieldUpload);
 
 let el, vm, field;
 
 function createField(test, schema = {}, model = null, disabled = false, options) {
-	[el, vm, field] = createVueField(test, "fieldFile", schema, model, disabled, options);
+	[el, vm, field] = createVueField(test, "fieldUpload", schema, model, disabled, options);
 }
 
-describe("fieldFile.vue", function () {
+describe("fieldUpload.vue", function () {
 
 	describe("check template", () => {
 		let schema = {
-			type: "file",
+			type: "upload",
 			label: "Upload",
 			inputName: "testupload",
 			placeholder: "Field placeholder",