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

--- Comment #5 from Nicolas Fella <nicolas.fe...@gmx.de> ---
(In reply to Alfred Wingate from comment #3)
> Created attachment 176601 [details]
> plasma/xdg-desktop-portal-kde: Don't ignore current_filter parameter with
> multiple filter strings
> 
> The current_filter one is a different issue (Scenario 5 vs 6), as filters
> with more than one string* are explicitly ignored. 
> 
> With this patch it works as expected on my end. (Scenario 6 gets me two
> working filters and it picks "Windows Executable/MSI Installer" as the
> default one even if I change order of the filters)
> 
> I'm not familiar with portal internals so I don't how much input validation
> is needed here, but I don't think its right reject this.
> 
> * I don't know what the correct terminology is. pattern? filter rule?

You're right, this seems wrong and is a leftover from previous refactoring. Can
you make a merge request with your change?

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

Reply via email to