music icon indicating copy to clipboard operation
music copied to clipboard

Drag & drop fails: gets empty file (invalid)

Open teamcons opened this issue 6 months ago • 1 comments

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 avatar Jun 14 '25 10:06 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

teamcons avatar Jun 14 '25 11:06 teamcons

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

zeebok avatar Aug 01 '25 06:08 zeebok

@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.

jeremypw avatar Aug 01 '25 13:08 jeremypw

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

teamcons avatar Aug 01 '25 14:08 teamcons

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

jeremypw avatar Aug 01 '25 14:08 jeremypw

@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 avatar Aug 01 '25 14:08 jeremypw

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

zeebok avatar Aug 01 '25 16:08 zeebok

Huh it works again

teamcons avatar Aug 11 '25 08:08 teamcons