D7596: Register objectPath before service

[David Edmundson]

davidedmundson created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  When the service is regsitered we want to have all the internal routing
  pre-configured. Otherwise pending calls (such as pending DBus activated
  calls) can be processed before the path is registered and then fail with
  an error that the path does not exist.
  
  Also remove the fairly redundant check on registerObject. That can return
  false if some object in this process has registered itself on that path; but in that
  case, the very last thing we'd want to do is call unregisterService.

TEST PLAN
  Generated a load of notifications
  Started plasma 
  Ouptut showed valid return on GetCapabilities, not an error on missing path

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

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

AFFECTED FILES
  dataengines/notifications/notificationsengine.cpp

To: davidedmundson, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170829/85efe7a4/attachment.html>