Passing a string in onclick function's in for loop

Question

I need to pass string as ID in onclick function. When I pass integer as id, it's working fine but when I pass string, it's not even going to the function.

HTML Code ( below is table row. This is inside a for loop)

<td class="col-md-2" width="50" style="background-color: transparent;">
<input class="btn btn-sm btn-primary fa" id='[email protected]' type="button" value="&#xf040" onclick='showedit(@item.ID);' />
</td>

Javascript Code:

function showedit(par) {
        if (editItem == par) {
            return;
        }
        if (editItem > 0) {
            cancel(editItem);
        }

       --code processing--
        editItem = par;
    }

The above code works fine if there is integer value in item.Id. But when I change the HTML code to have string value in item.Id, nothing happens. It isn't even going to the function (checked by putting breakpoint).

HTML Code- with item.ID as string ***the below isn't working***

<td class="col-md-2" width="50" style="background-color: transparent;">
<input class="btn btn-sm btn-primary fa" id='iedit-\"" [email protected] +"\"' type="button" value="&#xf040" onclick='showedit(\"" [email protected]+ "\");' />
</td>

I searched and tried couple of similar options but nothing seem to be working for me. Please advise.

Thanks. But I am very new to all this and this is my first project involving JS & HTML. Unfortunately never used jsfiddle. After your comment I am trying to use jsfiddle. Gmme some time to try and post back. Thanks — Gaurav
– Gaurav, Commented Apr 4, 2016 at 10:37
see @lukbl answer in which quotes are added to the showedit() parameter. Missing quotes in your code seems the mostly likely problem. — Yogi
– Yogi, Commented Apr 4, 2016 at 10:50

lukbl · Accepted Answer · 2016-04-04 10:52:05Z

3

Are you using asp.net mvc on server?

 <td class="col-md-2" width="50" style="background-color: transparent;">
<input class="btn btn-sm btn-primary fa" id='[email protected]' type="button" value="&#xf040"  onclick='showedit("@(item.ID)");' />
</td>

Also notice that id of an input will be different than item.ID which you are passing to function - it is prefixed with "iedit" - make sure that does not cause any problems in your function.

edited Apr 4, 2016 at 10:52

answered Apr 4, 2016 at 10:46

lukbl

1,7831 gold badge9 silver badges13 bronze badges

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

1 Comment

Gaurav Over a year ago

Thanks a lot. Yes I am using asp.net mvc. This worked. Was struggling for last 3-4 hours. Will check the other point too. Thanks again for everything.

Collectives™ on Stack Overflow

Passing a string in onclick function's in for loop

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related