Only consider own properties in Data.Hash.

substance · May 5, 2011 · 4ead317 · 4ead317
1 parent b261345
commit 4ead317
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/data.js b/data.js
@@ -331,7 +331,7 @@
 
     // Get value at given *key*
     get: function (key) {
-      return this.data[key];
+      return this.data.hasOwnProperty(key) ? this.data[key] : undefined;
     },
 
     // Get value at given *index*

diff --git a/test/testsuite.js b/test/testsuite.js
@@ -254,6 +254,17 @@ test("Data.Hash#union", function() {
 });
 
 
+test("Only consider own properties", function() {
+  var hsh = new Data.Hash();
+
+  ok(hsh.get('toString') === undefined);
+  ok(hsh.get('toLocaleString') === undefined);
+  ok(hsh.get('watch') === undefined);
+  ok(hsh.get('hasOwnProperty') === undefined);
+});
+
+
+
 test("Data.Hash Events", function() {
   persons = new Data.Hash();
   persons.bind('set', function(key) { ok(key); });