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