mirror of
https://github.com/NodeBB/NodeBB.git
synced 2025-11-18 03:31:03 +01:00
less db calls
no db call for settings if uid 0 no db call for isFollowing if uid 0
This commit is contained in:
@@ -15,6 +15,9 @@ module.exports = function(Topics) {
|
|||||||
|
|
||||||
|
|
||||||
Topics.isFollowing = function(tid, uid, callback) {
|
Topics.isFollowing = function(tid, uid, callback) {
|
||||||
|
if (!parseInt(uid, 10)) {
|
||||||
|
return callback(null, false);
|
||||||
|
}
|
||||||
db.isSetMember('tid:' + tid + ':followers', uid, callback);
|
db.isSetMember('tid:' + tid + ':followers', uid, callback);
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|||||||
@@ -1,24 +1,16 @@
|
|||||||
|
|
||||||
'use strict';
|
'use strict';
|
||||||
|
|
||||||
var meta = require('./../meta'),
|
var meta = require('../meta'),
|
||||||
db = require('./../database'),
|
db = require('../database'),
|
||||||
plugins = require('./../plugins');
|
plugins = require('../plugins');
|
||||||
|
|
||||||
module.exports = function(User) {
|
module.exports = function(User) {
|
||||||
|
|
||||||
User.getSettings = function(uid, callback) {
|
User.getSettings = function(uid, callback) {
|
||||||
db.getObject('user:' + uid + ':settings', function(err, settings) {
|
function onSettingsLoaded(settings) {
|
||||||
if(err) {
|
|
||||||
return callback(err);
|
|
||||||
}
|
|
||||||
|
|
||||||
if(!settings) {
|
|
||||||
settings = {};
|
|
||||||
}
|
|
||||||
|
|
||||||
plugins.fireHook('filter:user.getSettings', {uid: uid, settings: settings}, function(err, data) {
|
plugins.fireHook('filter:user.getSettings', {uid: uid, settings: settings}, function(err, data) {
|
||||||
if(err) {
|
if (err) {
|
||||||
return callback(err);
|
return callback(err);
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -41,6 +33,18 @@ module.exports = function(User) {
|
|||||||
|
|
||||||
callback(null, settings);
|
callback(null, settings);
|
||||||
});
|
});
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!parseInt(uid, 10)) {
|
||||||
|
return onSettingsLoaded({});
|
||||||
|
}
|
||||||
|
|
||||||
|
db.getObject('user:' + uid + ':settings', function(err, settings) {
|
||||||
|
if (err) {
|
||||||
|
return callback(err);
|
||||||
|
}
|
||||||
|
|
||||||
|
onSettingsLoaded(settings ? settings : {});
|
||||||
});
|
});
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user