Merge pull request #2 from truststamp/fixed-validator-promise-check

Fixed validationResult type check
truststamp · Sep 26, 2017 · 99fef18 · CezaryDanielNowak · Sep 26, 2017 · 99fef18
2 parents 4a9d046 + dc4497c
commit 99fef18
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/atom.js b/atom.js
@@ -161,7 +161,9 @@
 				var validationResult = validation[validationKey](value, keyOrMap);
 				if (typeof validationResult === 'boolean') {
 					return !validationResult;
-				} else if (validationResult instanceof Promise) {
+				// better check if validationResult is a Promise; instanceof Promise was false on Safari
+				} else if (typeof validationResult.then === 'function'
+					&& typeof validationResult.catch === 'function') {
 					promises[validationKey] = new Promise(function(resolve, reject) {
 						validationResult
 						.then(resolve, function() {