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

Alexander Lohnau <alexander.loh...@gmx.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/plasma-workspace/commit/ |ma/plasma-workspace/commit/
                   |165b3b0e3b354c0f8ca80b10fcc |01a74949fa1d5b0624f8a57bd07
                   |9f0d40682e98b               |8a55e678ea213
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #11 from Alexander Lohnau <alexander.loh...@gmx.de> ---
Git commit 01a74949fa1d5b0624f8a57bd078a55e678ea213 by Alexander Lohnau.
Committed on 21/10/2021 at 07:12.
Pushed by alex into branch 'master'.

Fix context menu actions for KCMs

Needs
https://invent.kde.org/plasma/systemsettings/commit/c815c7a9bc658b7d4778c3fc4ebfff9756ebb199

Currently the logic is very fragile and depends on the data being an url.
However the data of the match is an implementation detail and is in case of the
KCM matches
a KPluginMetaData instance.

To avoid this issue the first url of the QueryMatch::urls() return value is
checked.
This data is intended for mime-type/scheme related operations.

In the future we need an extra url for systemsettings & should consistently use
the
urls of the query match. I will work on this before continuing to port the
plasma KCMs from KServiceTypeTrader.

M  +10   -2    applets/kicker/plugin/runnermatchesmodel.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/01a74949fa1d5b0624f8a57bd078a55e678ea213

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

Reply via email to