Uncaught SyntaxError: missing ) after argument list throwing error in html line

Question

can anyone tell me what's wrong in this line?

Template:

<c:forEach items="${templateList}" var ="temp">
   <td>
      <button onclick = "edit(${temp.id});" data-target="#myModal1" data-toggle="modal" title="Edit" class="btn btn-success btn-sm"><i class="fa fa-edit"></i>Edit</button>
   </td>
   <td>
      <button data-target="#myModal2" title="Attach Link" data-toggle="modal" class="btn btn-success btn-sm"><i class="fa fa-paperclip"></i></button>
   </td>
   <td>
      <button data-target="" data-toggle="modal" title="duplicate" class="btn btn-success btn-sm"><i class="fa fa-copy"></i></button>
   </td>
</c:forEach>

and as well as this function

<script>
   function edit(id){
      $.get("get/" + id,function(result){
         $("#myModal1").html(result);
         alert("dnrbufevc ");
      });
   }
</script>

Why cant you use ng-click and use your function inside your controller? — Senal
– Senal, Commented Jun 6, 2018 at 1:24
@Senal actually taglib I am using so I am just selecting single value from the list of value that's why I want mandatorialy go with $ selector — ram
– ram, Commented Jun 6, 2018 at 1:29
Does ${temp.id} render as an actual string when you inspect the button? Otherwise you're just sending a string to the JS not the ID you're intending — Doug
– Doug, Commented Jun 6, 2018 at 1:30
@Doug <c:forEach items="${templateList}" var ="temp"> <td><button onclick = "edit(${temp.id});" data-target="#myModal1" data-toggle="modal" title="Edit" class="btn btn-success btn-sm"><i class="fa fa-edit"></i>Edit</button></td> <td><button data-target="#myModal2" title="Attach Link" data-toggle="modal" class="btn btn-success btn-sm"><i class="fa fa-paperclip"></i></button></td> <td><button data-target="" data-toggle="modal" title="duplicate" class="btn btn-success btn-sm"><i class="fa fa-copy"></i></button></td> </c:forEach> here is my whole code — ram
– ram, Commented Jun 6, 2018 at 2:11

Senal · Accepted Answer · 2018-06-06 02:50:45Z

1

Change your template for onClick to the following

<button onclick = "edit('${temp.id}')" data-target="#myModal1" data-toggle="modal" title="Edit" class="btn btn-success btn-sm"><i class="fa fa-edit"></i>Edit</button>

Make sure to have single quotes around '${temp.id}' so that javascript will treat it as a string literal.

answered Jun 6, 2018 at 2:50

Senal

1,6202 gold badges16 silver badges23 bronze badges

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

2 Comments

ram Over a year ago

I don't want that '${temp.id}' as string literal that is actal id int type

Senal Over a year ago

What do you get inside your onclick method as id after doing that change?

Collectives™ on Stack Overflow

Uncaught SyntaxError: missing ) after argument list throwing error in html line

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related