Drag & drop fails: gets empty file (invalid)
What Happened?
dragging and dropping some songs from Music leds to failure for me
Ive rolled back to from before my PRs, and the issue persists (so not introduced by me at least)
apparently Application.loop_through_files receives a (null)
investigating as this pisses me off.
Steps to Reproduce
open Music drop music files where music
Expected Behavior
Opens dropped file
OS Version
8.x (Circe)
Session Type
Secure Session (Wayland)
Software Version
Latest release (I have run all updates)
Log Output
(io.elementary.music:2): GLib-CRITICAL **: 12:43:46.089: g_file_test: assertion 'filename != NULL' failed
(io.elementary.music:2): GLib-CRITICAL **: 12:43:47.872: g_file_test: assertion 'filename != NULL' failed
(io.elementary.music:2): GLib-CRITICAL **: 12:43:51.495: g_file_test: assertion 'filename != NULL' failed
(io.elementary.music:2): GLib-CRITICAL **: 12:43:53.846: g_file_test: assertion 'filename != NULL' failed
Hardware Info
No response
Yes, apparently we receive an empty Gdk.Filelist ? But it is a Filelist ?
Bug present in 8.0.0 release
EDIT: So apparently Amberol also gets a null. So this isnt strictly Music but maybe a portal missing or a wayland thing giving full access to Home doesnt cut it either
It may be a bug with Files as well. I know @jeremypw has been a bunch of fixes for various D&D bugs there so maybe he has some insight if this may be related to anything he's worked/working on
@teamcons Could you confirm from where you are dropping the files? Is it io.elementary.files? There have been some regressions in copy/paste and drag/drop recently which need https://github.com/elementary/files/pull/2616 to be merged and released.
I confirm it is from io.elementary.files this might be more of a Files issue than a Music one then...
Indeed, you could confirm by dragging from a different app.
@zeebok Are you available to look at https://github.com/elementary/files/pull/2616 some time? After getting regression when this code was last touched I am concerned to get as thorough testing as possible before release.
@jeremypw Yeah I will take a look at it late tonight!
Huh it works again