Skip to content
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

Music v0.17.2 interferes delete on the Files app #798

Closed
AlexShelest opened this issue Nov 5, 2020 · 6 comments
Closed

Music v0.17.2 interferes delete on the Files app #798

AlexShelest opened this issue Nov 5, 2020 · 6 comments

Comments

@AlexShelest
Copy link

AlexShelest commented Nov 5, 2020

NC 19.0.4

After updating music app to 0.17.2, delete button not working.

Uncaught TypeError: _.pluck is not a function
_onClickDeleteSelected https://cloudhelpme.ru/js/files/merged-index.js?v=54e31561-37:4951
multiSelectMenuClick https://cloudhelpme.ru/js/files/merged-index.js?v=54e31561-37:4386
_onClickAction https://cloudhelpme.ru/js/files/merged-index.js?v=54e31561-37:7806
dispatch https://cloudhelpme.ru/core/js/dist/main.js?v=54e31561-37:1
handle https://cloudhelpme.ru/core/js/dist/main.js?v=54e31561-37:1

@paulijar
Copy link
Collaborator

paulijar commented Nov 5, 2020

Thanks for the report. Just to clarify, the problem case is when deleting a file within the Files app.

The problem is that the lodash.js library bundled with the Music app is leaked to the global namespace even though that is not supposed to happen. In the global namespace, it shadows the underscore.js library which is used by the Files app. As lodash.js is only partially API-compatible with underscore.js, the Files app can't function correctly with the unexpected library.

Now why this happens, is a bit of a mystery. But it seems that the problem happens when I build the webpack bundle using Cygwin (as I did with the release package). And the problem doesn't happen if I build the bundle using the Windows Power Shell. This kind of difference is quite surprising as the bundle built on Cygwin seems to otherwise work just fine.

@i-iooi-i
Copy link

i-iooi-i commented Nov 6, 2020

I also encountered the same problem, new folder and delete file can not be used, need to refresh to get results. It's a great program, but it's not going to be used for now.

@paulijar
Copy link
Collaborator

paulijar commented Nov 7, 2020

I was busy yesterday but I should be able to make a fix release today or latest tomorrow.

@paulijar paulijar changed the title After updating, delete button not working Music v0.17.2 interferes delete on the Files app Nov 7, 2020
@paulijar
Copy link
Collaborator

paulijar commented Nov 7, 2020

The Music v0.17.3 fixing this issue has now been published.

@paulijar paulijar closed this as completed Nov 7, 2020
@i-iooi-i
Copy link

i-iooi-i commented Nov 8, 2020

Fortunately, I can use it again! Great work!

@dusnm
Copy link

dusnm commented Nov 9, 2020

Thanks! I was wondering why it wasn't working, but since I use the Music app heavily I decided to just stick with the problem until it was fixed. Awesome work guys!

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

No branches or pull requests

4 participants