Fix error messaging

Author: Shuwen Qian

src/mm-repeater/index.html

@@ -110,6 +110,7 @@
 			<mm-repeater id="withData">
 				<template preserve-content>
 					<mm-input name="name" validation="alpha"></mm-input>
+					<mm-input name="number"></mm-input>
 					<mm-dropdown name="firstOption">
 						<mm-list-item>Test Item 1</mm-list-item>
 						<mm-list-item>Test Item 2</mm-list-item>
@@ -137,6 +138,10 @@
 					},
 					errorMessage: 'Name is not in the list of accepted values'
 				},
+				'number': {
+					validation: 'int',
+					errorMessage: 'Not an integer'
+				},
 				'firstOption': {
 					validation: function() {
 						return true;

src/mm-repeater/mm-repeater.js

@@ -143,7 +143,9 @@
 
 		validateRow: function(row) {
 			var keys = Object.keys(row);
-			var error = keys.reduce(function(sum, key) {
+			var errorMessage = "";
+
+			var valid = keys.reduce(function(sum, key) {
 				var validation = DataUtils.getPathValue(key+'.validation', this.config),
 					valid = true;
 
@@ -152,24 +154,25 @@
 						valid = validation.apply(this.config[key], [row[key], row, row._ref]);
 						break;
 					case "string":
-						valid = Validator.rules[item](value);
+						validatef = Validator.rules[validation];
+						if(validatef) valid = validatef(row[key]);
 						break;
 					default:
 						break;
 				}
-				return sum && valid;
 
+				if(!valid) {
+					var message = DataUtils.getPathValue(key+'.errorMessage', this.config);
+					errorMessage += ' '+message;
+				}
+				return sum && valid;
 			}.bind(this), true);
-			var errorMessage = keys.reduce(function(sum, key) {
-				var message = DataUtils.getPathValue(key+'.errorMessage', this.config);
-				return sum + " " + (message || "");
-			}.bind(this), "");
 
 			var prefix = 'data.' + this.data.indexOf(row) + '.';
-			this.set(prefix+'error', error);
+			this.set(prefix+'error', !valid);
 			this.set(prefix+'errorMessage', errorMessage.trim());
 
-			return error;
+			return valid;
 		},
 
 		validation: this.validate,