Отправить значение переменной функции onclick

Question

У меня есть таблица, и в каждой ячейке есть изображение редактирования, поэтому, когда пользователи нажимают на него, они будут видеть окно ввода в ячейке. (Ниже приведен пример одного из клеток)

<td> 
     echo 
      "<p id=$idl>" . $var . "</p>" . "<img style ='cursor:pointer; ' class='onInput' src='http://nimbuzz007.hexat.com/icon4/icon%2015.png' id='input_img' onclick='legacyFunction($idl,$var)'>"; 

    </td> 

и это 'legacyFunction':

function legacyFunction($idl,$var) { 

    document.getElementById($idl).innerHTML = 
    "<input value=$var></input>"; 

};

поэтому я хочу отправить текущие данные ячейки функции, чтобы показать это в поле ввода. Но это не работает. Какая у меня ошибка? можно ли это сделать?

Answer 1

Вам нужны цитаты вокруг аргументов, так как они являются строками.

onclick='legacyFunction(\"$idl\",\"$var\")' 

Answer 2

Как другие заявили о своей плохой практике для смешивания html, js и php. Я всегда стараюсь разделить код, когда это возможно.

Пример:

<td> 
    <p id="<?php echo $idl; ?>"><?php echo $var; ?></p> 
    <img style="cursor: pointer;" class="onInput" src="http://nimbuzz007.hexat.com/icon4/icon%2015.png" onclick="legacyFunction(<?php echo "$idl, $var"; ?>);"> 
</td> 
<script> 
    function legacyFunction($idl, $var) { 
     // Inputs are void elements and as such they do not need closing tags - </input> 
     document.getElementById($idl).innerHTML = '<input value="'+ $var + '">'; 
    }; 
</script>