Drag & drop fails: gets empty file (invalid)

[teamcons]

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

[teamcons]

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

[zeebok]

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

[jeremypw]

@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 elementary/files#2616 to be merged and released.

[teamcons]

I confirm it is from io.elementary.files
this might be more of a Files issue than a Music one then...

[jeremypw]

Indeed, you could confirm by dragging from a different app.

[jeremypw]

@zeebok Are you available to look at elementary/files#2616 some time? After getting regression when this code was last touched I am concerned to get as thorough testing as possible before release.

[zeebok]

@jeremypw Yeah I will take a look at it late tonight!

[teamcons]

Huh it works again