D29082: WIP Dolphin Service: Show error message when uninstalling packages

[Kai Uwe Broulik]

broulik added inline comments.

INLINE COMMENTS

> servicemenuinstaller.cpp:276
> +                                          "The filename is %1", archive);
> +            QProcess::startDetached("kdialog", {"--error", errorMsg});
> +            QFile::remove(archive);

Why not show a `KMessageBox`?
Alternatively, offer the user to do that?
"This thing needs to be installed [Open in whatever app for the mime type, e.g. Discover] [Cancel]"

REPOSITORY
  R318 Dolphin

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

To: alex, #dolphin, ngraham, nicolasfella, elvisangelaccio
Cc: broulik, kfm-devel, azyx, nikolaik, pberestov, iasensio, aprcela, fprice, fbampaloukas, alexde, Codezela, feverfew, meven, spoorun, navarromorales, firef, ngraham, andrebarros, emmanuelp, rdieter, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200422/67d6baa7/attachment.htm>