Bug: Firefox thinks files are named "",

When manipulating certain files in firefox (either via right click or hotkeys, doesn’t seem to matter), cloudcmd seems to lose track of the files actual name. See the following screenshot, the result of dragging the file “KICK.png” from the left pane to the right pane. .

Attempting to continue this operation will instead copy the entire folder containing KICK. Deleteing one of these strange files will delete the whole folder (rip my files).

Behavior varies between files, between refreshes, and even over the course of repeated delete-cancel cycles of the same file without refreshing the page.

Here dragging NEUTRAL.png from left to right produces a valid prompt:

And moments later it ceases to work:

Seems to trend unerringly towards everything showing up as “”.

Same site works fine on Chrome. Error only in firefox. Erroneous firefox behavior persists even in incognito tab, and after having cleared cache on the site.

Console is filled with the following error: e.classList is undefined Unclear if it is related. Does not show up in chrome, so maybe?

• Version: v15.2.0
• Node Version node -v: 14.15.4
• OS (uname -a on Linux): `Linux localhost 5.4.0-62-generic #70-Ubuntu SMP Tue Jan 12 12:45:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
• Browser name/version: Firefox 84.0.2
• Used Command Line Parameters: none (see config changes below / attached file)
• Changed Config:

{
    "name": "Tartarus!",
    "auth": true,
    "username": "root",
    "password": "redacted",
     ...
    "editor": "dword",
     ...
    "port": 40100,
     ...
    "root": "/home/tartarus",
    "prefix": "/tartarusfiles",
     ...
    "contact": false,
     ...
    "configDialog": false,
     ...
    "showConfig": false,
}

[coderaiser]

I am sorta neck deep in another project right now 😅,

So do I 🙂 .

but I could maybe give it a shot?

Nice 👍 Create two functions:

const createNameAttribute = (a) => `js-file-${btoa(encodeURI(a))}`;
const parseNameAttribute = (a) => a.replace(`js-file-, '');

And use createNameAttribute in setCurrentName and parseNameAttribute in getCurrentName:

module.exports.getCurrentName = (currentFile) => {
    const current = currentFile || DOM.getCurrentFile();
    
    if (!current)
        return '';
      
+      const attribute = current.getAttribute('data-name');  
+      return parseNameAttribute(attribute);

-    const link = DOM.getCurrentLink(current);
- 
-    if (!link)
-        return '';
-    
-    return decode(link.title)
-        .replace(NBSP_REG, SPACE);
};

[coderaiser]

Make a PR, I’ll take a look.