Skip to content

a fix - but I don't know how to GIT it on there. :) #463

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
nocturnalguru opened this issue Sep 17, 2024 · 0 comments · May be fixed by #461
Open

a fix - but I don't know how to GIT it on there. :) #463

nocturnalguru opened this issue Sep 17, 2024 · 0 comments · May be fixed by #461

Comments

@nocturnalguru
Copy link

had this issue..
Uncaught ReferenceError: tmp is not defined

/home/minipc/.pulsar/packages/ftp-remote-edit/lib/views/tree-view.js:612
Hide Stack Trace
ReferenceError: tmp is not defined
at TreeView.remoteKeyboardNavigationDown (tree-view.js:612:10)
at TreeView.remoteKeyboardNavigation (tree-view.js:556:14)
at HTMLDivElement. (tree-view.js:126:38)
at HTMLDivElement.dispatch (/home/minipc/.pulsar/packages/ftp-remote-edit/node_modules/jquery/dist/jquery.js:4435:9)
at HTMLDivElement.elemData.handle (/home/minipc/.pulsar/packages/ftp-remote-edit/node_modules/jquery/dist/jquery.js:4121:28)
The error was thrown from the ftp-remote-edit package.

Took a look at line 612 of this file: https://github.com/h3imdall/ftp-remote-edit/blob/master/lib/views/tree-view.js

tmp = current;

changed it to

let tmp = current;

No more error.

It's an issue I've had to fix half a dozen times, and I would update, but I dont know how to do a pull / push with the update and don't want to mess this project up.

It's a great project and I wish I could help directly.

Can someone push this one. It's a pretty obvious error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant