fix array rule return value issue (again).

Author: icebob

CHANGELOG.md

@@ -1,9 +1,10 @@
 --------------------------------------------------
-<a name="1.0.0-beta3"></a>
-# 1.0.0-beta3 (2019-11-17)
+<a name="1.0.0-beta4"></a>
+# 1.0.0-beta4 (2019-11-17)
 
 ## Changes
 - fix optional multi rule.
+- fix array rule return value issue (again). 
 
 --------------------------------------------------
 <a name="1.0.0-beta2"></a>

lib/rules/array.js

@@ -87,11 +87,13 @@ module.exports = function({ schema, messages }, path, context) {
 		*/
 		src.push(`
 			}
-
-			return value;
 		`);
 	}
 
+	src.push(`
+		return value;
+	`);
+
 	return {
 		source: src.join("\n")
 	};

test/rules/array.spec.js

@@ -93,6 +93,22 @@ describe("Test rule: array", () => {
 
 	describe("Test sanitization", () => {
 
+		it("should untouch the checked obj", () => {
+			let schema = {
+				roles: { type: "array" }
+			};
+			let check = v.compile(schema);
+
+			const obj = {
+				roles: ["admin", "user", "moderator"]
+			};
+
+			expect(check(obj)).toEqual(true);
+			expect(obj).toEqual({
+				roles: ["admin", "user", "moderator"]
+			});
+		});
+
 		it("should trim all items", () => {
 			let schema = {
 				roles: { type: "array", items: "string|trim" }

test/typescript/rules/array.spec.ts

@@ -99,6 +99,22 @@ describe('TypeScript Definitions', () => {
 
 	describe("Test sanitization", () => {
 
+		it("should untouch the checked obj", () => {
+			let schema = {
+				roles: { type: "array" } as RuleArray
+			};
+			let check = v.compile(schema);
+
+			const obj = {
+				roles: ["admin", "user", "moderator"]
+			};
+
+			expect(check(obj)).toEqual(true);
+			expect(obj).toEqual({
+				roles: ["admin", "user", "moderator"]
+			});
+		});
+
 		it("should trim all items", () => {
 			let schema = {
 				roles: { type: "array", items: "string|trim" } as RuleArray