D11567: Fix use of assert/crash when removing global shortcut

[David Edmundson]

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

REVISION SUMMARY
  I'm not the author, but it's pretty clear what the intention was.
  
  We expect to get one result back from proxyModel->match
  We use results.first(), so we want to check we have at least one entry.
  The assert was written backwards
  
  BUG: 387444

TEST PLAN
  Can now remove a group without a crash

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

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

AFFECTED FILES
  kcms/keys/kglobalshortcutseditor.cpp

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