added undefined and null support fixes #105

canjs · Jun 2, 2018 · 9ebeadf · 9ebeadf
1 parent ca389b5
commit 9ebeadf
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/reflections/type/type-test.js b/reflections/type/type-test.js
@@ -82,6 +82,8 @@ QUnit.test("isMapLike", function(){
 QUnit.test("isMoreListLikeThanMapLike", function(){
 	QUnit.equal(typeReflections.isMoreListLikeThanMapLike({}), false, "Object");
 	QUnit.equal(typeReflections.isMoreListLikeThanMapLike([]), true, "Array");
+	QUnit.equal(typeReflections.isMoreListLikeThanMapLike(undefined), "undefined", "undefined");
+	QUnit.equal(typeReflections.isMoreListLikeThanMapLike(null), "undefined", "null");
 });
 
 QUnit.test("isObservableLike", function(){

diff --git a/reflections/type/type.js b/reflections/type/type.js
@@ -453,6 +453,9 @@ module.exports = {
 		if(obj instanceof Array) {
 			return true;
 		}
+		if(obj === undefined || obj === null) {
+			return "undefined";
+		}
 		var value = obj[canSymbol.for("can.isMoreListLikeThanMapLike")];
 		if(value !== undefined) {
 			return value;