mirror of
https://github.com/LibreSign/libresign.git
synced 2025-12-18 05:20:45 +01:00
fix: close sidebar when deleting the selected file
- Changed condition from checking 'uuid' to 'nodeId' in delete method - Added logic to close sidebar when the deleted file is the currently selected one - Simplified deleteMultiple method to rely on delete method's cleanup This fixes the issue where the sidebar remained open after deleting the only/selected file, both when the file was just uploaded or when the page was already loaded. Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
This commit is contained in:
parent
778067f5f1
commit
dbfd75e7ff
1 changed files with 8 additions and 5 deletions
|
|
@ -254,7 +254,7 @@ export const useFilesStore = function(...args) {
|
|||
},
|
||||
async delete(file, deleteFile) {
|
||||
file = this.getFile(file)
|
||||
if (file?.uuid !== undefined) {
|
||||
if (file?.nodeId) {
|
||||
const url = deleteFile
|
||||
? '/apps/libresign/api/v1/file/file_id/{fileId}'
|
||||
: '/apps/libresign/api/v1/sign/file_id/{fileId}'
|
||||
|
|
@ -262,6 +262,11 @@ export const useFilesStore = function(...args) {
|
|||
fileId: file.nodeId,
|
||||
}))
|
||||
.then(() => {
|
||||
if (this.selectedNodeId === file.nodeId) {
|
||||
const sidebarStore = useSidebarStore()
|
||||
sidebarStore.hideSidebar()
|
||||
this.selectedNodeId = 0
|
||||
}
|
||||
del(this.files, file.nodeId)
|
||||
const index = this.ordered.indexOf(file.nodeId)
|
||||
if (index > -1) {
|
||||
|
|
@ -273,11 +278,9 @@ export const useFilesStore = function(...args) {
|
|||
},
|
||||
async deleteMultiple(nodeIds, deleteFile) {
|
||||
this.loading = true
|
||||
nodeIds.forEach(async nodeId => {
|
||||
for (const nodeId of nodeIds) {
|
||||
await this.delete(this.files[nodeId], deleteFile)
|
||||
})
|
||||
const toRemove = nodeIds.filter(nodeId => (!this.files[nodeId]?.uuid))
|
||||
del(this.files, ...toRemove)
|
||||
}
|
||||
this.loading = false
|
||||
},
|
||||
async upload({ file, name }) {
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue