maxrd2 added a comment.

  In https://phabricator.kde.org/D7676#143033, @broulik wrote:
  
  > Well it is now "openService" calling "runApplication", that's inconsistent 
isn't it
  
  
  Yes, but it will start services that have Exec line though.
  Should i just rename the method to "openApplication" or "runApplication" ?
  
  Adding something like this wont change much it seems.
  
    if(!service->isApplication())
        KRun::runService(*service, QList<QUrl>(), nullptr) != 0;

REPOSITORY
  R296 KDeclarative

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

To: maxrd2, davidedmundson
Cc: broulik, #frameworks

Reply via email to