added 'all' method to hasMany in abstract-class.js to correctly support a 'many' collection. added .idea to .gitignore to ignore WebStorm projects.

This commit is contained in:
Robb Lovell 2013-01-17 11:16:52 -08:00
parent fb8d0acc80
commit 6f9c495720
2 changed files with 19 additions and 0 deletions

1
.gitignore vendored
View File

@ -3,5 +3,6 @@ doc
coverage.html
coverage
v8.log
.idea**
.DS_Store

View File

@ -839,6 +839,7 @@ AbstractClass.hasMany = function hasMany(anotherClass, params) {
return {where: x};
}, {
find: find,
all: all,
destroy: destroy
});
@ -857,6 +858,23 @@ AbstractClass.hasMany = function hasMany(anotherClass, params) {
}.bind(this));
}
function all(filter,cb) {
if (!filter) {
filter = {};
}
if (!filter.where) {
filter.where = {};
}
filter.where[fk] = this.id.toString();
anotherClass.all(filter,function(err,inst){
if (err) return cb(err);
if (!inst) return cb(null,[]);
else return cb(null,inst);
}.bind(this));
}
function destroy(id, cb) {
this.find(id, function (err, inst) {
if (err) return cb(err);