closes #2851

src/plugins.js

@@ -272,11 +272,7 @@ var fs = require('fs'),
 
 				async.filter(dirs, function(dir, callback){
 					fs.stat(dir, function(err, stats){
-						if (err) {
+						callback(!err && stats.isDirectory());
-							return callback(false);
-						}
-
-						callback(stats.isDirectory());
 					});
 				}, function(plugins){
 					next(null, plugins);
@@ -287,25 +283,11 @@ var fs = require('fs'),
 				var plugins = [];
 
 				async.each(files, function(file, next) {
-					var configPath;
-
 					async.waterfall([
 						function(next) {
 							Plugins.loadPluginInfo(file, next);
 						},
 						function(pluginData, next) {
-							var packageName = path.basename(file);
-
-							if (!pluginData) {
-								winston.warn("Plugin `" + packageName + "` is corrupted or invalid. Please check either package.json or plugin.json for errors.");
-								return next(null, {
-									id: packageName,
-									installed: true,
-									error: true,
-									active: null
-								});
-							}
-
 							Plugins.isActive(pluginData.name, function(err, active) {
 								if (err) {
 									return next(new Error('no-active-state'));
@@ -319,12 +301,12 @@ var fs = require('fs'),
 								next(null, pluginData);
 							});
 						}
-					], function(err, config) {
+					], function(err, pluginData) {
 						if (err) {
 							return next(); // Silently fail
 						}
 
-						plugins.push(config);
+						plugins.push(pluginData);
 						next();
 					});
 				}, function(err) {

src/plugins/load.js

@@ -14,14 +14,12 @@ module.exports = function(Plugins) {
 	Plugins.loadPlugin = function(pluginPath, callback) {
 		Plugins.loadPluginInfo(pluginPath, function(err, pluginData) {
 			if (err) {
+				if (err.message === '[[error:parse-error]]') {
+					return callback();
+				}
 				return callback(pluginPath.match('nodebb-theme') ? null : err);
 			}
 
-			var staticDir;
-			if (!pluginData) {
-				return callback();
-			}
-
 			versionWarning(pluginData);
 
 			async.parallel([
@@ -229,9 +227,9 @@ module.exports = function(Plugins) {
 				var pluginDir = pluginPath.split(path.sep);
 				pluginDir = pluginDir[pluginDir.length -1];
 
-				winston.error('[plugins/' + pluginDir + '] Error in plugin.json/package.json! ' + err.message);
+				winston.error('[plugins/' + pluginDir + '] Error in plugin.json or package.json! ' + err.message);
 
-				callback();
+				callback(new Error('[[error:parse-error]]'));
 			}
 		});
 	};