Merge pull request #4205 from pichalite/user-search

search banned users only on /users/banned route

public/src/client/users.js

@@ -107,7 +107,8 @@ define('forum/users', ['translator'], function(translator) {
 			page: page,
 			searchBy: 'username',
 			sortBy: $('.search select').val() || getSortBy(),
-			onlineOnly: $('.search .online-only').is(':checked') || (getActiveSection() === 'online')
+			onlineOnly: $('.search .online-only').is(':checked') || (getActiveSection() === 'online'),
+			bannedOnly: getActiveSection() === 'banned'
 		}, function(err, data) {
 			if (err) {
 				return app.alertError(err.message);

src/socket.io/user/search.js

@@ -19,6 +19,7 @@ module.exports = function(SocketUser) {
 			searchBy: data.searchBy,
 			sortBy: data.sortBy,
 			onlineOnly: data.onlineOnly,
+			bannedOnly: data.bannedOnly,
 			uid: socket.uid
 		}, function(err, result) {
 			if (err) {

src/user/search.js

@@ -84,7 +84,7 @@ module.exports = function(User) {
 	function filterAndSortUids(uids, data, callback) {
 		var sortBy = data.sortBy || 'joindate';
 
-		var fields = ['uid', 'status', 'lastonline', sortBy];
+		var fields = ['uid', 'status', 'lastonline', 'banned', sortBy];
 
 		User.getUsersFields(uids, fields, function(err, userData) {
 			if (err) {
@@ -96,6 +96,12 @@ module.exports = function(User) {
 					return user && user.status !== 'offline' && (Date.now() - parseInt(user.lastonline, 10) < 300000);
 				});
 			}
+			
+			if(data.bannedOnly) {
+				userData = userData.filter(function(user) {
+					return user && user.banned;
+				});
+			}
 
 			sortUsers(userData, sortBy);