Make getObject("", false, obj) return obj rather than undefined.

Matches 1.9 behavior and fixes many dojox/mvc/tests failures. Refs #16291.
JoeNemo · May 21, 2014 · 2957d1b · 2957d1b
1 parent f41414d
commit 2957d1b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/_base/lang.js b/_base/lang.js
@@ -216,7 +216,7 @@ define(["./kernel", "../has", "../sniff"], function(dojo, has){
 			// context: Object?
 			//		Optional. Object to use as root of path. Defaults to
 			//		'dojo.global'. Null may be passed.
-			return getProp(name.split("."), create, context); // Object
+			return getProp(name ? name.split(".") : [], create, context); // Object
 		},
 
 		exists: function(name, obj){

diff --git a/tests/_base/lang.js b/tests/_base/lang.js
@@ -43,6 +43,10 @@ define(["doh/main", "dojo/_base/array", "dojo/_base/lang", "dojo/_base/kernel"
 
 			// strangely, parser does this
 			t.assertEqual("undefined", typeof lang.getObject("./TestWidget"), 'typeof lang.getObject("./TestWidget")');
+
+			// empty path should return the same object
+			t.assertEqual(test, lang.getObject("", false, test));
+			t.assertEqual(kernel.global, lang.getObject(""));
 		},
 
 		function setObject(t){