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

Delete Files: move files to trash instead of deleting instantly #10760

Closed
mixxxbot opened this issue Aug 23, 2022 · 11 comments
Closed

Delete Files: move files to trash instead of deleting instantly #10760

mixxxbot opened this issue Aug 23, 2022 · 11 comments

Comments

@mixxxbot
Copy link
Collaborator

Reported by: stefan-franz
Date: 2022-06-27T17:09:54Z
Status: Confirmed
Importance: Low
Launchpad Issue: lp1980001
Attachments: [Bildschirmfoto vom 2022-06-28 06-22-40.jpg](https://bugs.launchpad.net/bugs/1980001/+attachment/5600085/+files/Bildschirmfoto vom 2022-06-28 06-22-40.jpg)


Linux Mint 20.3 Cinnamon

After rightclick in the browser list / delete track files from list

  1. The track is deleted without trash (is it possible to send it to the trash?)
  2. A small window appears that 1 song is removed from the database - but the windows does not close itsself.
@mixxxbot
Copy link
Collaborator Author

Commented by: stefan-franz
Date: 2022-06-27T17:15:02Z


hmmm....now, after deleting 3 files more the windows close istsself....
I report if the error comes again.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-06-27T21:02:53Z


The "N track files were removed" dialog is not supposed to close itself.
Are you saying the dialog pops up, and while you do nothing (no button on keyboard or controller pressed, no mouse click), it is closed magically?

Note that controller with a Browse or Trax push/rotary encoder have [Library],GoToItem mapped. Per context this will trigger different actions, see https://manual.mixxx.org/2.4/en/chapters/appendix/mixxx_controls.html#control-[Library]-GoToItem, e.g. in dialogs it can press buttons.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-06-27T21:06:54Z


  1. The track is deleted without trash (is it possible to send it to the trash?)

Yes, this is noted as TODO.
It's just someone needs to implement it :)

@mixxxbot
Copy link
Collaborator Author

mixxxbot commented Aug 23, 2022

Commented by: ronso0
Date: 2022-06-27T21:41:52Z


QFile::moveToTrash is only supported in Qt 5.15 and newer.

IIRC (someone correct me if I'm wrong) we wait for the first point release of the current Ubuntu LTS (22.04.1) in order to drop support for Ubuntu 20.04 which is the only operating system that requires supporting Qt 5.12, and use Qt 5.15+
https://wiki.ubuntu.com/Releases#Future

We could as well add it now with Qt version switches, but then we wouldn't have a consistent implementation on all supported OS :|

@mixxxbot
Copy link
Collaborator Author

Commented by: stefan-franz
Date: 2022-06-28T04:34:02Z
Attachments: [Bildschirmfoto vom 2022-06-28 06-22-40.jpg](https://bugs.launchpad.net/mixxx/+bug/1980001/+attachment/5600085/+files/Bildschirmfoto vom 2022-06-28 06-22-40.jpg)


The small window in the front comes not always if i delete a track. If it comes, it stands as here on the screenshot and the progress bar does not work.
The cancel Button works and the file is deletet from the database. I think only the gui does sometimes not wirk right - the database correction works.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-06-28T09:19:59Z


I thought you were referring to the Success dialog

To clarify, the workflow has multiple dialogs:

  1. if you open the menu from a deck and click Delete while the track is
    still being played, Mixxx will ask you to stop the deck first
  2. confirmation "Permanently delete these files from disk?"
  3. progress dialog, only shown if required. quoted from source code:
    /// This dialog
    /// only appears if processing takes longer than the given grace
    /// period. This avoids that an open context menu gets closed
    /// while processing only a few tracks.
  4. Success: "Track File(s) Deleted"
  5. Failure report if file(s) could not be deleted

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-06-28T09:22:43Z


I hope you didn't discover any other issues with the workflow.

This bug now only covers moveToTrash. Please file a separate report for dialog issues so we can track those independently.
Thank you for testing btw!

@mixxxbot
Copy link
Collaborator Author

Commented by: daschuer
Date: 2022-06-28T13:50:33Z


The limiting OS is macOs
It required Qt 5.12 because of the waveforms.
This will only be solved if one writes new waveforms, so we are not talking about a few month to wait.

It feels bad to hold this useful feature back for all users just for sake of consistent behavior.
I would welcome a solution with a version guard.

@mixxxbot
Copy link
Collaborator Author

Commented by: stefan-franz
Date: 2022-07-09T10:52:08Z


Linux Mint 20.3 Cinnamon - Mixxx 2.4 alpha2 - 1225
Deleting Files works now without an hanging window as - point 2 of my original report.

@mixxxbot
Copy link
Collaborator Author

Commented by: ronso0
Date: 2022-07-10T17:49:39Z


yes, that was a separate bug resolved by #4831

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@mixxxbot mixxxbot added this to the 2.4.0 milestone Aug 24, 2022
@ronso0 ronso0 added feature and removed bug labels Aug 23, 2023
@ronso0
Copy link
Member

ronso0 commented Aug 28, 2023

Fixed for builds using Qt => 5.15 by #11842

@ronso0 ronso0 closed this as completed Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants