Extending stjs.extend works for everything except the constructor

[onigoetz]

Hi, it's me again :D

You know I'm working on a st-js bridge for React. I'm stuck on the part where I call React.createClass() automatically in stjs.extend() (see https://github.com/onigoetz/react-bridge/blob/master/src/main/js/react/stjs-react.js )

You see in this method, that we completely bypass the stjs.extend() method in favor of React.createClass()

The problem is that the return we do at the end is not taken into account because how the STJS code is generated:

The generated STJS code is:

var Panel = function() {};
stjs.extend(Panel, null, [], function(constructor, prototype) {
    prototype.render = function() {
        return "content";
    };
}, {}, {_: {"IsReactClass": {}}});

But should actually be:

var Panel = stjs.extend(function() {}, null, [], function(constructor, prototype) {
    prototype.render = function() {
        return "content";
    };
}, {}, {_: {"IsReactClass": {}}});

Even when I do _constructor.prototype = React.createClass({}).prototype; the constructor is still the original one which is not what we want...

If you accept this change of generation, I can make a pull Request with the fix.

[onigoetz]

Oops this should have been closed a long time ago, released in 3.1.5