fix(media-viewer): fix davProperties

mediaViewer still used davProperties from component scope, updated to
use shared DavProperties

changelog/4.1.0_2021-08-19/bugfix-load-folder-in-media-viewer

@@ -4,4 +4,5 @@ We've fixed the loading of a folder in the Media viewer extension.
 If a user reloads the Media viewer now, it load all the medias both in private and public context.
 
 https://github.com/owncloud/web/issues/5427
-https://github.com/owncloud/web/pull/5585
+https://github.com/owncloud/web/pull/5585
+https://github.com/owncloud/web/pull/5710

packages/web-app-media-viewer/src/mixins/loader.js

@@ -2,14 +2,15 @@ import { mapGetters, mapMutations } from 'vuex'
 import queryString from 'query-string'
 import { basename, dirname } from 'path'
 import { buildResource } from '../../../web-app-files/src/helpers/resources' // TODO: place the helper in more global space not to create dep on files app
+import { DavProperties } from 'web-pkg/src/constants'
 
 // TODO: this file is a first attempt to separate file/folder loading logic out of the mediaviewer
 // Discussion how to progress from here can be found in this issue:
 // https://github.com/owncloud/web/issues/3301
 
 export default {
   computed: {
-    ...mapGetters('Files', ['publicLinkPassword', 'davProperties']),
+    ...mapGetters('Files', ['publicLinkPassword']),
     ...mapGetters(['configuration']),
     $_loader_publicContext() {
       return this.$route.params.contextRouteName === 'files-public-list'
@@ -39,17 +40,14 @@ export default {
       this.CLEAR_CURRENT_FILES_LIST()
 
       try {
-        const properties = this.davProperties.concat([
-          this.$client.publicFiles.PUBLIC_LINK_ITEM_TYPE,
-          this.$client.publicFiles.PUBLIC_LINK_PERMISSION,
-          this.$client.publicFiles.PUBLIC_LINK_EXPIRATION,
-          this.$client.publicFiles.PUBLIC_LINK_SHARE_DATETIME,
-          this.$client.publicFiles.PUBLIC_LINK_SHARE_OWNER
-        ])
         const absolutePath = filePath.substring(0, filePath.lastIndexOf('/'))
         const promise = this.$_loader_publicContext
-          ? this.$client.publicFiles.list(absolutePath, this.publicLinkPassword, properties)
-          : this.$client.files.list(absolutePath, 1, this.davProperties)
+          ? this.$client.publicFiles.list(
+              absolutePath,
+              this.publicLinkPassword,
+              DavProperties.PublicLink
+            )
+          : this.$client.files.list(absolutePath, 1, DavProperties.Default)
         let resources = await promise
 
         resources = resources.map(buildResource)