Modules and Javascript Inheritance

[satori]

Hello. I am having trouble understanding how javascript inheritence
and nodejs modules work.
The code below works ok for me if it is one module, but not when in
sperate modules.
I do not understand why. Any help would be appreciated.

######## main.js ########
var assert = require("assert");
var BaseObject = require("./base").BaseObject;
var ObjectA = require("./a").ObjectA;
var objA = new ObjectA();
assert.ok(objA instanceof BaseObject); // FAILS!

######## base.js ########
function BaseObject() {};
exports.BaseObject = BaseObject;

######## a.js ########
var BaseObject = require("./base").BaseObject;
function ObjectA() {
BaseObject.call(this);
};
process.inherits(ObjectA, BaseObject);
exports.ObjectA = ObjectA;

[christkv]

Hi Satori

I had the same problem and it happened when node 0.1.28 was released
(I think it has something to do with the new way the modules are
loaded). I solved it by adding a className variable in my objects and
then using that to check the type of object instead of instanceof

exports.BaseObject = function() {
this.className = "BaseObject";
}

assert(objA.className == "BaseObject")

Anyone know any other way ?

Christian

[Felix Geisendörfer]

Sorry about that. I got this wrong when I simplified the module
system, but this patch should fix it:

http://github.com/felixge/node/commit/4718208a138a68a6b0047d6734130981b827e9ee

Let me know what you think ryan,
--fg

[satori]

Thanks! I thought was doing something incorrect.

On Feb 11, 10:14 pm, Felix Geisendörfer <fe...@debuggable.com> wrote:
> Sorry about that. I got this wrong when I simplified the module
> system, but this patch should fix it:
>

> http://github.com/felixge/node/commit/4718208a138a68a6b0047d673413098...

[Ryan Dahl]

On Thu, Feb 11, 2010 at 4:32 AM, satori <blake....@gmail.com> wrote:
> Thanks! I thought was doing something incorrect.
>
> On Feb 11, 10:14 pm, Felix Geisendörfer <fe...@debuggable.com> wrote:
>> Sorry about that. I got this wrong when I simplified the module
>> system, but this patch should fix it:
>>
>> http://github.com/felixge/node/commit/4718208a138a68a6b0047d673413098...
>>
>> Let me know what you think ryan,

Thanks Felix, landed in b02b54e.