Replace all with find to make it consistent

This commit is contained in:
Raymond Feng 2013-11-18 11:39:07 -08:00
parent 6eb7f1939c
commit 05eae4aa3e
4 changed files with 11 additions and 6 deletions

View File

@ -30,10 +30,15 @@ Customer.hasMany(Order, {as: 'orders', foreignKey: 'customerId'});
Customer.create({name: 'Ray'}, function (err, customer) {
Order.create({customerId: customer.id, orderDate: new Date()}, function (err, order) {
customer.orders(console.log);
customer.orders.create({orderDate: new Date()}, console.log);
customer.orders.create({orderDate: new Date()}, function(err, order) {
console.log(order);
Customer.include([customer], 'orders', function(err, results) {
console.log('Results: ', results);
});
customer.orders.findById('2', console.log);
customer.orders.destroy('2', console.log);
});
});
});

View File

@ -119,7 +119,7 @@ Inclusion.include = function (objects, include, cb) {
req['include'] = subInclude;
return function(cb) {
relation.modelTo.all(req, function(err, objsIncluded) {
relation.modelTo.find(req, function(err, objsIncluded) {
for (var i = 0; i < objsIncluded.length; i++) {
delete keysToBeProcessed[objsIncluded[i][relation.keyTo]];
var objectsFrom = objsByKeys[relation.keyFrom][objsIncluded[i][relation.keyTo]];

View File

@ -55,7 +55,7 @@ Relation.hasMany = function hasMany(anotherClass, params) {
};
// each instance of this class should have method named
// pluralize(anotherClass.modelName)
// which is actually just anotherClass.all({where: {thisModelNameId: this[idName]}}, cb);
// which is actually just anotherClass.find({where: {thisModelNameId: this[idName]}}, cb);
var scopeMethods = {
findById: find,
destroy: destroy

View File

@ -139,7 +139,7 @@ function defineScope(cls, targetClass, name, params, methods) {
- If fetching the Elements on which destroyAll is called results in an error
*/
function destroyAll(cb) {
targetClass.all(this._scope, function (err, data) {
targetClass.find(this._scope, function (err, data) {
if (err) {
cb(err);
} else {