D17677: Add workaround for gtk file chooser filter patterns

[David Edmundson]

davidedmundson added inline comments.

INLINE COMMENTS

> jgrulich wrote in filechooser.cpp:287
> Does it really support regexp in this form? If so, why it doesn't work when used by plasma-integration? I see plasma-integration internally uses KFileWidget and passes filters to it.

It should do:

It goes through:

  QRegExp rx(p);
        rx.setPatternSyntax(QRegExp::Wildcard);

Wildcard in the Qt docs sounds very much like the "glob" format that's in the XDG spec.

Running ./kfilewidgettest_gui  in kio/bin  you can type a filter manually

I did:
*.[Pp][Dd][Ff]  and that worked for me.

It doesn't work in plasma-integration because it goes through a method qt2KdeFilter - to turn Qt's weird custom syntax into reg ex patterns.

REPOSITORY
  R838 Flatpak Support: KDE Portal for XDG Desktop

REVISION DETAIL
  https://phabricator.kde.org/D17677

To: jgrulich, broulik
Cc: heikobecker, ngraham, davidedmundson, plasma-devel, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181220/a1744807/attachment-0001.html>