Server: little sort refractoring

1 files changed, 14 insertions, 9 deletions

diff --git a/server/middlewares/validators/sort.js b/server/middlewares/validators/sort.js
index b7eec0316..017d266e6 100644
--- a/server/middlewares/validators/sort.js
+++ b/server/middlewares/validators/sort.js
@@ -10,22 +10,21 @@ const validatorsSort = {
   videosSort
 }
 
-function usersSort (req, res, next) {
-  const sortableColumns = constants.SORTABLE_COLUMNS.USERS
+// Initialize constants here for better performances
+const SORTABLE_USERS_COLUMNS = createSortableColumns(constants.SORTABLE_COLUMNS.USERS)
+const SORTABLE_VIDEO_ABUSES_COLUMNS = createSortableColumns(constants.SORTABLE_COLUMNS.VIDEO_ABUSES)
+const SORTABLE_VIDEOS_COLUMNS = createSortableColumns(constants.SORTABLE_COLUMNS.VIDEOS)
 
-  checkSort(req, res, next, sortableColumns)
+function usersSort (req, res, next) {
+  checkSort(req, res, next, SORTABLE_USERS_COLUMNS)
 }
 
 function videoAbusesSort (req, res, next) {
-  const sortableColumns = constants.SORTABLE_COLUMNS.VIDEO_ABUSES
-
-  checkSort(req, res, next, sortableColumns)
+  checkSort(req, res, next, SORTABLE_VIDEO_ABUSES_COLUMNS)
 }
 
 function videosSort (req, res, next) {
-  const sortableColumns = constants.SORTABLE_COLUMNS.VIDEOS
-
-  checkSort(req, res, next, sortableColumns)
+  checkSort(req, res, next, SORTABLE_VIDEOS_COLUMNS)
 }
 
 // ---------------------------------------------------------------------------
@@ -41,3 +40,9 @@ function checkSort (req, res, next, sortableColumns) {
 
   checkErrors(req, res, next)
 }
+
+function createSortableColumns (sortableColumns) {
+  const sortableColumnDesc = sortableColumns.map(sortableColumn => '-' + sortableColumn)
+
+  return sortableColumns.concat(sortableColumnDesc)
+}