From 045d585df439dca02e084e3b4f909124f3ea967a Mon Sep 17 00:00:00 2001
From: kuaifan <aipaw@live.cn>
Date: Thu, 23 Jun 2022 14:37:54 +0800
Subject: [PATCH] =?UTF-8?q?perf:=20=E4=BC=98=E5=8C=96pdf=E6=B5=8F=E8=A7=88?=
 =?UTF-8?q?=E6=96=B9=E5=BC=8F?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/Http/Controllers/IndexController.php      | 35 +++++++++++++++++++
 app/Models/File.php                           | 19 ++++------
 app/Models/FileContent.php                    | 19 ++++------
 resources/assets/js/components/OnlyOffice.vue | 10 ++----
 .../pages/manage/components/FileContent.vue   | 16 ++++-----
 .../pages/manage/components/FilePreview.vue   | 16 ++++-----
 .../js/pages/manage/components/IFrame.vue     |  8 ++++-
 resources/assets/js/pages/single/fileMsg.vue  |  3 +-
 resources/assets/js/pages/single/fileTask.vue |  3 +-
 9 files changed, 73 insertions(+), 56 deletions(-)

diff --git a/app/Http/Controllers/IndexController.php b/app/Http/Controllers/IndexController.php
index 0a17dbde2..c68a7fb73 100755
--- a/app/Http/Controllers/IndexController.php
+++ b/app/Http/Controllers/IndexController.php
@@ -2,6 +2,7 @@
 
 namespace App\Http\Controllers;
 
+use App\Models\File;
 use App\Module\Base;
 use App\Module\Ihttp;
 use App\Module\RandomColor;
@@ -276,6 +277,40 @@ class IndexController extends InvokeController
         ];
     }
 
+    /**
+     * 预览文件
+     * @return array|mixed
+     */
+    public function online__preview()
+    {
+        $key = trim(Request::input('key'));
+        //
+        $data = parse_url($key);
+        $path = Arr::get($data, 'path');
+        $file = public_path($path);
+        //
+        if (file_exists($file)) {
+            parse_str($data['query'], $query);
+            $name = Arr::get($query, 'name');
+            $ext = strtolower(Arr::get($query, 'ext'));
+            if ($ext === 'pdf') {
+                return response()->download($file, $name, [], 'inline');
+            }
+            //
+            if (in_array($ext, File::localExt)) {
+                $url = Base::fillUrl($path);
+            } else {
+                $url = 'http://' . env('APP_IPPR') . '.3/' . $path;
+            }
+            $url = Base::urlAddparameter($url, [
+                'fullfilename' => $name
+            ]);
+            $toUrl = Base::fillUrl("fileview/onlinePreview?url=" . urlencode(base64_encode($url)));
+            return Redirect::to($toUrl, 301);
+        }
+        return abort(404);
+    }
+
     /**
      * 设置语言和皮肤
      * @return \Illuminate\Contracts\Foundation\Application|\Illuminate\Contracts\View\Factory|\Illuminate\Contracts\View\View
diff --git a/app/Models/File.php b/app/Models/File.php
index 25ffad551..234f30b65 100644
--- a/app/Models/File.php
+++ b/app/Models/File.php
@@ -455,21 +455,14 @@ class File extends AbstractModel
                 else
                 {
                     // 其他预览
-                    if (in_array($fileExt, File::localExt)) {
-                        $url = Base::fillUrl($filePath);
-                    } else {
-                        $url = 'http://' . env('APP_IPPR') . '.3/' . $filePath;
-                    }
-                    if ($fileExt != 'pdf') {
-                        $fileDotExt = ".{$fileExt}";
-                        $fullFileName = Base::rightDelete($fileName, $fileDotExt) . $fileDotExt;
-                        $url = Base::urlAddparameter($url, [
-                            'fullfilename' => $fullFileName
-                        ]);
-                    }
+                    $name = Base::rightDelete($fileName, ".{$fileExt}") . ".{$fileExt}";
                     $data['content'] = [
                         'preview' => true,
-                        'url' => base64_encode($url),
+                        'name' => $name,
+                        'key' => urlencode(Base::urlAddparameter($filePath, [
+                            'name' => $name,
+                            'ext' => $fileExt
+                        ])),
                     ];
                     $data['file_mode'] = 'preview';
                 }
diff --git a/app/Models/FileContent.php b/app/Models/FileContent.php
index 8a3e20b80..85b530689 100644
--- a/app/Models/FileContent.php
+++ b/app/Models/FileContent.php
@@ -50,19 +50,12 @@ class FileContent extends AbstractModel
         $fileExt = $array['ext'];
         $fileName = $array['name'];
         $filePath = $array['path'];
-        if (in_array($fileExt, File::localExt)) {
-            $url = Base::fillUrl($filePath);
-        } else {
-            $url = 'http://' . env('APP_IPPR') . '.3/' . $filePath;
-        }
-        if ($fileExt != 'pdf') {
-            $fileDotExt = ".{$fileExt}";
-            $fullFileName = Base::rightDelete($fileName, $fileDotExt) . $fileDotExt;
-            $url = Base::urlAddparameter($url, [
-                'fullfilename' => $fullFileName
-            ]);
-        }
-        return Base::fillUrl("fileview/onlinePreview?url=" . urlencode(base64_encode($url)));
+        $name = Base::rightDelete($fileName, ".{$fileExt}") . ".{$fileExt}";
+        $key = urlencode(Base::urlAddparameter($filePath, [
+            'name' => $name,
+            'ext' => $fileExt
+        ]));
+        return Base::fillUrl("online/preview/{$name}?key={$key}");
     }
 
     /**
diff --git a/resources/assets/js/components/OnlyOffice.vue b/resources/assets/js/components/OnlyOffice.vue
index 30c9d5b6a..c540da61f 100644
--- a/resources/assets/js/components/OnlyOffice.vue
+++ b/resources/assets/js/components/OnlyOffice.vue
@@ -4,7 +4,7 @@
             <Alert v-if="loadError" class="load-error" type="error" show-icon>{{$L('组件加载失败！')}}</Alert>
             <div :id="id" class="placeholder"></div>
         </template>
-        <IFrame v-else class="preview-iframe" :src="previewUrl" @on-message="onMessage"/>
+        <IFrame v-else class="preview-iframe" :src="previewUrl" @on-load="onFrameLoad"/>
         <div v-if="loading" class="office-loading"><Loading/></div>
     </div>
 </template>
@@ -187,12 +187,8 @@ export default {
     },
 
     methods: {
-        onMessage(data) {
-            switch (data.action) {
-                case 'ready':
-                    this.loading = false;
-                    break
-            }
+        onFrameLoad() {
+            this.loading = false;
         },
 
         getType(type) {
diff --git a/resources/assets/js/pages/manage/components/FileContent.vue b/resources/assets/js/pages/manage/components/FileContent.vue
index 543b76a7c..d3188a0a9 100644
--- a/resources/assets/js/pages/manage/components/FileContent.vue
+++ b/resources/assets/js/pages/manage/components/FileContent.vue
@@ -1,6 +1,6 @@
 <template>
     <div v-if="ready" class="file-content">
-        <IFrame v-if="isPreview" class="preview-iframe" :src="previewUrl" @on-message="onMessage"/>
+        <IFrame v-if="isPreview" class="preview-iframe" :src="previewUrl" @on-load="onFrameLoad"/>
         <template v-else-if="contentDetail">
             <EPopover
                 v-if="['word', 'excel', 'ppt'].includes(file.type)"
@@ -278,10 +278,10 @@ export default {
 
         previewUrl() {
             if (this.isPreview) {
-                return $A.apiUrl("../fileview/onlinePreview?url=" + encodeURIComponent(this.contentDetail.url))
-            } else {
-                return '';
+                const {name, key} = this.contentDetail;
+                return $A.apiUrl(`../online/preview/${name}?key=${key}`)
             }
+            return '';
         },
     },
 
@@ -311,12 +311,8 @@ export default {
             }
         },
 
-        onMessage(data) {
-            switch (data.action) {
-                case 'ready':
-                    this.loadPreview = false;
-                    break
-            }
+        onFrameLoad() {
+            this.loadPreview = false;
         },
 
         keySave(e) {
diff --git a/resources/assets/js/pages/manage/components/FilePreview.vue b/resources/assets/js/pages/manage/components/FilePreview.vue
index 6f8d20d08..41574111d 100644
--- a/resources/assets/js/pages/manage/components/FilePreview.vue
+++ b/resources/assets/js/pages/manage/components/FilePreview.vue
@@ -1,6 +1,6 @@
 <template>
     <div class="file-preview">
-        <IFrame v-if="isPreview" class="preview-iframe" :src="previewUrl" @on-message="onMessage"/>
+        <IFrame v-if="isPreview" class="preview-iframe" :src="previewUrl" @on-load="onFrameLoad"/>
         <template v-else-if="contentDetail">
             <div v-show="headerShow && !['word', 'excel', 'ppt'].includes(file.type)" class="edit-header">
                 <div class="header-title">
@@ -110,20 +110,16 @@ export default {
 
         previewUrl() {
             if (this.isPreview) {
-                return $A.apiUrl("../fileview/onlinePreview?url=" + encodeURIComponent(this.contentDetail.url))
-            } else {
-                return '';
+                const {name, key} = this.contentDetail;
+                return $A.apiUrl(`../online/preview/${name}?key=${key}`)
             }
+            return '';
         },
     },
 
     methods: {
-        onMessage(data) {
-            switch (data.action) {
-                case 'ready':
-                    this.loadPreview = false;
-                    break
-            }
+        onFrameLoad() {
+            this.loadPreview = false;
         },
 
         getContent() {
diff --git a/resources/assets/js/pages/manage/components/IFrame.vue b/resources/assets/js/pages/manage/components/IFrame.vue
index 524e6b327..4aa44d0f0 100644
--- a/resources/assets/js/pages/manage/components/IFrame.vue
+++ b/resources/assets/js/pages/manage/components/IFrame.vue
@@ -1,5 +1,5 @@
 <template>
-    <iframe v-if="src" ref="iframe" :src="src"></iframe>
+    <iframe v-show="src" ref="iframe" :src="src"></iframe>
 </template>
 
 <script>
@@ -13,14 +13,20 @@ export default {
     },
 
     mounted() {
+        this.$refs.iframe.addEventListener('load', this.handleLoad)
         window.addEventListener('message', this.handleMessage)
     },
 
     beforeDestroy() {
+        this.$refs.iframe.removeEventListener('load', this.handleLoad)
         window.removeEventListener('message', this.handleMessage)
     },
 
     methods: {
+        handleLoad() {
+            this.$emit("on-load")
+        },
+
         handleMessage({data, source}) {
             if (source !== this.$refs.iframe?.contentWindow) {
                 return;
diff --git a/resources/assets/js/pages/single/fileMsg.vue b/resources/assets/js/pages/single/fileMsg.vue
index 3b8111a7d..18d5abdd9 100644
--- a/resources/assets/js/pages/single/fileMsg.vue
+++ b/resources/assets/js/pages/single/fileMsg.vue
@@ -132,7 +132,8 @@ export default {
         },
 
         previewUrl() {
-            return $A.apiUrl("../fileview/onlinePreview?url=" + encodeURIComponent(this.msgDetail.content.url))
+            const {name, key} = this.msgDetail.content;
+            return $A.apiUrl(`../online/preview/${name}?key=${key}`)
         }
     },
     methods: {
diff --git a/resources/assets/js/pages/single/fileTask.vue b/resources/assets/js/pages/single/fileTask.vue
index 932634435..1e03ab106 100644
--- a/resources/assets/js/pages/single/fileTask.vue
+++ b/resources/assets/js/pages/single/fileTask.vue
@@ -119,7 +119,8 @@ export default {
         },
 
         previewUrl() {
-            return $A.apiUrl("../fileview/onlinePreview?url=" + encodeURIComponent(this.fileDetail.content.url))
+            const {name, key} = this.fileDetail.content;
+            return $A.apiUrl(`../online/preview/${name}?key=${key}`)
         }
     },
     methods: {