Mongoose new Model() about haxe-js-kit HOT 9 CLOSED

After some digging I end up doing

var stuff = StuffManager.build(db,"Stuff");
var s : Stuff = untyped stuff();

Seems to me that the mongoose oop model is somewhat messy.

from haxe-js-kit.

Yeah, the externs are still hot :)
I've changed the signature of Document::new so that you should be able to do :

var s : Stuff = new Stuff({ foo : "bar" });
s.save(...);

There still seem to be some issues, though (trying to trace(s) for instance)...

from haxe-js-kit.

The parameters not being optional is not the issue, the issue is that the Document (super) constructor is not being called as it is missing in js.npm.macro.Model. I would put it there myself, but the problem is it does not work anyway, because you have to call the constructor on which moongoose.model() returns, which is Manager.

from haxe-js-kit.

Yeah, right...
Mongoose's OOP model is broken indeed...
I'll think about a better solution (probably a helper method...)

from haxe-js-kit.

maybe stuff.construct(?doc) return untyped this(doc)
I hate so much having to do that... :\

from haxe-js-kit.

or _new...

from haxe-js-kit.

I vote for construct ;). Thank a lot

from haxe-js-kit.

construct it is...
I updated it, feel free to close this issue.
Thanks for reporting :)

from haxe-js-kit.

Thanks!

from haxe-js-kit.