D5612: [Notification Jobs] Enforce plain text and escape HTML

[David Edmundson]

davidedmundson requested changes to this revision.
davidedmundson added a comment.
This revision now requires changes to proceed.


  > Since notifications do interpret HTML tags by design, we manually escape them there.
  
  I don't know hwat you mean.
  
  tags are important, we have notifications with links and emoticons.
  
  > Copied a file <h1>foo</h1> and no longer got huge text in summary, details, and finish popup. Verified that speed and ETA label still works.
  
  That needs fixing before sending a notification. Not in the UI.

REPOSITORY
  R120 Plasma Workspace

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

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