Parse error. missing name after . operator

Question

I have the following code where I am using VAST player to play my ads. I am using a check to see if the user has clicked already to avoid the DOM exception that is present in modern browsers.

(function(VASTPlayer) {
    'use strict';

    var player = new VASTPlayer(document.getElementById(playerID));

    player.once('AdStopped', function() {
        console.log('Ad finished playback! ' + playerID);
        interstitialInstance.close();
    });

    player.load( << Ad Tag >> ).then(function startAd() {
        console.log(player.adDuration + " " + playerID);

        var s = document.getElementById(playerID).childNodes[0];
        if (s) {
            if (!ryads.mouseClick) {
                s.muted = true;
            } else
                s.muted = false;
        } else {
            console.log("Error while fetching video element!!!");
        }

        return player.startAd();
    }).catch(function(reason) {
        console.log('Ad failed to play ' + playerID);
        interstitialInstance.close();
        setTimeout(function() {
            throw reason;
        }, 0);
    });
}(window.VASTPlayer));

I am getting the following error when I run the Compress job in Jenkins.

project.js:891: ERROR - Parse error. missing name after . operator }).catch(function(reason) { ^

project.js:892: ERROR - Parse error. syntax error console.log('Ad failed to play '+playerID); ^

project.js:896: ERROR - Parse error. missing ; before statement }(window.VASTPlayer));

You are missing a closing parenthesis. Change it to })(window.VASTPlayer)); — Mani
– Mani, Commented May 14, 2019 at 6:34
Is it all? Looks like some part of the code at the beginning is omitted. Also what is <<Ad Tag>>? It's not a valid JavaScript — Evgeniy Malyutin
– Evgeniy Malyutin, Commented May 14, 2019 at 6:37

noChance · Accepted Answer · 2019-05-14 08:22:28Z

3

This is a well known issue since years for yuicompressor. An easy fix for is to extract the resolve and reject functions of the promise like this:

promise.then(successFunction, failureFunction);

function successFunction() {
   console.log('success');
}

function failureFunction(err) {
    console.error(err);
}

edited May 14, 2019 at 8:22

answered May 14, 2019 at 7:18

noChance

5232 silver badges14 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

connexo · Accepted Answer · 2019-05-14 06:43:42Z

1

Replace this line

(function(VASTPlayer) {

by

;(function(VASTPlayer) {

Otherwise, when the compression job is trying to concatenate files, your IIFE might be considered an argument to the code that the end of the file contained which was concatenated right before this file.

Can you please elaborate on what

player.load( << Ad Tag >>

is? That is not valid Javascript. Some sort of JSX dialect? Or just a copy/paste error?

edited May 14, 2019 at 6:43

answered May 14, 2019 at 6:34

connexo

57.3k15 gold badges112 silver badges150 bronze badges

7 Comments

Barmar Over a year ago

I don't think there's a missing ). The second ) in )) matches the ( on the first line.

connexo Over a year ago

@Barmar I just saw it was not even your comment, lol. Thanks for pointing out!

yogesh sriraman Over a year ago

player.load actually loads the Ad for playing. The tag is the value provided for the Ad.

connexo Over a year ago

@yogeshsriraman Again, that is not valid Javascript. What language is that? JSX? I'm referring to the << Ad Tag >> part.

connexo Over a year ago

nodeJS is Javascript, and also there << Ad Tag >> is not valid Javascript.

|

Collectives™ on Stack Overflow

Parse error. missing name after . operator

2 Answers 2

Comments

7 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

7 Comments

Your Answer

Sign up or log in

Post as a guest

Related