D11028: [media controller applet] Fix JS error on no xesam:url data set

Kai Uwe Broulik noreply at phabricator.kde.org
Mon Mar 5 08:25:56 UTC 2018


broulik accepted this revision.
broulik added a comment.
This revision is now accepted and ready to land.


  My approach would have been
  
    var xesamUrl = (metadata["xesam:url"] || "").toString();
  
  but this is fine and more understandable for non-JS developers :)

REPOSITORY
  R120 Plasma Workspace

BRANCH
  fixtoStringfromnull

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

To: kossebau, broulik
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/20180305/2a090123/attachment-0001.html>


More information about the Plasma-devel mailing list