2015-08-24 2 views
0

У меня есть таблица, где значения заполняются на основе SQL-запроса. Значения заполняются в диалоговом окне, но когда я нажимаю на запись, она не выполняет мою функцию. Я хочу, чтобы моя функция только выбирала идентификатор записи, на который я нажал. Теперь я поставил предупреждение в функцию, чтобы увидеть, работает ли моя функция и что она не показывает ничего. My onClick = "addlink();" не работает.JQuery-диалог не работает моя функция

function addreferral() 
{ 
    $.getJSON('libs/getsoldreferrals.php', function(data) { 
    $('#soldlist tr').not(':first').not(':last').remove(); 

$.each(data, function(key, val) 
{ 
    $('#soldlist').append('<tr><td><span id="myreferral" style="cursor:pointer;" onClick = "addlink();" >'+val.id +'</span></td><td></td><td>' + val.office + '</td><td></td><td>'+val.clientdetails+'</td><td></td><td>'+val.buyerorseller+'</td><td></td><td></td><td>'+val.address+'</td><td>&nbsp;</td></tr>'+'<tr><td></td><td></td><td></td><td></td><td></td><td></td><td></td><td></td><td></td><td></td><td></td></tr>'+'<tr><td></td></tr>'); 
}); 

    $('#soldlist tr').first().after(); 


}); 


var dialog, form 
window.addlink = function addlink() { 
     alert("Hello"); 
} 

    dialog = $("#dialog-form-referral").dialog({ 
     autoOpen: true, 
     height: 450, 
     width: 800, 
     modal: true, 
     buttons: { 
     Cancel: function() { 
      dialog.dialog("close"); 
     } 
     }, 

    }); 
+0

Вы передаете возвращаемое значение из addlink для обработчика onclick, попробуйте использовать onclick = "addlink", также "onClick" должен быть "onclick" – stubiklaus

+0

Не имеет отношения к вопросу - Но - Логическая ошибка здесь: .each (data, function (key, val) - вы создаете более одного с тем же id = "myreferral" - в HTML ваш идентификатор должен быть уникальным на той же странице - –

ответ

1

У вас есть синтаксическая ошибка. Вы забыли закрытие кавычки в своем теге стиля.

Изменение разметки вашего диапазона в поле ниже:

<span id="myreferral" style="cursor:pointer;" onClick = "addlink();"> 

Довольно уверен, что должен это сделать.

+0

Это сработало спасибо – Taps101

+0

Отредактируйте свой ответ, чтобы показать HTML - Выбрать HTML и Ctrl + K –

0

Попробуйте использовать onclick (маленький C), а не onClick. Кроме того, постарайтесь не смешивать JavaScript с HTML.

HTH's Станко.