anthonyfieroni added inline comments.

INLINE COMMENTS

> recentusagemodel.cpp:286
>          if (!resource.startsWith(QLatin1String("applications:"))) {
> -            new KRun(docData(resource, Kicker::UrlRole).toUrl(), 0);
> +            QUrl resourceUrl = docData(resource, Kicker::UrlRole).toUrl();
> +

const

> recentusagemodel.cpp:288
> +
> +            QList<QUrl> urlsList;
> +            urlsList.append(resourceUrl);

const QList<QUrl> urlsList{ resourceUrl };

> recentusagemodel.cpp:294
> +            KService::Ptr service = 
> KMimeTypeTrader::self()->preferredService(mime.name());
> +            if (service != nullptr) {
> +                KRun::runApplication(*service, urlsList, nullptr);

if (service) {

REPOSITORY
  R119 Plasma Desktop

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

To: aleksejshilin, #plasma
Cc: anthonyfieroni, broulik, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, 
jensreuterberg, abetts, sebas, apol, mart

Reply via email to