https://bugs.kde.org/show_bug.cgi?id=521748

David Redondo <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/xdg-desktop-portal-kde/- |ma/xdg-desktop-portal-kde/-
                   |/commit/1f244dc3753ea423a25 |/commit/b9d06180bd65966fc3f
                   |28f542ba1e1b160f02a6d       |5a119910a64aaf73a702a

--- Comment #5 from David Redondo <[email protected]> ---
Git commit b9d06180bd65966fc3f5a119910a64aaf73a702a by David Redondo.
Committed on 06/07/2026 at 08:16.
Pushed by davidre into branch 'Plasma/6.6'.

appchooser: Unconfuse url vs filename

The code was a bit confused whether to pass a file name, url or
some display string to the dialog.
The portal path always passed the file name for local files, for
other URIs would fallback to the mime type instead of the URI.
The private plasma path would pass a URL as string which the dialog
would wrap into another url, which somehow breaks the decoding when
retrieving the filename. When multiple files are passed it used a
free form string, the code in the dialog obviously doesnt make sense
then.
Disentangle this by making it clear you are passing a display string,
convert the url string to an url directly. The portal always passes
content-type, move resolving of the mimetype to the private entry
point where we can deal with the url.
FIXED-IN:6.6.6


(cherry picked from commit db2cd7a90ce279cb126f2509e708a31863ddc265)

Co-authored-by: David Redondo <[email protected]>

(cherry picked from commit 1f244dc3753ea423a2528f542ba1e1b160f02a6d)

Co-authored-by: David Redondo <[email protected]>

M  +19   -8    src/appchooser.cpp
M  +12   -19   src/appchooserdialog.cpp
M  +1    -1    src/appchooserdialog.h

https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/commit/b9d06180bd65966fc3f5a119910a64aaf73a702a

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to