From bc95f55cb080ec1e4c35a7a2a8e7a93d4a25d701 Mon Sep 17 00:00:00 2001
From: Dermot Duffy <dermot.duffy@gmail.com>
Date: Mon, 23 Dec 2024 16:22:10 -0800
Subject: [PATCH] fix: Always fetch media for "most recent media" views (#1791)

---
 .../view/view-query-executor.ts               | 13 ++------
 .../view/view-query-executor.test.ts          | 31 -------------------
 2 files changed, 3 insertions(+), 41 deletions(-)

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<QueryExecutor>();
-
-        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();