1

Хочу по клике на ссылку передать два значения в функцию, делаю так:

onclick="gov('.$id.', "info")

при клике получаю ошибку в консоле хрома: Uncaught ReferenceError: info is not defined at HTMLAnchorElement.onclick

Функция: function gov(id, type) { ... }

при передачи одного значения все работает. В чем может быть проблема?

UPD. Заметил что если вторым значением передавать цифру - все работает, а с текстом нет

4
  • 2
    Перестаньте использовать on... атрибуты. Очень рекомендую Commented 31 янв. 2018 в 12:48
  • похоже на очепятку, попробуйте обернуть 'info' в одинарные кавычки
    – yolosora
    Commented 31 янв. 2018 в 12:49
  • @yolosora в одинарные нельзя, потому что это ссылка в echo '<a ...>';
    – iKey
    Commented 31 янв. 2018 в 12:57
  • @Денис экранируйте их Commented 31 янв. 2018 в 13:11

2 ответа 2

0

В зависимости от того, чем является info. Если просто строкой, значит надо писать либо так

onclick="gov('.$id.', 'info')"

либо экранировать двойные кавычки

onclick="gov('.$id.', \"info\")"

в одинарные нельзя, потому что это ссылка в echo '<a ...>';

Значит надо работать с экранированием внутри вывода строки в PHP:

<?php
$id = 10;
echo '<a href="#" onclick="gov('.$id.', \'infoHello\')">click me</a>'

?>

<script>
function gov(id, type) {
    alert(id + ', ' + type);
}
</script>
3
  • info просто строка, слово. не помогло((
    – iKey
    Commented 31 янв. 2018 в 12:56
  • не верю jsfiddle.net/vupf3zuk Commented 31 янв. 2018 в 13:01
  • @Денис взглянул на комментарий под вопросом. Добавил в ответ. В общем всё сводится к экранированию. Если не в JS, то в PHP. Надо это помнить всегда, если уж рисуете много кавычек в строке Commented 31 янв. 2018 в 13:05
0

Так не надо делать

echo '<a href="" onclick="gov('.$id.', \'info\')">';

Поскольку у вас есть jQuery, можно сделать, например, так:

$params = htmlspecialchars(json_encode(array('id' => $id, 'type' => 'info')));
echo '<a href="" data-params="' . $params . '">';

Дальше в <script>

$('[data-params]').on('click', function(e) {
    var params = $(this).data('params');
    gov(params.id, params.type);
}

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.