feat: enforce unique names for node registration

SomeKittens · Oct 1, 2015 · a957efb · a957efb
1 parent f79b812
commit a957efb
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/index.js b/index.js
@@ -15,10 +15,14 @@ var Gustav = (function () {
     // TODO: new type of registration that's just a singleton
     // Just calls NodeFactory and returns the symbol
     Gustav.prototype.register = function (type, name, factory) {
-        var _this = this;
         // TODO: Return some sort of object so this can be chained
         // let splitText = SplitText()
         // .addDep(fetchPageText);
+        var _this = this;
+        // Names must be unique
+        if (this.registeredNodes[type].indexOf(name)) {
+            throw new Error(name + ' already registered');
+        }
         this.registeredNodes[type].push(name);
         return function () {
             var config = [];

diff --git a/index.ts b/index.ts
@@ -34,6 +34,11 @@ class Gustav {
     // TODO: Return some sort of object so this can be chained
     // let splitText = SplitText()
     // .addDep(fetchPageText);
+
+    // Names must be unique
+    if (this.registeredNodes[type].indexOf(name)) {
+      throw new Error(name + ' already registered');
+    }
     this.registeredNodes[type].push(name);
 
     return (...config) => {