From d7ee3ad667ad2f771fce1b9d5c052b63f1fb6f6d Mon Sep 17 00:00:00 2001 From: BlasenhauerJ Date: Wed, 18 Jan 2023 18:18:12 +0100 Subject: [PATCH] fix file manager dropdown *fix the way to select and display dropdown actions for file manager to avoid error with folders that have same names attributs --- src/ui/static/js/utils/file.manager.js | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/src/ui/static/js/utils/file.manager.js b/src/ui/static/js/utils/file.manager.js index 3c2fe7c28..a2944e943 100644 --- a/src/ui/static/js/utils/file.manager.js +++ b/src/ui/static/js/utils/file.manager.js @@ -185,12 +185,9 @@ class FolderDropdown { if ( e.target.closest("div").hasAttribute(`${this.prefix}-action-button`) ) { - const att = e.target - .closest("div") - .getAttribute(`${this.prefix}-action-button`); - const dropEl = document.querySelector( - `[${this.prefix}-action-dropdown="${att}"]` - ); + const dropEl = e.target + .closest(`div[${this.prefix}-element]`) + .querySelector(`[${this.prefix}-action-dropdown]`); //avoid multiple dropdown if (prevActionBtn === "") prevActionBtn = dropEl; if (prevActionBtn !== dropEl) this.hideDropEls(); @@ -557,7 +554,7 @@ class FolderModal { getInfoFromActionBtn(btnEl) { const action = btnEl.getAttribute("value"); const name = btnEl.getAttribute(`${this.prefix}-action-dropdown-btn`); - const folder = document.querySelector(`[${this.prefix}-element='${name}']`); + const folder = btnEl.closest(`[${this.prefix}-element]`); const level = folder.getAttribute("level"); const path = folder.getAttribute("path"); const type = folder.getAttribute("_type");