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

[Friedrich W. H. Kossebau]

kossebau added a comment.


  In D11028#218774 <https://phabricator.kde.org/D11028#218774>, @broulik wrote:
  
  > My approach would have been
  >
  >   var xesamUrl = (metadata["xesam:url"] || "").toString();
  >
  >
  > but this is fine and more understandable for non-JS developers :)
  
  
  It took me years to no longer be a JS developer ;) But never seen this pattern before, think this is more nice, so will go for that :)

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/0d19d7ab/attachment.html>