mirror of
https://github.com/NodeBB/NodeBB.git
synced 2025-11-01 03:26:04 +01:00
added more error first to some functions
This commit is contained in:
22
src/posts.js
22
src/posts.js
@@ -449,8 +449,9 @@ var db = require('./database'),
|
|||||||
|
|
||||||
Posts.uploadPostImage = function(image, callback) {
|
Posts.uploadPostImage = function(image, callback) {
|
||||||
|
|
||||||
if(!image)
|
if(!image) {
|
||||||
return callback('invalid image', null);
|
return callback('invalid image', null);
|
||||||
|
}
|
||||||
|
|
||||||
require('./imgur').upload(meta.config.imgurClientID, image.data, 'base64', function(err, data) {
|
require('./imgur').upload(meta.config.imgurClientID, image.data, 'base64', function(err, data) {
|
||||||
if(err) {
|
if(err) {
|
||||||
@@ -465,22 +466,29 @@ var db = require('./database'),
|
|||||||
}
|
}
|
||||||
|
|
||||||
Posts.getPostsByUid = function(uid, start, end, callback) {
|
Posts.getPostsByUid = function(uid, start, end, callback) {
|
||||||
user.getPostIds(uid, start, end, function(pids) {
|
user.getPostIds(uid, start, end, function(err, pids) {
|
||||||
|
if(err) {
|
||||||
|
return callback(err);
|
||||||
|
}
|
||||||
|
|
||||||
if (pids && pids.length) {
|
if (pids && pids.length) {
|
||||||
plugins.fireHook('filter:post.getTopic', pids, function(err, posts) {
|
plugins.fireHook('filter:post.getTopic', pids, function(err, posts) {
|
||||||
|
if(err) {
|
||||||
|
return callback(err);
|
||||||
|
}
|
||||||
|
|
||||||
if (!err & 0 < posts.length) {
|
if (posts && posts.length) {
|
||||||
Posts.getPostsByPids(pids, function(err, posts) {
|
Posts.getPostsByPids(pids, function(err, posts) {
|
||||||
plugins.fireHook('action:post.gotTopic', posts);
|
plugins.fireHook('action:post.gotTopic', posts);
|
||||||
callback(posts);
|
callback(null, posts);
|
||||||
});
|
});
|
||||||
} else {
|
} else {
|
||||||
callback(posts);
|
callback(null, []);
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
} else
|
} else {
|
||||||
callback([]);
|
callback(null, []);
|
||||||
|
}
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -393,18 +393,24 @@ var fs = require('fs'),
|
|||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
app.get('/api/user/:userslug', function (req, res) {
|
app.get('/api/user/:userslug', function (req, res, next) {
|
||||||
var callerUID = req.user ? req.user.uid : '0';
|
var callerUID = req.user ? req.user.uid : '0';
|
||||||
|
|
||||||
getUserDataByUserSlug(req.params.userslug, callerUID, function (userData) {
|
getUserDataByUserSlug(req.params.userslug, callerUID, function (userData) {
|
||||||
if (userData) {
|
if (userData) {
|
||||||
user.isFollowing(callerUID, userData.theirid, function (isFollowing) {
|
user.isFollowing(callerUID, userData.theirid, function (isFollowing) {
|
||||||
posts.getPostsByUid(userData.theirid, 0, 9, function (posts) {
|
posts.getPostsByUid(userData.theirid, 0, 9, function (err, posts) {
|
||||||
|
|
||||||
|
if(err) {
|
||||||
|
return next(err);
|
||||||
|
}
|
||||||
|
|
||||||
userData.posts = posts.filter(function (p) {
|
userData.posts = posts.filter(function (p) {
|
||||||
return p && parseInt(p.deleted, 10) !== 1;
|
return p && parseInt(p.deleted, 10) !== 1;
|
||||||
});
|
});
|
||||||
|
|
||||||
userData.isFollowing = isFollowing;
|
userData.isFollowing = isFollowing;
|
||||||
|
|
||||||
if (!userData.profileviews) {
|
if (!userData.profileviews) {
|
||||||
userData.profileviews = 1;
|
userData.profileviews = 1;
|
||||||
}
|
}
|
||||||
|
|||||||
17
src/user.js
17
src/user.js
@@ -444,21 +444,18 @@ var bcrypt = require('bcrypt'),
|
|||||||
|
|
||||||
User.getPostIds = function(uid, start, stop, callback) {
|
User.getPostIds = function(uid, start, stop, callback) {
|
||||||
db.getListRange('uid:' + uid + ':posts', start, stop, function(err, pids) {
|
db.getListRange('uid:' + uid + ':posts', start, stop, function(err, pids) {
|
||||||
if (!err) {
|
if(err) {
|
||||||
if (pids && pids.length) {
|
return callback(err);
|
||||||
callback(pids);
|
}
|
||||||
} else {
|
|
||||||
callback([]);
|
if (pids && pids.length) {
|
||||||
}
|
callback(null, pids);
|
||||||
} else {
|
} else {
|
||||||
console.log(err);
|
callback(null, []);
|
||||||
callback([]);
|
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
User.follow = function(uid, followid, callback) {
|
User.follow = function(uid, followid, callback) {
|
||||||
db.setAdd('following:' + uid, followid, function(err, data) {
|
db.setAdd('following:' + uid, followid, function(err, data) {
|
||||||
if (!err) {
|
if (!err) {
|
||||||
|
|||||||
Reference in New Issue
Block a user