From 267556b39dc7f067f03107b29d1ec365dcbaaedd Mon Sep 17 00:00:00 2001
From: Peter Papp <peterpapp@makingcg.com>
Date: Sat, 30 Jan 2021 14:01:36 +0100
Subject: [PATCH] - Prevent submit shared form - Send shared link from
 FileInfoPanel.vue

---
 app/Http/Resources/ShareResource.php                   |  1 -
 resources/js/components/FilesView/FileInfoPanel.vue    |  2 +-
 resources/js/components/Others/Forms/CopyInput.vue     | 10 +++++++---
 .../js/components/Others/Forms/MultiEmailInput.vue     |  1 +
 resources/js/components/Others/ShareCreate.vue         |  6 +-----
 resources/js/components/Others/ShareEdit.vue           |  9 ++++-----
 6 files changed, 14 insertions(+), 15 deletions(-)

diff --git a/app/Http/Resources/ShareResource.php b/app/Http/Resources/ShareResource.php
index a089a92a..3f37d5ae 100644
--- a/app/Http/Resources/ShareResource.php
+++ b/app/Http/Resources/ShareResource.php
@@ -21,7 +21,6 @@ class ShareResource extends JsonResource
                 'attributes' => [
                     'permission' => $this->permission,
                     'protected'  => (int) $this->protected,
-                    'email_share'=> $request->has('emails') ? true : false,
                     'item_id'    => (int) $this->item_id,
                     'expire_in'  => (int) $this->expire_in,
                     'token'      => $this->token,
diff --git a/resources/js/components/FilesView/FileInfoPanel.vue b/resources/js/components/FilesView/FileInfoPanel.vue
index 4de0ba74..680179aa 100644
--- a/resources/js/components/FilesView/FileInfoPanel.vue
+++ b/resources/js/components/FilesView/FileInfoPanel.vue
@@ -53,7 +53,7 @@
                 <div class="sharelink">
                     <lock-icon v-if="isLocked" @click="shareItemOptions" class="lock-icon" size="17"></lock-icon>
                     <unlock-icon v-if="! isLocked" @click="shareItemOptions" class="lock-icon" size="17"></unlock-icon>
-                    <CopyInput class="copy-sharelink" size="small" :value="fileInfoDetail[0].shared.link"/>
+                    <CopyInput class="copy-sharelink" size="small" :item="fileInfoDetail[0]"/>
                 </div>
             </ListInfoItem>
 
diff --git a/resources/js/components/Others/Forms/CopyInput.vue b/resources/js/components/Others/Forms/CopyInput.vue
index 6c3b1912..db069a4c 100644
--- a/resources/js/components/Others/Forms/CopyInput.vue
+++ b/resources/js/components/Others/Forms/CopyInput.vue
@@ -1,6 +1,6 @@
 <template>
     <div class="inline-wrapper icon-append copy-input" :class="size" @click="copyUrl">
-        <input ref="sel" :value="value" id="link-input" type="text" class="input-text" readonly>
+        <input ref="sel" :value="item.shared.link" id="link-input" type="text" class="input-text" readonly>
         <div class="multi-icon">
             <div class="icon-item">
                 <link-icon v-if="! isCopiedLink" size="14"></link-icon>
@@ -19,7 +19,7 @@ import { events } from '@/bus'
 
 export default {
     name: 'CopyInput',
-    props: ['size', 'value'],
+    props: ['size', 'item'],
     components: {
         CheckIcon,
         LinkIcon,
@@ -32,7 +32,11 @@ export default {
     },
     methods: {
         menuForEmail() {
-            events.$emit('openSendToRecipientsMenu')
+            events.$emit('popup:open', {
+                name: 'share-edit',
+                item: this.item,
+                sentToEmail: true,
+            })
         },
         copyUrl() {
 
diff --git a/resources/js/components/Others/Forms/MultiEmailInput.vue b/resources/js/components/Others/Forms/MultiEmailInput.vue
index 318fcb98..b6ff31d9 100644
--- a/resources/js/components/Others/Forms/MultiEmailInput.vue
+++ b/resources/js/components/Others/Forms/MultiEmailInput.vue
@@ -97,6 +97,7 @@ export default {
 }
 
 .input-wrapper {
+    margin-bottom: 0;
     background: white;
     max-width: 100%;
     display: flex;
diff --git a/resources/js/components/Others/ShareCreate.vue b/resources/js/components/Others/ShareCreate.vue
index 13d3b02d..6276b4d9 100644
--- a/resources/js/components/Others/ShareCreate.vue
+++ b/resources/js/components/Others/ShareCreate.vue
@@ -70,7 +70,7 @@
             <div v-if="isGeneratedShared" class="form-wrapper">
                 <div class="input-wrapper">
                     <label class="input-label">{{ this.shareViaEmail ? $t('shared_form.label_share_vie_email') : $t('shared_form.label_shared_url') }}:</label>
-                    <CopyInput size="small" :value="shareLink" />
+                    <CopyInput size="small" :item="pickedItem" />
                 </div>
             </div>
         </PopupContent>
@@ -210,10 +210,6 @@
 
                         // End loading
                         this.isLoading = false
-
-                        this.shareViaEmail = response.data.data.attributes.email_share
-
-                        this.shareLink = response.data.data.attributes.link
                         this.isGeneratedShared = true
 
                         this.$store.commit('UPDATE_SHARED_ITEM', response.data.data.attributes)
diff --git a/resources/js/components/Others/ShareEdit.vue b/resources/js/components/Others/ShareEdit.vue
index 26cec458..70703670 100644
--- a/resources/js/components/Others/ShareEdit.vue
+++ b/resources/js/components/Others/ShareEdit.vue
@@ -16,7 +16,7 @@
 
             <div v-if="! sendToRecipientsMenu || (sendToRecipientsMenu && isEmailSended)" class="input-wrapper copy-input">
                 <label class="input-label">{{ $t('shared_form.label_share_vie_email') }}:</label>
-                <CopyInput size="small" :value="pickedItem.shared.link" />
+                <CopyInput size="small" :item="pickedItem" />
             </div>
 
             <ValidationObserver v-if="sendToRecipientsMenu && !isEmailSended" v-slot="{ invalid }" ref="shareEmail" tag="form" class="form-wrapper">
@@ -315,10 +315,6 @@
                 this.emails = emails
             })
 
-            events.$on('openSendToRecipientsMenu', () => {
-                this.sendToRecipientsMenu = true
-            })
-
             // Show popup
             events.$on('popup:open', args => {
 
@@ -339,6 +335,9 @@
                 if (args.item.shared.expire_in)
                     this.isMoreOptions = true
 
+                if (args.sentToEmail)
+                    this.sendToRecipientsMenu = true
+
                 this.canChangePassword = args.item.shared.protected
             })