| Summary: | Open/save dialog shows no files when app specifies its list of mimetypes with a glob | ||
|---|---|---|---|
| Product: | [Plasma] xdg-desktop-portal-kde | Reporter: | Patrick Silva <bugseforuns> |
| Component: | general | Assignee: | Plasma Bugs List <plasma-bugs-null> |
| Status: | CONFIRMED --- | ||
| Severity: | normal | CC: | aleixpol, bizyaev, eweapon713, jgrulich, justin, nate, nicolas.fella, pollux66, postix |
| Priority: | NOR | ||
| Version First Reported In: | 5.24.1 | ||
| Target Milestone: | --- | ||
| Platform: | openSUSE | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
|
Description
Patrick Silva
2022-02-20 14:32:19 UTC
The problem is that Celluloid passes mimetype filters with globs ("audio/*", "video/*", "images/*"), and KFileWidget doesn't handle those
In other words
KFileWidget w(QUrl("file:///home/nico/Music"));
w.setMimeFilter({"audio/*"});
doesn't work, but
KFileWidget w(QUrl("file:///home/nico/Music"));
w.setMimeFilter({"audio/mpeg"});
does
There's more: Celluloid uses both filename glob based filters and mimetype filters, and KFileWidget can't handle both at the same time A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kio/-/merge_requests/766 (In reply to Nicolas Fella from comment #3) > There's more: > > Celluloid uses both filename glob based filters and mimetype filters, and > KFileWidget can't handle both at the same time https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/merge_requests/138 addresses that Git commit e6ed8972d7fe64ce12bc8fa52993317f80d38160 by Nicolas Fella. Committed on 07/11/2023 at 17:43. Pushed by nicolasfella into branch 'master'. [filedialog] Port to KFileFilter-based API This matches what we get from the front-end more closely, thus simplifying the code and making it more correct M +63 -93 src/filechooser.cpp M +10 -5 src/filechooser.h https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/commit/e6ed8972d7fe64ce12bc8fa52993317f80d38160 (In reply to Nicolas Fella from comment #5) > (In reply to Nicolas Fella from comment #3) > > There's more: > > > > Celluloid uses both filename glob based filters and mimetype filters, and > > KFileWidget can't handle both at the same time > > https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/merge_requests/138 > addresses that This was merged, so is this now resolved? No, there is still no support for globs, which is why I did not mark it as resolved *** Bug 470653 has been marked as a duplicate of this bug. *** *** Bug 485866 has been marked as a duplicate of this bug. *** I can't reproduce this issue with the latest version of Celluloid anymore, FWIW. Perhaps they ported away from the use of globs. Try with https://flathub.org/apps/io.gitlab.adhami3310.Footage See https://bugs.kde.org/show_bug.cgi?id=485866 Thanks. Seems ti be fixed with https://invent.kde.org/plasma/xdg-desktop-portal-kde/-/merge_requests/324, https://invent.kde.org/frameworks/kio/-/merge_requests/1751, and https://invent.kde.org/frameworks/kio/-/merge_requests/1750 No, this is a different issue entirely This is about using mimetypes like "video/*", it has nothing to do with the "All files" type that Nate is seeing in Dragon. The portal API isn't really clear whether this is something expected to be supported, so I opened https://github.com/flatpak/xdg-desktop-portal/discussions/1487 for clarification Nate can you confirm the patches actually did the right thing? Are they able to be cleanly applied on top of 24.08.2 or .3? The patches in question fix an issue (the one blocking getting Dragon on Flathub), but apparently not this one, which was something else all along. Hmm OK. I'll try to find some time this week to attempt to apply them on top of the stable release tarball. *** Bug 509293 has been marked as a duplicate of this bug. *** https://flathub.org/en/apps/page.kramo.Sly is another affected app |