Component defaultLoader should use hasOwnProperty for isRegistered check

asereware · Apr 8, 2015 · 04596e8 · 04596e8
1 parent 1768108
commit 04596e8
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/spec/components/defaultLoaderBehaviors.js b/spec/components/defaultLoaderBehaviors.js
@@ -18,6 +18,14 @@ describe('Components: Default loader', function() {
         expect(ko.components.isRegistered(testComponentName)).toBe(false);
     });
 
+    it('Allows registering component names that may conflict with properties on Object.prototype', function() {
+        var prototypeProperty = 'toString';
+
+        expect(ko.components.isRegistered(prototypeProperty)).toBe(false);
+        ko.components.register(prototypeProperty, {});
+        expect(ko.components.isRegistered(prototypeProperty)).toBe(true);
+    });
+
     it('Throws if you try to register a component that is already registered', function() {
         ko.components.register(testComponentName, {});
 

diff --git a/src/components/defaultLoader.js b/src/components/defaultLoader.js
@@ -22,16 +22,16 @@
         }
 
         defaultConfigRegistry[componentName] = config;
-    }
+    };
 
     ko.components.isRegistered = function(componentName) {
-        return componentName in defaultConfigRegistry;
-    }
+        return defaultConfigRegistry.hasOwnProperty(componentName);
+    };
 
     ko.components.unregister = function(componentName) {
         delete defaultConfigRegistry[componentName];
         ko.components.clearCachedDefinition(componentName);
-    }
+    };
 
     ko.components.defaultLoader = {
         'getConfig': function(componentName, callback) {