I have this error: Uncaught SyntaxError: missing ) after argument list

Question

I keep getting this error, but I can not figure it out what the problem seems to be. Apparently the error is in the line of currentProduct.find()

var $productTemplate = $(".product-item");
$.ajax({
    url: 'data/applicationLayer.php',
    type: 'POST',
    dataType: 'json',
    contentType: "application/x-www-form-urlencoded",
    data: {
        action: "MOSTRARIMAGENES",
    },
})
.done(function(jsonResponse) {
    $.each(jsonResponse, function(index, val) {
        if (jsonResponse.length) {
            $("#initialProductMessage").remove();
            var $currentProduct = $productTemplate.clone(true);
            $currentProduct.find('.producto1').html(val.nombre + "<br>" + "<img src="val.foto" width="200" height="400">");
            $currentProduct.show();
            $("#products-display").prepend($currentProduct);
        }
    });
})

You're missing some + to concatenate your string with the val.foto — Rory McCrossan
– Rory McCrossan, Commented Nov 14, 2016 at 14:33
Escape the quotes in the <img> html snippet you concatenate with val.nombre and "<br>" — Quagaar
– Quagaar, Commented Nov 14, 2016 at 14:35

Rory McCrossan · Accepted Answer · 2016-11-14 14:48:43Z

1

You're missing some + to concatenate your string with the val.foto. You also should escape some quotes to wrap the values. Try this:

$currentProduct.find('.producto1').html(val.nombre + "<br><img src=\"" + val.foto + "\" width=\"200\" height=\"400\">");

Or using single quotes to avoid escaping entirely:

$currentProduct.find('.producto1').html(val.nombre + '<br><img src="' + val.foto + "' width="200" height="400">');

edited Nov 14, 2016 at 14:48

answered Nov 14, 2016 at 14:34

Rory McCrossan

338k41 gold badges321 silver badges353 bronze badges

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

4 Comments

Neo Over a year ago

Beat me by just a bit!!

Zeke Over a year ago

Yup, but there's more escaping to do: the width and height attributes. Or simply use single quotes for these strings to avoid escaping.

Zeke Over a year ago

@RoryMcCrossan you're welcome - We're all here to help!

Zeke Over a year ago

@RoryMcCrossan actually, there's still one double quote unedited at the very end! I can't edit it because edits must have at least 6 characters :/

Collectives™ on Stack Overflow

I have this error: Uncaught SyntaxError: missing ) after argument list

1 Answer 1

4 Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Related