From 63e79f73bacb72795911bb6b603ba6c362f94558 Mon Sep 17 00:00:00 2001
From: Lukas Hirt <lhirt@owncloud.com>
Date: Thu, 5 Sep 2019 14:51:25 +0200
Subject: [PATCH] Redirect to private link after log in

Added Url to name and default value to null
---
 apps/files/src/components/FileList.vue    | 15 ++++++------
 apps/files/src/components/FilesApp.vue    | 15 +++++++++---
 apps/files/src/components/FilesAppBar.vue |  2 +-
 src/router/index.js                       |  5 +++-
 src/store/index.js                        |  8 +++---
 src/store/links.js                        | 30 +++++++++++++++++++++++
 6 files changed, 59 insertions(+), 16 deletions(-)
 create mode 100644 src/store/links.js

diff --git a/apps/files/src/components/FileList.vue b/apps/files/src/components/FileList.vue
index 20b2cc69258..1dfbca5f9ee 100644
--- a/apps/files/src/components/FileList.vue
+++ b/apps/files/src/components/FileList.vue
@@ -1,7 +1,7 @@
 <template>
   <div class="uk-height-1-1">
     <div class="uk-flex uk-flex-column uk-height-1-1">
-      <div class="uk-overflow-auto uk-flex-auto">
+      <div id="files-list-container" class="uk-overflow-auto uk-flex-auto">
         <oc-table middle divider class="oc-filelist uk-margin-remove-bottom" id="files-list" v-show="!loadingFolder">
           <thead>
             <oc-table-row>
@@ -146,19 +146,18 @@ export default {
         absolutePath = !this.item ? this.configuration.rootFolder : this.item
       }
 
-      const self = this
       this.loadFolder({
         client: this.$client,
         absolutePath: absolutePath,
         $gettext: this.$gettext,
         routeName: this.$route.name
       }).then(() => {
-        const scrollTo = self.$route.query.scrollTo
-        if (scrollTo) {
-          self.$nextTick(() => {
-            self.setHighlightedFile(scrollTo)
-            const file = self.highlightedFile
-            self.$scrollTo(`#file-row-${file.id}`, 500, {
+        const scrollTo = this.$route.query.scrollTo
+        if (scrollTo && this.activeFiles.length > 0) {
+          this.$nextTick(() => {
+            const file = this.activeFiles.find(item => item.name === scrollTo)
+            this.setHighlightedFile(file)
+            this.$scrollTo(`#file-row-${file.id}`, 500, {
               container: '#files-list-container'
             })
           })
diff --git a/apps/files/src/components/FilesApp.vue b/apps/files/src/components/FilesApp.vue
index 86748be50af..a399ea522af 100644
--- a/apps/files/src/components/FilesApp.vue
+++ b/apps/files/src/components/FilesApp.vue
@@ -2,7 +2,7 @@
     <div id="files">
       <files-app-bar />
       <oc-grid class="uk-height-1-1" oc-scroll-offset=".oc-app-bar">
-        <div class="uk-width-expand uk-overflow-auto uk-height-1-1" @dragover="$_ocApp_dragOver" :class="{ 'uk-visible@m' : _sidebarOpen }" id="files-list-container">
+        <div class="uk-width-expand uk-overflow-auto uk-height-1-1" @dragover="$_ocApp_dragOver" :class="{ 'uk-visible@m' : _sidebarOpen }">
           <oc-loader id="files-list-progress" v-if="loadingFolder"></oc-loader>
           <trash-bin v-if="$route.name === 'files-trashbin'" :fileData="activeFiles" />
           <SharedFilesList v-else-if="sharedList" @toggle="toggleFileSelect" :fileData="activeFiles" />
@@ -53,7 +53,7 @@ export default {
   },
   methods: {
     ...mapActions('Files', ['resetFileSelection', 'addFileSelection', 'removeFileSelection', 'dragOver', 'setHighlightedFile', 'toggleFileSelect']),
-    ...mapActions(['openFile', 'showMessage']),
+    ...mapActions(['openFile', 'showMessage', 'setPrivateLinkUrlPath']),
 
     trace () {
       console.info('trace', arguments)
@@ -122,7 +122,7 @@ export default {
 
   computed: {
     ...mapGetters('Files', ['selectedFiles', 'activeFiles', 'dropzone', 'loadingFolder', 'highlightedFile']),
-    ...mapGetters(['extensions']),
+    ...mapGetters(['extensions', 'privateLinkUrlPath']),
 
     _sidebarOpen () {
       return this.highlightedFile !== null
@@ -136,6 +136,15 @@ export default {
     $route () {
       this.setHighlightedFile(null)
     }
+  },
+  beforeMount () {
+    // Redirect to private link after authorization
+    if (this.privateLinkUrlPath) {
+      this.$router.push(this.privateLinkUrlPath)
+      // Resets private link path in store
+      // Needed due to persistance of that state
+      this.setPrivateLinkUrlPath(null)
+    }
   }
 }
 </script>
diff --git a/apps/files/src/components/FilesAppBar.vue b/apps/files/src/components/FilesAppBar.vue
index 3cddf7aeb02..7939785916c 100644
--- a/apps/files/src/components/FilesAppBar.vue
+++ b/apps/files/src/components/FilesAppBar.vue
@@ -5,7 +5,7 @@
       <div class="uk-width-expand">
         <oc-breadcrumb id="files-breadcrumb" :items="breadcrumbs" v-if="showBreadcrumb" home></oc-breadcrumb>
         <span class="uk-flex uk-flex-middle" v-if="!showBreadcrumb">
-          <oc-icon :name="pageIcon" class="uk-margin-small-right"></oc-icon>
+          <oc-icon v-if="pageIcon" :name="pageIcon" class="uk-margin-small-right" />
           <span class="uk-text-lead">{{pageTitle}}</span>
         </span>
       </div>
diff --git a/src/router/index.js b/src/router/index.js
index 94201587db3..f6f7de1b811 100644
--- a/src/router/index.js
+++ b/src/router/index.js
@@ -39,7 +39,7 @@ const router = new Router({
       path: '/f/:fileId',
       name: 'privateLink',
       redirect: '/files/private-link/:fileId',
-      meta: { auth: false, hideHeadbar: true }
+      meta: { hideHeadbar: true }
     },
     {
       path: '/s/:token',
@@ -66,6 +66,9 @@ const router = new Router({
 })
 
 router.beforeEach(function (to, from, next) {
+  if (to.name === 'private-link') {
+    store.dispatch('setPrivateLinkUrlPath', to.path)
+  }
   let authRequired = true
   if (to.meta.auth === false) {
     authRequired = false
diff --git a/src/store/index.js b/src/store/index.js
index 482ce0e87f7..3d616447edd 100644
--- a/src/store/index.js
+++ b/src/store/index.js
@@ -8,14 +8,15 @@ import app from './app'
 import apps from './apps'
 import config from './config'
 import user from './user'
+import links from './links'
 
 Vue.use(Vuex)
 
 const vuexPersist = new VuexPersistence({
   key: 'phoenixState',
   storage: window.localStorage,
-  filter: (mutation) => (['SET_USER', 'SET_TOKEN'].indexOf(mutation.type) > -1),
-  modules: ['user']
+  filter: (mutation) => (['SET_USER', 'SET_TOKEN', 'SET_PRIVATE_LINK_URL_PATH'].indexOf(mutation.type) > -1),
+  modules: ['user', 'links']
 })
 
 const strict = process.env.NODE_ENV === 'development'
@@ -29,7 +30,8 @@ export const Store = new Vuex.Store({
     app,
     apps,
     user,
-    config
+    config,
+    links
   },
   strict
 })
diff --git a/src/store/links.js b/src/store/links.js
new file mode 100644
index 00000000000..96e81af2bee
--- /dev/null
+++ b/src/store/links.js
@@ -0,0 +1,30 @@
+const state = {
+  /**
+   * Private link path in case user needs to log in first
+   * @return {string} path to the file/folder
+   */
+  privateLinkUrlPath: null
+}
+
+const actions = {
+  setPrivateLinkUrlPath ({ commit }, path) {
+    commit('SET_PRIVATE_LINK_URL_PATH', path)
+  }
+}
+
+const mutations = {
+  SET_PRIVATE_LINK_URL_PATH (state, path) {
+    state.privateLinkUrlPath = path
+  }
+}
+
+const getters = {
+  privateLinkUrlPath: state => state.privateLinkUrlPath
+}
+
+export default {
+  state,
+  actions,
+  mutations,
+  getters
+}