hasOne query based on referenced table

[Locke]

I have a hasOne relationship and want to query based on the referenced table.

var Person = db.define('person', {
    name : String
});
var Animal = db.define('animal', {
    name : String
});
Animal.hasOne("owner", Person); // creates column 'owner_id' in 'animal' table

I want to do do something like:

SELECT * FROM animal JOIN person ON animal.owner_id = person.id WHERE person.name = "John";

Any ideas how I can achieve this? In the drivers there is opts.merge that seems to be exactly what I am searching for, but I can not find any documentation of it.

I already tried to pass find the options "__merge" and "extra", but ended up in a rather complex statement and failed to pass the condition.

It would be great if I could just do:

Animal.find(owner: Person.find("name": "John"))

[dresende]

I think there was a question before about this, I'm not sure, I can't remember how we solved it. This is interesting, I'm going to analyse it.

[dresende]

What do you think about this:

// .findByOwner() assuming you defined .hasOne("owner", ...)
Animal.findByOwner({ name: "John" }, function (err, animals) {
    // ...
});

[dresende]

I just added a commit with this new functionality. Please try it.

[Locke]

Thank you, that what I was looking for.

I now want to apply conditions to the animals.

My first try was chaining it, but then in findByX the cb stays null, wich results in Model.js that the argument[2] is an object (Model.js:237) and so the conditions are reset to null (Model.js:249) which then results in TypeError: Cannot call method 'hasOwnProperty' of null in line 251.

I got arround that problem adding if(arguments[i] === null) continue; and could do:

Animal.findByOwner({ name: "John" }).find({name: "myCatName"}).run(function (err, animals) {
    // ...
});

That works fine for me, but I've to mention that this could result into database errors if you have models with identical columnnames.

[dresende]

The problem with models with same property names is solvable, I'm going to pull an update that should fix this.

[Locke]

Thanks, that works fine.