Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-11-03 04:25:55 +01:00
Code Issues Packages Projects Releases Wiki Activity

no need for exists check

Browse Source 
if topic doesnt exist topicData will be null
This commit is contained in:
barisusakli
2014-03-21 18:20:06 -04:00
parent a695c6134b
commit ca0dbf84be
2 changed files with 7 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/topics.js
View File

@@ -272,28 +272,25 @@ var async = require('async'),
};
Topics.getTopicWithPosts = function(tid, uid, start, end, callback) {
threadTools.exists(tid, function(err, exists) {
if (err || !exists) {
Topics.getTopicData(tid, function(err, topicData) {
if (err || !topicData) {
return callback(err || new Error('Topic tid \'' + tid + '\' not found'));
}
async.parallel({
topicData : function(next) {
Topics.getTopicData(tid, next);
},
posts : function(next) {
posts: function(next) {
Topics.getTopicPosts(tid, start, end, uid, false, next);
},
privileges : function(next) {
privileges: function(next) {
threadTools.privileges(tid, uid, next);
},
category : function(next) {
category: function(next) {
Topics.getCategoryData(tid, next);
},
pageCount : function(next) {
pageCount: function(next) {
Topics.getPageCount(tid, uid, next);
},
threadTools : function(next) {
threadTools: function(next) {
plugins.fireHook('filter:topic.thread_tools', [], next);
}
}, function(err, results) {
@@ -301,7 +298,6 @@ var async = require('async'),
return callback(err);
}
var topicData = results.topicData;
topicData.category = results.category;
topicData.posts = results.posts;
topicData.thread_tools = results.threadTools;

3
src/topics/create.js
View File

@@ -2,15 +2,12 @@
'use strict';
var async = require('async'),
db = require('./../database'),
utils = require('./../../public/src/utils'),
plugins = require('./../plugins'),
user = require('./../user'),
meta = require('./../meta'),
posts = require('./../posts'),
threadTools = require('./../threadTools'),
categoryTools = require('./../categoryTools');