diff --git a/lib/server/controllers/lhcPeriodStatistics.controller.js b/lib/server/controllers/lhcPeriodStatistics.controller.js index c70b04b67c..8784e3871e 100644 --- a/lib/server/controllers/lhcPeriodStatistics.controller.js +++ b/lib/server/controllers/lhcPeriodStatistics.controller.js @@ -42,7 +42,7 @@ const listLhcPeriodStatisticsHandler = async (req, res) => { ); if (validatedDTO) { try { - const { filter, page: { limit = ApiConfig.pagination.limit, offset } = {}, sort = { name: 'DESC' } } = validatedDTO.query; + const { filter, page: { limit = ApiConfig.pagination.limit, offset } = {}, sort = { id: 'DESC' } } = validatedDTO.query; const { count, rows: items } = await lhcPeriodStatisticsService.getAllForPhysicsRuns({ filter, limit, diff --git a/lib/server/services/lhcPeriod/LhcPeriodStatisticsService.js b/lib/server/services/lhcPeriod/LhcPeriodStatisticsService.js index 4e92ae675e..f9b803144d 100644 --- a/lib/server/services/lhcPeriod/LhcPeriodStatisticsService.js +++ b/lib/server/services/lhcPeriod/LhcPeriodStatisticsService.js @@ -85,11 +85,13 @@ class LhcPeriodStatisticsService { sort, } = {}) { const queryBuilder = this.prepareQueryBuilder(); - if (sort) { for (const property in sort) { let expression; switch (property) { + case 'id': + expression = (sequelize) => sequelize.col('`lhcPeriod`.`id`');; + break; case 'name': expression = (sequelize) => sequelize.col('`lhcPeriod`.`name`'); break;