D21434: [Notifications KCM] Try both model indices

[Kai Uwe Broulik]

broulik created this revision.
broulik added reviewers: Plasma, nicolasfella.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  We prefer applicatons over services, however, when configuring a KDE application we get passed both an application and a service. That application might be `NoDisplay` and then we end up not finding an index and event configuration files.

TEST PLAN
  5.16 branch
  
  I can now configure KDE Connect notifications by clicking the configure icon on the notification.

REPOSITORY
  R119 Plasma Desktop

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

AFFECTED FILES
  kcms/notifications/package/contents/ui/SourcesPage.qml

To: broulik, #plasma, nicolasfella
Cc: plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190527/39310ebd/attachment.html>