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

cannot delete file in network mounted folder, saying permission problem. #97

Closed
chchia opened this issue Jan 26, 2023 · 7 comments
Closed
Labels
bug Something isn't working Windows
Milestone

Comments

@chchia
Copy link

chchia commented Jan 26, 2023

specific a folder for trash work... but can't delete directly.

@theophanemayaud
Copy link
Owner

Hi ! Thanks for the report ! On what operating system are you ? Also, when you manually go to the file and delete it, do you have the permissions ?

@chchia
Copy link
Author

chchia commented Jan 27, 2023

Hi I am using Windows 11 with purchased the software from Microsoft Store.
image

For example if i did not define the trash folder this error will show when i click Trash, left or right result the same
image

but if i do select folder instead of trash, i can then Trash the file but it result in longer time as the folder was a remote network share.
image

i hope you can understand what i say, i think you can simply replicate it by add a folder that remote mounted as network storage, and you should know the problem.

thanks.

@theophanemayaud
Copy link
Owner

Ok thanks for all the details ! I’ll try to check this out this weekend and see what might be the cause.

@chchia

This comment was marked as off-topic.

@theophanemayaud
Copy link
Owner

Ok so I just checked, and I have exactly the same problem : trash when in network drive doesn't work, except when setting a custom location to move the files to instead of the trash. I'm on windows 10.
Investigation :
As per https://superuser.com/questions/160990/put-files-deleted-over-the-network-into-a-windows-trash-can, there is no trash on network drives. Thus, because the app uses a move to trash function, it will never work on systems with no trash, as per https://doc.qt.io/qt-6/qfile.html#moveToTrash it is the expected behavior.

Idea :

  • check if drive has trash, else warn user
  • propose option to delete directly

Will keep investigating

@theophanemayaud
Copy link
Owner

It doesn't seem straightforward to check if the drive has a trash. Therefore, I simply added in the error message that it could be a reason for failing to move to trash.
I also added the overall setting, instead of moving to trash or moving to a specified folder, to delete outright. If you have lots of files on a network drive, you can enable this as soon as the update is release by microsoft :)
Closed by 22f366c

@theophanemayaud
Copy link
Owner

theophanemayaud commented Mar 19, 2023

and please allow to save setting, so that user do not need to add folder and set other setting again every time the program is restarted.

By the way I'm also tracking this in #101

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Windows
Development

No branches or pull requests

2 participants