closes #2396

src/topics.js

@@ -69,7 +69,7 @@ var async = require('async'),
 				return callback(null, 1);
 			}
 			user.getSettings(uid, function(err, settings) {
-				if(err) {
+				if (err) {
 					return callback(err);
 				}
 

src/topics/create.js

@@ -103,11 +103,14 @@ module.exports = function(Topics) {
 
 		if (!title || title.length < parseInt(meta.config.minimumTitleLength, 10)) {
 			return callback(new Error('[[error:title-too-short, ' + meta.config.minimumTitleLength + ']]'));
-		} else if(title.length > parseInt(meta.config.maximumTitleLength, 10)) {
+		} else if (title.length > parseInt(meta.config.maximumTitleLength, 10)) {
 			return callback(new Error('[[error:title-too-long, ' + meta.config.maximumTitleLength + ']]'));
 		}
 
 		async.waterfall([
+			function(next) {
+				checkContentLength(content, next);
+			},
 			function(next) {
 				categories.exists(cid, next);
 			},
@@ -220,11 +223,10 @@ module.exports = function(Topics) {
 					content = content.trim();
 				}
 
-				if (!content || content.length < parseInt(meta.config.miminumPostLength, 10)) {
+				checkContentLength(content, next);
-					return callback(new Error('[[error:content-too-short, '  + meta.config.minimumPostLength + ']]'));
+			},
-				}
+			function(next) {
-
+				posts.create({uid: uid, tid: tid, content: content, toPid: toPid}, next);
-				posts.create({uid:uid, tid:tid, content:content, toPid:toPid}, next);
 			},
 			function(data, next) {
 				postData = data;
@@ -277,4 +279,11 @@ module.exports = function(Topics) {
 		], callback);
 	};
 
+	function checkContentLength(content, callback) {
+		if (!content || content.length < parseInt(meta.config.miminumPostLength, 10)) {
+			return callback(new Error('[[error:content-too-short, '  + meta.config.minimumPostLength + ']]'));
+		}
+		callback();
+	}
+
 };