diff --git a/src/postTools.js b/src/postTools.js index 8d8969a768..b722095e05 100644 --- a/src/postTools.js +++ b/src/postTools.js @@ -41,7 +41,8 @@ var RDB = require('./redis.js'), } function hasEnoughRep(next) { - user.getUserField(uid, 'reputation', function(reputation) { + user.getUserField(uid, 'reputation', function(err, reputation) { + if (err) return next(null, false); next(null, reputation >= meta.config['privileges:manage_content']); }); } diff --git a/src/routes/user.js b/src/routes/user.js index a4c6c38c88..cd17791c31 100644 --- a/src/routes/user.js +++ b/src/routes/user.js @@ -79,8 +79,7 @@ var user = require('./../user.js'), if(!req.user) return res.redirect('/403'); - user.getUserField(req.user.uid, 'userslug', function(userslug) { - + user.getUserField(req.user.uid, 'userslug', function(err, userslug) { if(req.params.userslug && userslug === req.params.userslug) { app.build_header({ req: req, res: res }, function(err, header) { res.send(header + app.create_route('users/'+req.params.userslug+'/edit','accountedit') + templates['footer']); @@ -96,7 +95,7 @@ var user = require('./../user.js'), if(!req.user) return res.redirect('/403'); - user.getUserField(req.user.uid, 'userslug', function(userslug) { + user.getUserField(req.user.uid, 'userslug', function(err, userslug) { if(req.params.userslug && userslug === req.params.userslug) { app.build_header({ req: req, res: res }, function(err, header) { res.send(header + app.create_route('users/'+req.params.userslug+'/settings','accountsettings') + templates['footer']); @@ -127,7 +126,7 @@ var user = require('./../user.js'), return; } - user.getUserField(req.user.uid, 'uploadedpicture', function(oldpicture) { + user.getUserField(req.user.uid, 'uploadedpicture', function(err, oldpicture) { if(!oldpicture) { uploadUserPicture(req.user.uid, path.extname(req.files.userPhoto.name), req.files.userPhoto.path, res); return; diff --git a/src/threadTools.js b/src/threadTools.js index 5bf5321a19..b3a4a9de01 100644 --- a/src/threadTools.js +++ b/src/threadTools.js @@ -31,7 +31,8 @@ var RDB = require('./redis.js'), } function hasEnoughRep(next) { - user.getUserField(uid, 'reputation', function(reputation) { + user.getUserField(uid, 'reputation', function(err, reputation) { + if (err) return next(null, false); next(null, reputation >= meta.config['privileges:manage_topic']); }); } diff --git a/src/topics.js b/src/topics.js index 98545bfbfe..1592fbcef9 100644 --- a/src/topics.js +++ b/src/topics.js @@ -33,8 +33,7 @@ marked.setOptions({ Topics.getTopicDataWithUsername = function(tid, callback) { Topics.getTopicData(tid, function(topic) { - user.getUserField(topic.uid, 'username', function(username) { - + user.getUserField(topic.uid, 'username', function(err, username) { topic.username = username; callback(topic); }); @@ -572,8 +571,8 @@ marked.setOptions({ return; } - user.getUserField(uid, 'lastposttime', function(lastposttime) { - + user.getUserField(uid, 'lastposttime', function(err, lastposttime) { + if (err) lastposttime = 0; if(Date.now() - lastposttime < meta.config.postDelay) { callback(new Error('too-many-posts'), null); return; diff --git a/src/webserver.js b/src/webserver.js index 9a57c2aa25..52cfb71c19 100644 --- a/src/webserver.js +++ b/src/webserver.js @@ -161,7 +161,7 @@ var express = require('express'), app.get('/' + route, function(req, res) { if ((route === 'login' || route ==='register') && (req.user && req.user.uid > 0)) { - user.getUserField(req.user.uid, 'userslug', function(userslug) { + user.getUserField(req.user.uid, 'userslug', function(err, userslug) { res.redirect('/users/'+userslug); }); return; diff --git a/src/websockets.js b/src/websockets.js index cb4f7b68dc..972e73ac5e 100644 --- a/src/websockets.js +++ b/src/websockets.js @@ -54,7 +54,7 @@ var SocketIO = require('socket.io').listen(global.server, { log:false }), socket.join('uid_' + uid); io.sockets.in('global').emit('api:user.isOnline', isUserOnline(uid)); - user.getUserField(uid, 'username', function(username) { + user.getUserField(uid, 'username', function(err, username) { socket.emit('event:connect', {status: 1, username:username}); }); } @@ -275,12 +275,12 @@ var SocketIO = require('socket.io').listen(global.server, { log:false }), } if(type === 'gravatar') { - user.getUserField(uid, 'gravatarpicture', function(gravatar) { + user.getUserField(uid, 'gravatarpicture', function(err, gravatar) { user.setUserField(uid, 'picture', gravatar); updateHeader(); }); } else if(type === 'uploaded') { - user.getUserField(uid, 'uploadedpicture', function(uploadedpicture) { + user.getUserField(uid, 'uploadedpicture', function(err, uploadedpicture) { user.setUserField(uid, 'picture', uploadedpicture); updateHeader(); }); @@ -514,7 +514,7 @@ var SocketIO = require('socket.io').listen(global.server, { log:false }), var msg = utils.strip_tags(data.message), numSockets = userSockets[touid].length; - user.getUserField(uid, 'username', function(username) { + user.getUserField(uid, 'username', function(err, username) { var finalMessage = username + ' says : ' + msg; for(var x=0;x