Skip to content

Commit d01cc50

Browse files
author
NarrowsProjects
committed
EXPERIMENT: undo changes for GetAllLogsUseCase
1 parent 48c0dfd commit d01cc50

1 file changed

Lines changed: 9 additions & 6 deletions

File tree

lib/usecases/log/GetAllLogsUseCase.js

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ const { Op } = require('sequelize');
3030
const { dataSource } = require('../../database/DataSource.js');
3131
const { checkForFilterExclusion } = require('../common/checkForFilterExclusion.js');
3232
const { setTimeRangeQuery } = require('../../utilities/setTimeRangeQuery.js');
33-
const { logSummary } = require('../../server/views/logs/summary.js');
3433

3534
/**
3635
* Apply the given filter on the given query builder
@@ -220,14 +219,18 @@ class GetAllLogsUseCase {
220219
* @returns {Promise} Promise object represents the result of this use case.
221220
*/
222221
async execute(dto = {}) {
223-
const queryBuilder = dataSource.createQueryBuilder();
222+
const queryBuilder = dataSource.createQueryBuilder(true);
224223
const { query = {} } = dto;
225224
const { filter, sort = { id: 'desc' }, page = {} } = query;
226225

227-
queryBuilder.selectAttributes([...logSummary.attributes]);
228-
queryBuilder.includeAll([...logSummary.include]);
229-
230226
const { count, rows } = await dataSource.transaction(async () => {
227+
queryBuilder.include('user')
228+
.include('tags')
229+
.include({ association: 'runs', attributes: ['id', 'runNumber'] })
230+
.include({ association: 'lhcFills', attributes: ['fillNumber'] })
231+
.include('attachments')
232+
.include({ association: 'environments', attributes: ['id'] });
233+
231234
if (filter) {
232235
await applyFilter(dataSource, queryBuilder, filter);
233236
}
@@ -253,7 +256,7 @@ class GetAllLogsUseCase {
253256

254257
return {
255258
count,
256-
logs: rowsWithReplies.map(logAdapter.toSummary),
259+
logs: rowsWithReplies.map(logAdapter.toEntity),
257260
};
258261
}
259262
}

0 commit comments

Comments
 (0)