mixxx icon indicating copy to clipboard operation
mixxx copied to clipboard

Delete Files: move files to trash instead of deleting instantly

Open mixxxbot opened this issue 2 years ago • 10 comments

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 avatar Aug 23 '22 02:08 mixxxbot

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 avatar Aug 23 '22 02:08 mixxxbot

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 avatar Aug 23 '22 02:08 mixxxbot

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 avatar Aug 23 '22 02:08 mixxxbot

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 avatar Aug 23 '22 02:08 mixxxbot

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 avatar Aug 23 '22 02:08 mixxxbot

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 avatar Aug 23 '22 02:08 mixxxbot

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 avatar Aug 23 '22 02:08 mixxxbot

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 avatar Aug 23 '22 02:08 mixxxbot

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 avatar Aug 23 '22 02:08 mixxxbot

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


yes, that was a separate bug resolved by https://github.com/mixxxdj/mixxx/pull/4831

mixxxbot avatar Aug 23 '22 02:08 mixxxbot

Fixed for builds using Qt => 5.15 by #11842

ronso0 avatar Aug 28 '23 13:08 ronso0