fix: #7831, fix pagination

convert to async/await

src/controllers/accounts/blocks.js

@@ -1,55 +1,39 @@
 'use strict';
 
-var async = require('async');
+const helpers = require('../helpers');
-
+const accountHelpers = require('./helpers');
-var helpers = require('../helpers');
+const pagination = require('../../pagination');
-var accountHelpers = require('./helpers');
+const user = require('../../user');
-var pagination = require('../../pagination');
+const plugins = require('../../plugins');
-var user = require('../../user');
+
-var plugins = require('../../plugins');
+const blocksController = module.exports;
-
+
-var blocksController = module.exports;
+blocksController.getBlocks = async function (req, res, next) {
-
+	const page = parseInt(req.query.page, 10) || 1;
-blocksController.getBlocks = function (req, res, callback) {
+	const resultsPerPage = 50;
-	var userData;
+	const start = Math.max(0, page - 1) * resultsPerPage;
-
+	const stop = start + resultsPerPage - 1;
-	var page = parseInt(req.query.page, 10) || 1;
+
-	var resultsPerPage = 50;
+	const userData = await accountHelpers.getUserDataByUserSlug(req.params.userslug, req.uid);
-	var start = Math.max(0, page - 1) * resultsPerPage;
+	if (!userData) {
-	var stop = start + resultsPerPage - 1;
+		return next();
-
+	}
-	async.waterfall([
+	const uids = await user.blocks.list(userData.uid);
-		function (next) {
+	const data = await plugins.fireHook('filter:user.getBlocks', {
-			accountHelpers.getUserDataByUserSlug(req.params.userslug, req.uid, next);
+		uids: uids,
-		},
+		uid: userData.uid,
-		function (_userData, next) {
+		start: start,
-			userData = _userData;
+		stop: stop,
-			if (!userData) {
+	});
-				return callback();
+
-			}
+	data.uids = data.uids.slice(start, stop + 1);
-
+	userData.users = await user.getUsers(data.uids, req.uid);
-			user.blocks.list(res.locals.uid, next);
+	userData.title = '[[pages:account/blocks, ' + userData.username + ']]';
-		},
+
-		function (uids, next) {
+	const pageCount = Math.ceil(userData.blocksCount / resultsPerPage);
-			plugins.fireHook('filter:user.getBlocks', {
+	userData.pagination = pagination.create(page, pageCount);
-				uids: uids,
+
-				uid: res.locals.uid,
+	userData.breadcrumbs = helpers.buildBreadcrumbs([{ text: userData.username, url: '/user/' + userData.userslug }, { text: '[[user:blocks]]' }]);
-				start: start,
+
-				stop: stop,
+	res.render('account/blocks', userData);
-			}, next);
-		},
-		function (data, next) {
-			user.getUsers(data.uids, res.locals.uid, next);
-		},
-		function (users) {
-			userData.users = users;
-			userData.title = '[[pages:account/blocks, ' + userData.username + ']]';
-			var count = userData.blocksCount;
-			var pageCount = Math.ceil(count / resultsPerPage);
-			userData.pagination = pagination.create(page, pageCount);
-			userData.breadcrumbs = helpers.buildBreadcrumbs([{ text: userData.username, url: '/user/' + userData.userslug }, { text: '[[user:blocks]]' }]);
-
-			res.render('account/blocks', userData);
-		},
-	], callback);
 };