diff --git a/src/card-controller/view/view-query-executor.ts b/src/card-controller/view/view-query-executor.ts index 942caa41..1953a6ef 100644 --- a/src/card-controller/view/view-query-executor.ts +++ b/src/card-controller/view/view-query-executor.ts @@ -103,21 +103,14 @@ export class ViewQueryExecutor { viewModifiers.push(...(await executeMediaQuery('clips'))); break; - // Gallery views: + case 'clip': case 'clips': + case 'snapshot': case 'snapshots': + case 'recording': case 'recordings': viewModifiers.push(...(await executeMediaQuery(mediaType))); break; - - // Viewer views: - case 'clip': - case 'snapshot': - case 'recording': - if (config.media_viewer.controls.thumbnails.mode !== 'none') { - viewModifiers.push(...(await executeMediaQuery(mediaType))); - } - break; } viewModifiers.push(...this._getTimelineWindowViewModifier(view)); diff --git a/tests/card-controller/view/view-query-executor.test.ts b/tests/card-controller/view/view-query-executor.test.ts index 33cf150f..9abbc384 100644 --- a/tests/card-controller/view/view-query-executor.test.ts +++ b/tests/card-controller/view/view-query-executor.test.ts @@ -345,37 +345,6 @@ describe('ViewQueryExecutor', () => { ); }); - describe('with an media viewer view', () => { - it('should not fetch anything if configured for no thumbnails', async () => { - const executor = mock(); - - const viewQueryExecutor = new ViewQueryExecutor( - createPopulatedAPI({ - media_viewer: { - controls: { - thumbnails: { - mode: 'none' as const, - }, - }, - }, - }), - executor, - ); - const view = new View({ - view: 'clip', - camera: 'camera.office', - }); - - const modifiers = await viewQueryExecutor.getNewQueryModifiers(view); - applyViewModifiers(view, modifiers); - - expect(view?.query).toBeNull(); - expect(view?.queryResults).toBeNull(); - expect(executor.executeDefaultEventQuery).not.toHaveBeenCalled(); - expect(executor.executeDefaultRecordingQuery).not.toHaveBeenCalled(); - }); - }); - describe('when setting or removing seek time', () => { it('should set seek time when results are selected based on time', async () => { const now = new Date();