From 49cb5fddcddbaa8ac117b1bf61d2fb21628aed10 Mon Sep 17 00:00:00 2001 From: Baris Usakli Date: Tue, 7 May 2013 11:53:40 -0400 Subject: [PATCH 1/2] removed some console.log --- src/webserver.js | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/src/webserver.js b/src/webserver.js index 96db917c00..5dac314341 100644 --- a/src/webserver.js +++ b/src/webserver.js @@ -314,7 +314,6 @@ passport.deserializeUser(function(uid, done) { //to baris, move this into account.js or sth later - just moved this out here for you to utilize client side tpl parsing //I didn't want to change too much so you should probably sort out the params etc function get_account_fn(req, res, callback) { - console.log("GOING TO ACCOUNT"); if (req.user === undefined) return res.redirect('/403'); @@ -322,8 +321,6 @@ passport.deserializeUser(function(uid, done) { user.getUserData(req.user.uid, function(data) { data.joindate = utils.relativeTime(data.joindate); - - console.log("user data" + JSON.stringify(data)); var userData = {user:data}; callback(userData); @@ -355,19 +352,22 @@ passport.deserializeUser(function(uid, done) { function handleUserProfile(req, res) { - console.log("userid " + req.params.uid); + if(req.params.uid == 0) { res.send("User doesn't exist!"); return; } user.getUserData(req.params.uid, function(data) { - - if(req.url.indexOf(data.username) == -1) - res.redirect(301, '/users/'+req.params.uid+'/'+data.username); + if(data) { + if(req.url.indexOf(data.username) == -1) + res.redirect(301, '/users/'+req.params.uid+'/'+data.username); + else + res.send(templates['header'] + '
'+JSON.stringify(data, null, 4)+'
' + templates['footer']); + //res.send(templates['header'] + '' + templates['footer']); + } else - res.send(templates['header'] + '
'+JSON.stringify(data, null, 4)+'
' + templates['footer']); -// res.send(templates['header'] + '' + templates['footer']); + res.send("User doesn't exist!"); }); } From ccf0f6dd7352162897f6558526126a350a71b018 Mon Sep 17 00:00:00 2001 From: Baris Usakli Date: Tue, 7 May 2013 12:11:22 -0400 Subject: [PATCH 2/2] removed get_username_by_uid, use getUserField(uid,'username',callback) directly --- src/user.js | 6 +----- src/websockets.js | 6 ------ 2 files changed, 1 insertion(+), 11 deletions(-) diff --git a/src/user.js b/src/user.js index bdfc30d306..52c13d4132 100644 --- a/src/user.js +++ b/src/user.js @@ -326,17 +326,13 @@ var config = require('../config.js'), RDB.get('username:' + username + ':uid', callback); }; - User.get_username_by_uid = function(uid, callback) { - User.getUserField(uid, 'username', callback); - }; - User.get_usernames_by_uids = function(uids, callback) { var usernames = []; for(var i=0, ii=uids.length; i