diff --git a/src/dom_components/model/ComponentVideo.js b/src/dom_components/model/ComponentVideo.js index f64e303f5..a21683e6b 100644 --- a/src/dom_components/model/ComponentVideo.js +++ b/src/dom_components/model/ComponentVideo.js @@ -6,6 +6,8 @@ const yt = 'yt'; const vi = 'vi'; const ytnc = 'ytnc'; +const hasParam = value => value && value !== '0'; + export default Component.extend( { defaults: { @@ -71,22 +73,21 @@ export default Component.extend( * Set attributes by src string */ parseFromSrc() { - var prov = this.get('provider'); - var uri = this.parseUri(this.get('src')); - var qr = uri.query; + const prov = this.get('provider'); + const uri = this.parseUri(this.get('src')); + const qr = uri.query; switch (prov) { case yt: case ytnc: case vi: - var videoId = uri.pathname.split('/').pop(); - this.set('videoId', videoId); + this.set('videoId', uri.pathname.split('/').pop()); qr.list && this.set('list', qr.list); - if (qr.autoplay) this.set('autoplay', 1); - if (qr.loop) this.set('loop', 1); - if (parseInt(qr.controls) === 0) this.set('controls', 0); - if (qr.color) this.set('color', qr.color); - if (qr.rel === '0') this.set('rel', 0); - if (qr.modestbranding === '1') this.set('modestbranding', 1); + hasParam(qr.autoplay) && this.set('autoplay', 1); + hasParam(qr.loop) && this.set('loop', 1); + parseInt(qr.controls) === 0 && this.set('controls', 0); + hasParam(qr.color) && this.set('color', qr.color); + qr.rel === '0' && this.set('rel', 0); + qr.modestbranding === '1' && this.set('modestbranding', 1); break; default: }