D10315: [Notifications] Fix grouping

Mon Feb 5 09:12:59 UTC 2018

fvogt added inline comments.

INLINE COMMENTS

> fvogt wrote in notificationsengine.cpp:230
> This results in `...Text<br/><?xml version="1.0"?><html>Some text...` AFAICT.
> 
> This also means that the if condition is always false.

According to @broulik the first part is expected, which IMO needs a comment as it does very much look like a bug.

The latter part is false though.

REPOSITORY
  R120 Plasma Workspace

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

To: broulik, #plasma, davidedmundson, aacid, fvogt
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180205/17e1c59f/attachment.html>