verdnatura-mirror
/
loopback-datasource-juggler
mirror of https://github.com/loopbackio/loopback-datasource-juggler
0
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

HasMany exists should use internal findById 

This limits the scope correctly, taking polymorphics into account.

(the foreign key check is actually obsolete I think)
This commit is contained in:
Fabien Franzen 2014-08-16 13:04:13 +02:00
parent 855270d14a
commit b4b1c784dd
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/relation-definition.js
View File

@ -692,12 +692,11 @@ HasMany.prototype.findById = function (fkId, cb) {
* @param {Function} cb The callback function * @param {Function} cb The callback function
*/ */
HasMany.prototype.exists = function (fkId, cb) { HasMany.prototype.exists = function (fkId, cb) {
var modelTo = this.definition.modelTo;
var fk = this.definition.keyTo; var fk = this.definition.keyTo;
var pk = this.definition.keyFrom; var pk = this.definition.keyFrom;
var modelInstance = this.modelInstance; var modelInstance = this.modelInstance;
modelTo.findById(fkId, function (err, inst) { this.findById(fkId, function (err, inst) {
if (err) { if (err) {
return cb(err); return cb(err);
} }