Add open action for editors

docs/embed-mode/_index.md

@@ -82,6 +82,35 @@ By default, the Embed mode allows users to select resources. In certain cases (e
 </script>
 ```
 
+## File picker
+
+The File Picker mode in ownCloud Web is designed for embedding an interface that allows users to pick a single file.
+This mode can be configured to restrict the file types that users can select. To enable the File Picker mode, you need
+to include the embed-target=file query parameter in the iframe URL. Furthermore, you can specify allowed file types
+using the embed-file-types parameter. The file types can be specified using file extensions, MIME types, or a
+combination of both. If the embed-file-types parameter is not provided, all file types will be selectable by default.
+
+### Example
+
+```html
+
+<iframe src="https://my-owncloud-web-instance?embed=true&embed-target=file&embed-file-types=txt,image/png"></iframe>
+
+<script>
+    function selectEventHandler(event) {
+        if (event.data?.name !== 'owncloud-embed:file-pick') {
+            return
+        }
+
+        const file = event.data.data
+
+        doSomethingWithPickedFile(file)
+    }
+
+    window.addEventListener('message', selectEventHandler)
+</script>
+```
+
 ## Delegate authentication
 
 If you already have a valid `access_token` that can be used to call the API from within the Embed mode and do not want to force the user to authenticate again, you can delegate the authentication. Delegating authentication will disable internal login form in ownCloud Web and will instead use events to obtain the token and update it.

packages/web-app-files/src/extensions.ts

@@ -1,6 +1,7 @@
 import {
   Extension,
   useCapabilityStore,
+  useConfigStore,
   useFileActionsCopyQuickLink,
   useFileActionsShowShares,
   useRouter,
@@ -14,6 +15,7 @@ import { quickActionsExtensionPoint } from './extensionPoints'
 
 export const extensions = () => {
   const capabilityStore = useCapabilityStore()
+  const configStore = useConfigStore()
   const router = useRouter()
   const { search: searchFunction } = useSearch()
 
@@ -30,7 +32,7 @@ export const extensions = () => {
       id: 'com.github.owncloud.web.files.search',
       extensionPointIds: ['app.search.provider'],
       type: 'search',
-      searchProvider: new SDKSearch(capabilityStore, router, searchFunction)
+      searchProvider: new SDKSearch(capabilityStore, router, searchFunction, configStore)
     },
     {
       id: 'com.github.owncloud.web.files.quick-action.collaborator',

packages/web-app-files/src/search/sdk/index.ts

@@ -3,6 +3,7 @@ import List from './list'
 import { Router } from 'vue-router'
 import {
   CapabilityStore,
+  ConfigStore,
   SearchFunction,
   SearchList,
   SearchPreview,
@@ -20,10 +21,15 @@ export default class Provider implements SearchProvider {
   public readonly listSearch: SearchList
   private readonly capabilityStore: CapabilityStore
 
-  constructor(capabilityStore: CapabilityStore, router: Router, searchFunction: SearchFunction) {
+  constructor(
+    capabilityStore: CapabilityStore,
+    router: Router,
+    searchFunction: SearchFunction,
+    configStore: ConfigStore
+  ) {
     this.id = 'files.sdk'
     this.displayName = $gettext('Files')
-    this.previewSearch = new Preview(router, searchFunction)
+    this.previewSearch = new Preview(router, searchFunction, configStore)
     this.listSearch = new List(searchFunction)
     this.capabilityStore = capabilityStore
   }

packages/web-app-files/src/search/sdk/preview.ts

@@ -1,4 +1,4 @@
-import { SearchFunction, SearchPreview, SearchResult } from '@ownclouders/web-pkg'
+import { ConfigStore, SearchFunction, SearchPreview, SearchResult } from '@ownclouders/web-pkg'
 import { Component, unref } from 'vue'
 import { Router } from 'vue-router'
 import { ResourcePreview } from '@ownclouders/web-pkg'
@@ -9,18 +9,23 @@ export default class Preview implements SearchPreview {
   public readonly component: Component
   private readonly router: Router
   private readonly searchFunction: SearchFunction
+  private readonly configStore: ConfigStore
 
-  constructor(router: Router, searchFunction: SearchFunction) {
+  constructor(router: Router, searchFunction: SearchFunction, configStore: ConfigStore) {
     this.component = ResourcePreview
     this.router = router
     this.searchFunction = searchFunction
+    this.configStore = configStore
   }
 
   public search(term: string): Promise<SearchResult> {
     return this.searchFunction(term, previewSearchLimit)
   }
 
   public get available(): boolean {
-    return unref(this.router.currentRoute).name !== 'search-provider-list'
+    return (
+      unref(this.router.currentRoute).name !== 'search-provider-list' &&
+      !this.configStore.options?.embed?.enabled
+    )
   }
 }

packages/web-app-files/tests/unit/search/sdk.spec.ts

@@ -3,7 +3,7 @@ import { RouteLocation, Router } from 'vue-router'
 import { mock } from 'vitest-mock-extended'
 import { ref } from 'vue'
 import { createTestingPinia } from 'web-test-helpers/src'
-import { useCapabilityStore } from '@ownclouders/web-pkg'
+import { ConfigStore, useCapabilityStore } from '@ownclouders/web-pkg'
 
 const getStore = (reports: string[] = []) => {
   createTestingPinia({
@@ -14,7 +14,7 @@ const getStore = (reports: string[] = []) => {
 
 describe('SDKProvider', () => {
   it('is only available if announced via capabilities', () => {
-    const search = new SDKSearch(getStore(), mock<Router>(), vi.fn())
+    const search = new SDKSearch(getStore(), mock<Router>(), vi.fn(), mock<ConfigStore>())
     expect(search.available).toBe(false)
   })
 
@@ -30,7 +30,8 @@ describe('SDKProvider', () => {
           mock<Router>({
             currentRoute: ref(mock<RouteLocation>({ name: v.route }))
           }),
-          vi.fn()
+          vi.fn(),
+          mock<ConfigStore>()
         )
 
         expect(!!search.previewSearch.available).toBe(!!v.available)

packages/web-app-search/src/portals/SearchBar.vue

@@ -47,6 +47,7 @@
       <oc-icon name="search" fill-type="line"></oc-icon>
     </oc-button>
     <oc-drop
+      v-if="showDrop"
       id="files-global-search-options"
       ref="optionsDropRef"
       mode="manual"
@@ -239,15 +240,15 @@ export default defineComponent({
       }
 
       if (unref(optionsDrop)) {
-        unref(optionsDrop).hide()
+        unref(optionsDrop)?.hide()
       }
 
       if (unref(activePreviewIndex) === null) {
         router.push(getSearchResultLocation('files.sdk'))
       }
       if (unref(activePreviewIndex) !== null) {
         unref(optionsDrop)
-          .$el.querySelectorAll('.preview')
+          ?.$el.querySelectorAll('.preview')
           [unref(activePreviewIndex)].firstChild.click()
       }
     }
@@ -283,17 +284,17 @@ export default defineComponent({
       if (!unref(term)) {
         return
       }
-      unref(optionsDrop).show()
+      unref(optionsDrop)?.show()
       await search()
     }
 
     const updateTerm = (input: string) => {
       restoreSearchFromRoute.value = false
       term.value = input
       if (!unref(term)) {
-        return unref(optionsDrop).hide()
+        return unref(optionsDrop)?.hide()
       }
-      return unref(optionsDrop).show()
+      return unref(optionsDrop)?.show()
     }
 
     const debouncedSearch = debounce(search, 500)
@@ -306,6 +307,12 @@ export default defineComponent({
       debouncedSearch()
     })
 
+    const showDrop = computed(() => {
+      return unref(availableProviders).some(
+        (provider) => provider?.previewSearch?.available === true
+      )
+    })
+
     return {
       userContextReady,
       publicLinkContextReady,
@@ -328,7 +335,8 @@ export default defineComponent({
       search,
       showPreview,
       updateTerm,
-      getSearchResultLocation
+      getSearchResultLocation,
+      showDrop
     }
   },
 
@@ -506,7 +514,7 @@ export default defineComponent({
       this.showCancelButton = false
     },
     hideOptionsDrop() {
-      this.optionsDrop.hide()
+      this.optionsDrop?.hide()
     }
   }
 })

packages/web-pkg/src/components/AppTemplates/AppWrapper.vue

@@ -87,6 +87,7 @@ import {
 import { DavPermission } from '@ownclouders/web-client/webdav'
 import { HttpError } from '@ownclouders/web-client'
 import { dirname } from 'path'
+import { useFileActionsOpenWithApp } from '../../composables/actions/files/useFileActionsOpenWithApp'
 
 export default defineComponent({
   name: 'AppWrapper',
@@ -135,6 +136,9 @@ export default defineComponent({
     const configStore = useConfigStore()
     const resourcesStore = useResourcesStore()
 
+    const { actions: openWithAppActions } = useFileActionsOpenWithApp({
+      appId: props.applicationId
+    })
     const { actions: createQuickLinkActions } = useFileActionsCopyQuickLink()
     const { actions: downloadFileActions } = useFileActionsDownloadFile()
     const { actions: showDetailsActions } = useFileActionsShowDetails()
@@ -459,7 +463,9 @@ export default defineComponent({
     })
 
     const menuItemsContext = computed(() => {
-      return [...unref(fileActionsSave)].filter((item) => item.isVisible(unref(actionOptions)))
+      return [...unref(openWithAppActions), ...unref(fileActionsSave)].filter((item) =>
+        item.isVisible(unref(actionOptions))
+      )
     })
     const menuItemsShare = computed(() => {
       return [...unref(showSharesActions), ...unref(createQuickLinkActions)].filter((item) =>

packages/web-pkg/src/components/AppTopBar.vue

@@ -44,7 +44,7 @@
             </oc-drop>
           </template>
           <span v-if="hasAutosave" class="oc-flex oc-flex-middle">
-            <oc-icon name="refresh" color="white" v-oc-tooltip="autoSaveTooltipText" />
+            <oc-icon v-oc-tooltip="autoSaveTooltipText" name="refresh" color="white" />
           </span>
           <template v-if="mainActions.length && resource">
             <context-action-menu

packages/web-pkg/src/components/FilesList/ResourceTable.vue

@@ -508,8 +508,9 @@ export default defineComponent({
     const {
       isLocationPicker,
       isFilePicker,
+      postMessage,
       isEnabled: isEmbedModeEnabled,
-      extensions: embedModeExtensions
+      fileTypes: embedModeFileTypes
     } = useEmbedMode()
 
     const configStore = useConfigStore()
@@ -540,13 +541,12 @@ export default defineComponent({
     const getTagToolTip = (text: string) => (text.length > 7 ? text : '')
 
     const isResourceDisabled = (resource: Resource) => {
-      if (
-        unref(isEmbedModeEnabled) &&
-        unref(embedModeExtensions)?.length &&
-        !unref(embedModeExtensions).includes(resource.extension) &&
-        !resource.isFolder
-      ) {
-        return true
+      if (unref(isEmbedModeEnabled) && unref(embedModeFileTypes)?.length) {
+        return (
+          !unref(embedModeFileTypes).includes(resource.extension) &&
+          !unref(embedModeFileTypes).includes(resource.mimeType) &&
+          !resource.isFolder
+        )
       }
       return resource.processing === true
     }
@@ -587,6 +587,7 @@ export default defineComponent({
         space: ref(props.space),
         targetRouteCallback: computed(() => props.targetRouteCallback)
       }),
+      postMessage,
       isFilePicker,
       isLocationPicker,
       isEmbedModeEnabled,
@@ -988,6 +989,13 @@ export default defineComponent({
        */
       const resource = data[0]
 
+      if (this.isEmbedModeEnabled && this.isFilePicker) {
+        return this.postMessage<Resource>(
+          'owncloud-embed:file-pick',
+          JSON.parse(JSON.stringify(resource))
+        )
+      }
+
       if (this.isResourceDisabled(resource)) {
         return
       }
@@ -1054,7 +1062,7 @@ export default defineComponent({
         return false
       }
 
-      if (this.isEmbedModeEnabled && !resource.isFolder) {
+      if (this.isEmbedModeEnabled && !this.isFilePicker && !resource.isFolder) {
         return false
       }
 

packages/web-pkg/src/components/FilesList/ResourceTile.vue

@@ -4,8 +4,8 @@
     :data-item-id="resource.id"
     :class="{
       'oc-tile-card-selected': isResourceSelected,
-      'oc-tile-card-disabled': resource.processing,
-      'state-trashed': resourceDisabled
+      'oc-tile-card-disabled': isResourceDisabled && !isProjectSpaceResource(resource),
+      'state-trashed': isResourceDisabled && isProjectSpaceResource(resource)
     }"
     @contextmenu="$emit('contextmenu', $event)"
   >
@@ -21,7 +21,7 @@
         <slot name="selection" :item="resource" />
       </div>
       <oc-tag
-        v-if="resourceDisabled"
+        v-if="isResourceDisabled && isProjectSpaceResource(resource)"
         class="resource-disabled-indicator oc-position-absolute"
         type="span"
       >
@@ -85,7 +85,7 @@ import { computed, defineComponent, PropType } from 'vue'
 import ResourceIcon from './ResourceIcon.vue'
 import ResourceListItem from './ResourceListItem.vue'
 import ResourceLink from './ResourceLink.vue'
-import { Resource } from '@ownclouders/web-client'
+import { isProjectSpaceResource, Resource } from '@ownclouders/web-client'
 import { useGettext } from 'vue3-gettext'
 import { isSpaceResource } from '@ownclouders/web-client'
 import { isResourceTxtFileAlmostEmpty } from '../../helpers'
@@ -115,6 +115,11 @@ export default defineComponent({
       required: false,
       default: true
     },
+    isResourceDisabled: {
+      type: Boolean,
+      required: false,
+      default: false
+    },
     isExtensionDisplayed: {
       type: Boolean,
       default: true
@@ -158,11 +163,6 @@ export default defineComponent({
       }
       return null
     })
-
-    const resourceDisabled = computed(() => {
-      return isSpaceResource(props.resource) && props.resource.disabled === true
-    })
-
     const resourceDescription = computed(() => {
       if (isSpaceResource(props.resource)) {
         return props.resource.description
@@ -178,11 +178,11 @@ export default defineComponent({
       statusIconAttrs,
       showStatusIcon,
       tooltipLabelIcon,
-      resourceDisabled,
       resourceDescription,
       shouldDisplayThumbnails
     }
-  }
+  },
+  methods: { isProjectSpaceResource }
 })
 </script>