Merge pull request #964 from strongloop/fixup-master

Fix
This commit is contained in:
Amir-61 2016-06-04 21:23:16 -04:00
commit b039b51610
2 changed files with 26 additions and 0 deletions

View File

@ -1562,6 +1562,9 @@ RelationDefinition.hasAndBelongsToMany = function hasAndBelongsToMany(modelFrom,
options.properties = params.properties;
options.scope = params.scope;
// Forward relation options like "disableInclude"
options.options = params.options;
if (params.polymorphic) {
var polymorphic = polymorphicParams(params.polymorphic);
options.polymorphic = polymorphic; // pass through

View File

@ -919,6 +919,29 @@ describe('include', function() {
});
});
});
it('should support disableInclude for hasAndBelongsToMany', function() {
var Patient = db.define('Patient', { name: String });
var Doctor = db.define('Doctor', { name: String });
var DoctorPatient = db.define('DoctorPatient');
Doctor.hasAndBelongsToMany('patients', {
model: 'Patient',
options: { disableInclude: true },
});
var doctor;
return db.automigrate(['Patient', 'Doctor', 'DoctorPatient']).then(function() {
return Doctor.create({ name: 'Who' });
}).then(function(inst) {
doctor = inst;
return doctor.patients.create({ name: 'Lazarus' });
}).then(function() {
return Doctor.find({ include: ['patients'] });
}).then(function(list) {
list.should.have.length(1);
list[0].toJSON().should.not.have.property('patients');
});
});
});
function setup(done) {