Merge pull request #19 from vitkarpov/fix-#18

[#18] implement multiple inheritance for component

Author: vitkarpov

.travis.yml

@@ -1,5 +1,4 @@
 language: node_js
-before_script: npm run-script build
 node_js:
   - 0.10
 notifications:

lib/index.js

@@ -10,6 +10,34 @@
     var gid = 0;
     var noop = function() {};
 
+    /**
+     * Returns constuctor for component with the given name.
+     * Implement the right chain of prototypes:
+     * instance of the component -> methods from decl -> base component methods
+     * @param  {String} name
+     * @return {Function}
+     */
+    var getComponentConstructor = function(name) {
+        var F = function() {
+            Component.apply(this, arguments);
+        };
+        var decl = declarations[name] || {};
+        var methods = decl.methods || {};
+
+        methods.oninit = methods.oninit || noop;
+        methods.ondestroy = methods.ondestroy || noop;
+
+        F.prototype = Object.create(Component.prototype);
+        F.prototype.constuctor = Component;
+
+        for (var method in methods) {
+            if (methods.hasOwnProperty(method)) {
+                F.prototype[method] = methods[method];
+            }
+        }
+        return F;
+    };
+
     /**
      * @namespace
      * @name jBlocks
@@ -78,6 +106,7 @@
             } catch (e) {
                 throw e;
             }
+            var Component = getComponentConstructor(name);
             var instance = new Component(node, name, props);
             var instanceId = instance.__id;
 
@@ -117,7 +146,6 @@
         this.__id = ++gid;
         this.__events = {};
         this.__handlerDomEvents = this.__handlerDomEvents.bind(this);
-        this.__bindMethods();
         this.__bindDomEvents();
 
         this.oninit();
@@ -199,25 +227,6 @@
             return null;
         },
 
-        /**
-         * Bind methods from decl to the instance
-         * @private
-         * @return {jBlocks.Component}
-         */
-        __bindMethods: function() {
-            var methods = this.__decl.methods || {};
-
-            methods.oninit = methods.oninit || noop;
-            methods.ondestroy = methods.ondestroy || noop;
-
-            for (var name in methods) {
-                if (methods.hasOwnProperty(name)) {
-                    this[name] = methods[name];
-                }
-            }
-            return this;
-        },
-
         /**
          * Bind DOM Events from decl
          * @private

tests/specs.js

@@ -69,6 +69,11 @@ describe('jblocks', function() {
 
             instance.name.should.eql('bar');
         });
+        it('should return an instance of jBlocks.Component', function() {
+            var instance = jBlocks.get(document.querySelector('.js-bar'));
+
+            (instance instanceof jBlocks.Component).should.eql(true);
+        });
     });
     describe('#destroy', function() {
         beforeEach(function() {