From 8ba84f68c78500809498c14199a0e5bb02f2cd63 Mon Sep 17 00:00:00 2001
From: Asif Takavala <asifmomin798@gmail.com>
Date: Tue, 15 Feb 2022 06:10:04 +0000
Subject: [PATCH] fixed uploaded attachment not removed

---
 .../V1/Admin/Settings/CompanyController.php   |  6 +++++
 .../V1/Customer/General/ProfileController.php |  3 +++
 app/Models/Expense.php                        | 11 +++++----
 resources/scripts/admin/stores/expense.js     |  9 ++++----
 .../scripts/admin/views/expenses/Create.vue   |  9 ++++++--
 .../admin/views/settings/AccountSetting.vue   | 11 +++++++--
 .../views/settings/CompanyInfoSettings.vue    | 23 ++++++++++++-------
 .../components/base/BaseFileUploader.vue      |  2 ++
 .../views/settings/CustomerSettings.vue       |  5 ++++
 9 files changed, 59 insertions(+), 20 deletions(-)

diff --git a/app/Http/Controllers/V1/Admin/Settings/CompanyController.php b/app/Http/Controllers/V1/Admin/Settings/CompanyController.php
index 5998f312..25381299 100644
--- a/app/Http/Controllers/V1/Admin/Settings/CompanyController.php
+++ b/app/Http/Controllers/V1/Admin/Settings/CompanyController.php
@@ -71,6 +71,9 @@ class CompanyController extends Controller
 
         $data = json_decode($request->company_logo);
 
+        if (isset($request->is_company_logo_removed) && (bool) $request->is_company_logo_removed) {
+            $company->clearMediaCollection('logo');
+        }
         if ($data) {
             $company = Company::find($request->header('company'));
 
@@ -98,6 +101,9 @@ class CompanyController extends Controller
     {
         $user = auth()->user();
 
+        if (isset($request->is_admin_avatar_removed) && (bool) $request->is_admin_avatar_removed) {
+            $user->clearMediaCollection('admin_avatar');
+        }
         if ($user && $request->hasFile('admin_avatar')) {
             $user->clearMediaCollection('admin_avatar');
 
diff --git a/app/Http/Controllers/V1/Customer/General/ProfileController.php b/app/Http/Controllers/V1/Customer/General/ProfileController.php
index a80c7706..711e120c 100644
--- a/app/Http/Controllers/V1/Customer/General/ProfileController.php
+++ b/app/Http/Controllers/V1/Customer/General/ProfileController.php
@@ -17,6 +17,9 @@ class ProfileController extends Controller
 
         $customer->update($request->validated());
 
+        if (isset($request->is_customer_avatar_removed) && (bool) $request->is_customer_avatar_removed) {
+            $customer->clearMediaCollection('customer_avatar');
+        }
         if ($customer && $request->hasFile('customer_avatar')) {
             $customer->clearMediaCollection('customer_avatar');
 
diff --git a/app/Models/Expense.php b/app/Models/Expense.php
index de02cc91..d5f69d0d 100644
--- a/app/Models/Expense.php
+++ b/app/Models/Expense.php
@@ -126,14 +126,14 @@ class Expense extends Model implements HasMedia
     {
         foreach (explode(' ', $search) as $term) {
             $query->whereHas('category', function ($query) use ($term) {
-                $query->where('name', 'LIKE', '%'.$term.'%');
+                $query->where('name', 'LIKE', '%' . $term . '%');
             });
         }
     }
 
     public function scopeWhereNotes($query, $search)
     {
-        $query->where('notes', 'LIKE', '%'.$search.'%');
+        $query->where('notes', 'LIKE', '%' . $search . '%');
     }
 
     public function scopeWhereCategory($query, $categoryId)
@@ -188,9 +188,9 @@ class Expense extends Model implements HasMedia
     {
         foreach (explode(' ', $search) as $term) {
             $query->whereHas('category', function ($query) use ($term) {
-                $query->where('name', 'LIKE', '%'.$term.'%');
+                $query->where('name', 'LIKE', '%' . $term . '%');
             })
-                ->orWhere('notes', 'LIKE', '%'.$term.'%');
+                ->orWhere('notes', 'LIKE', '%' . $term . '%');
         }
     }
 
@@ -262,6 +262,9 @@ class Expense extends Model implements HasMedia
             ExchangeRateLog::addExchangeRateLog($this);
         }
 
+        if (isset($request->is_attachment_receipt_removed) && (bool) $request->is_attachment_receipt_removed) {
+            $this->clearMediaCollection('receipts');
+        }
         if ($request->hasFile('attachment_receipt')) {
             $this->clearMediaCollection('receipts');
             $this->addMediaFromRequest('attachment_receipt')->toMediaCollection('receipts');
diff --git a/resources/scripts/admin/stores/expense.js b/resources/scripts/admin/stores/expense.js
index 7543b12a..706762f2 100644
--- a/resources/scripts/admin/stores/expense.js
+++ b/resources/scripts/admin/stores/expense.js
@@ -61,15 +61,15 @@ export const useExpenseStore = (useWindow = false) => {
                 Object.assign(this.currentExpense, response.data.data)
                 this.currentExpense.selectedCurrency =
                   response.data.data.currency
-
-                if (response.data.data.attachment_receipt) {
+                this.currentExpense.attachment_receipt = null
+                if (response.data.data.attachment_receipt_url) {
                   if (
                     utils.isImageFile(
                       response.data.data.attachment_receipt_meta.mime_type
                     )
                   ) {
                     this.currentExpense.receiptFiles = [
-                      { image: `/expenses/${id}/receipt` },
+                      { image: `/reports/expenses/${id}/receipt?${response.data.data.attachment_receipt_meta.uuid}` },
                     ]
                   } else {
                     this.currentExpense.receiptFiles = [
@@ -118,12 +118,13 @@ export const useExpenseStore = (useWindow = false) => {
         })
       },
 
-      updateExpense({ id, data }) {
+      updateExpense({ id, data, isAttachmentReceiptRemoved }) {
         const notificationStore = useNotificationStore()
 
         const formData = utils.toFormData(data)
 
         formData.append('_method', 'PUT')
+        formData.append('is_attachment_receipt_removed', isAttachmentReceiptRemoved)
 
         return new Promise((resolve) => {
           axios.post(`/api/v1/expenses/${id}`, formData).then((response) => {
diff --git a/resources/scripts/admin/views/expenses/Create.vue b/resources/scripts/admin/views/expenses/Create.vue
index ca36c8ad..d79ade62 100644
--- a/resources/scripts/admin/views/expenses/Create.vue
+++ b/resources/scripts/admin/views/expenses/Create.vue
@@ -21,7 +21,7 @@
 
         <template #actions>
           <BaseButton
-            v-if="isEdit && expenseStore.currentExpense.attachment_receipt"
+            v-if="isEdit && expenseStore.currentExpense.attachment_receipt_url"
             :href="receiptDownloadUrl"
             tag="a"
             variant="primary-outline"
@@ -317,6 +317,7 @@ const globalStore = useGlobalStore()
 let isSaving = ref(false)
 let isFetchingInitialData = ref(false)
 const expenseValidationScope = 'newExpense'
+const isAttachmentReceiptRemoved = ref(false)
 
 const rules = computed(() => {
   return {
@@ -381,7 +382,7 @@ const pageTitle = computed(() =>
 )
 
 const receiptDownloadUrl = computed(() =>
-  isEdit.value ? `/expenses/${route.params.id}/download-receipt` : ''
+  isEdit.value ? `/reports/expenses/${route.params.id}/download-receipt` : ''
 )
 
 expenseStore.resetCurrentExpenseData()
@@ -395,6 +396,7 @@ function onFileInputChange(fileName, file) {
 
 function onFileInputRemove() {
   expenseStore.currentExpense.attachment_receipt = null
+  isAttachmentReceiptRemoved.value = true
 }
 
 function openCategoryModal() {
@@ -460,11 +462,14 @@ async function submitForm() {
       await expenseStore.updateExpense({
         id: route.params.id,
         data: formData,
+        isAttachmentReceiptRemoved: isAttachmentReceiptRemoved.value
       })
     } else {
       await expenseStore.addExpense(formData)
     }
     isSaving.value = false
+    expenseStore.currentExpense.attachment_receipt = null
+    isAttachmentReceiptRemoved.value = false
     router.push('/admin/expenses')
   } catch (err) {
     console.error(err)
diff --git a/resources/scripts/admin/views/settings/AccountSetting.vue b/resources/scripts/admin/views/settings/AccountSetting.vue
index aaf8b8d8..b7fdeede 100644
--- a/resources/scripts/admin/views/settings/AccountSetting.vue
+++ b/resources/scripts/admin/views/settings/AccountSetting.vue
@@ -118,6 +118,7 @@ const { t } = useI18n()
 let isSaving = ref(false)
 let avatarFileBlob = ref(null)
 let imgFiles = ref([])
+const isAdminAvatarRemoved = ref(false)
 
 if (userStore.currentUser.avatar) {
   imgFiles.value.push({
@@ -170,6 +171,7 @@ function onFileInputChange(fileName, file) {
 
 function onFileInputRemove() {
   avatarFileBlob.value = null
+  isAdminAvatarRemoved.value = true
 }
 
 async function updateUserData() {
@@ -209,12 +211,17 @@ async function updateUserData() {
     if (response.data.data) {
       isSaving.value = false
 
-      if (avatarFileBlob.value) {
+      if (avatarFileBlob.value || isAdminAvatarRemoved.value) {
         let avatarData = new FormData()
 
-        avatarData.append('admin_avatar', avatarFileBlob.value)
+        if (avatarFileBlob.value) {
+          avatarData.append('admin_avatar', avatarFileBlob.value)
+        }
+        avatarData.append('is_admin_avatar_removed', isAdminAvatarRemoved.value)
 
         await userStore.uploadAvatar(avatarData)
+        avatarFileBlob.value = null
+        isAdminAvatarRemoved.value = false
       }
 
       userForm.password = ''
diff --git a/resources/scripts/admin/views/settings/CompanyInfoSettings.vue b/resources/scripts/admin/views/settings/CompanyInfoSettings.vue
index 6d0cb70f..357b7dd6 100644
--- a/resources/scripts/admin/views/settings/CompanyInfoSettings.vue
+++ b/resources/scripts/admin/views/settings/CompanyInfoSettings.vue
@@ -180,6 +180,7 @@ utils.mergeSettings(companyForm, {
 let previewLogo = ref([])
 let logoFileBlob = ref(null)
 let logoFileName = ref(null)
+const isCompanyLogoRemoved = ref(false)
 
 if (companyForm.logo) {
   previewLogo.value.push({
@@ -218,6 +219,7 @@ function onFileInputChange(fileName, file, fileCount, fileList) {
 
 function onFileInputRemove() {
   logoFileBlob.value = null
+  isCompanyLogoRemoved.value = true
 }
 
 async function updateCompanyData() {
@@ -232,18 +234,23 @@ async function updateCompanyData() {
   const res = await companyStore.updateCompany(companyForm)
 
   if (res.data.data) {
-    if (logoFileBlob.value) {
+    if (logoFileBlob.value || isCompanyLogoRemoved.value) {
       let logoData = new FormData()
 
-      logoData.append(
-        'company_logo',
-        JSON.stringify({
-          name: logoFileName.value,
-          data: logoFileBlob.value,
-        })
-      )
+      if (logoFileBlob.value) {
+        logoData.append(
+          'company_logo',
+          JSON.stringify({
+            name: logoFileName.value,
+            data: logoFileBlob.value,
+          })
+        )
+      }
+      logoData.append('is_company_logo_removed', isCompanyLogoRemoved.value)
 
       await companyStore.updateCompanyLogo(logoData)
+      logoFileBlob.value = null
+      isCompanyLogoRemoved.value = false
     }
 
     isSaving.value = false
diff --git a/resources/scripts/components/base/BaseFileUploader.vue b/resources/scripts/components/base/BaseFileUploader.vue
index ccc0a9fa..6fc78c56 100644
--- a/resources/scripts/components/base/BaseFileUploader.vue
+++ b/resources/scripts/components/base/BaseFileUploader.vue
@@ -33,6 +33,7 @@
       :name="inputFieldName"
       :accept="accept"
       class="absolute z-10 w-full h-full opacity-0 cursor-pointer"
+      @click="$event.target.value=null"
       @change="
         onChange(
           $event.target.name,
@@ -557,6 +558,7 @@ function onAvatarRemove(image) {
 
 function onFileRemove(index) {
   localFiles.value.splice(index, 1)
+  emit('remove', index)
 }
 
 function getDefaultAvatar() {
diff --git a/resources/scripts/customer/views/settings/CustomerSettings.vue b/resources/scripts/customer/views/settings/CustomerSettings.vue
index dbfb2fd2..0a4ab3c1 100644
--- a/resources/scripts/customer/views/settings/CustomerSettings.vue
+++ b/resources/scripts/customer/views/settings/CustomerSettings.vue
@@ -153,6 +153,7 @@ let isSaving = ref(false)
 let avatarFileBlob = ref(null)
 let isShowPassword = ref(false)
 let isShowConfirmPassword = ref(false)
+const isCustomerAvatarRemoved = ref(false)
 
 if (userStore.userForm.avatar) {
   imgFiles.value.push({
@@ -206,6 +207,7 @@ function onFileInputChange(fileName, file) {
 
 function onFileInputRemove() {
   avatarFileBlob.value = null
+  isCustomerAvatarRemoved.value = true
 }
 
 function updateCustomerData() {
@@ -230,6 +232,7 @@ function updateCustomerData() {
   if (avatarFileBlob.value) {
     data.append('customer_avatar', avatarFileBlob.value)
   }
+  data.append('is_customer_avatar_removed', isCustomerAvatarRemoved.value)
 
   userStore
     .updateCurrentUser({
@@ -243,6 +246,8 @@ function updateCustomerData() {
           state.userForm.password = ''
           state.userForm.confirm_password = ''
         })
+        avatarFileBlob.value = null
+        isCustomerAvatarRemoved.value = false
       }
     })
     .catch((error) => {