fix(web-app-preview): optionally access route query

When filtering out files in preview app, access route query optionally in case it is undefined to prevent broken loading.

changelog/unreleased/bugfix-optionally-access-preview-app-route-query.md

@@ -0,0 +1,6 @@
+Bugfix: Optionally access preview app route query
+
+We've fixed an issue where loading of preview app would fail due to missing route query. The route query is now accessed optionally meaning even when it undefined, the app still loads as expected.
+
+https://github.com/owncloud/web/pull/12112
+https://github.com/owncloud/web/issues/12108

packages/web-app-preview/src/App.vue

@@ -181,14 +181,14 @@ export default defineComponent({
 
       const files = props.activeFiles.filter((file) => {
         if (
-          unref(props.currentFileContext.routeQuery)['q_share-visibility'] === 'hidden' &&
+          unref(props.currentFileContext.routeQuery)?.['q_share-visibility'] === 'hidden' &&
           !(file as IncomingShareResource).hidden
         ) {
           return false
         }
 
         if (
-          unref(props.currentFileContext.routeQuery)['q_share-visibility'] !== 'hidden' &&
+          unref(props.currentFileContext.routeQuery)?.['q_share-visibility'] !== 'hidden' &&
           (file as IncomingShareResource).hidden
         ) {
           return false

packages/web-app-preview/tests/unit/app.spec.ts

@@ -126,6 +126,13 @@ describe('Preview app', () => {
       })
       expect(wrapper.vm.filteredFiles.length).toStrictEqual(2)
     })
+
+    it('should filter files even when routeQuery is undefined', () => {
+      const { wrapper } = createShallowMountWrapper({
+        currentFileContext: { routeQuery: undefined }
+      })
+      expect(wrapper.vm.filteredFiles.length).toStrictEqual(7)
+    })
   })
 })