From ed57a41d177e396d9f1e655b9d12e6e62d67efe2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bar=C4=B1=C5=9F=20Soner=20U=C5=9Fakl=C4=B1?= Date: Mon, 6 Apr 2015 16:17:29 -0400 Subject: [PATCH] #2943 --- src/controllers/admin.js | 37 ++++++++++++++++++++----------------- 1 file changed, 20 insertions(+), 17 deletions(-) diff --git a/src/controllers/admin.js b/src/controllers/admin.js index c659c53446..91f8310a94 100644 --- a/src/controllers/admin.js +++ b/src/controllers/admin.js @@ -135,12 +135,12 @@ adminController.categories.get = function(req, res, next) { if (err) { return next(err); } - + plugins.fireHook('filter:admin.category.get', {req: req, res: res, category: data.category[0], privileges: data.privileges}, function(err, data) { if (err) { return next(err); } - + res.render('admin/manage/category', { category: data.category, privileges: data.privileges @@ -153,24 +153,27 @@ adminController.categories.getAll = function(req, res, next) { var active = [], disabled = []; - categories.getAllCategories(req.uid, function (err, categoryData) { + async.waterfall([ + function(next) { + db.getSortedSetRange('categories:cid', 0, -1, next); + }, + function(cids, next) { + categories.getCategoriesData(cids, next); + }, + function(categories, next) { + plugins.fireHook('filter:admin.categories.get', {req: req, res: res, categories: categories}, next); + } + ], function(err, data) { if (err) { return next(err); } - - plugins.fireHook('filter:admin.categories.get', {req: req, res: res, categories: categoryData}, function(err, data) { - if (err) { - return next(err); - } - - data.categories.filter(Boolean).forEach(function(category) { - (category.disabled ? disabled : active).push(category); - }); - - res.render('admin/manage/categories', { - active: active, - disabled: disabled - }); + data.categories.filter(Boolean).forEach(function(category) { + (category.disabled ? disabled : active).push(category); + }); + + res.render('admin/manage/categories', { + active: active, + disabled: disabled }); }); };