Заменить это:
var getvalue = $('this > p > img').attr('rel');
С этим:
var getvalue = $(this).find('p > img').attr('rel');
Как сейчас, вы выполняете глобальный поиск элементов с буквенным именем тега this
Эквивалентный код также будет:
var getvalue = $('p > img', this).attr('rel');
Хотя, судя по всему, элементы представляют собой комбинации изображений / надписей, и вы не ожидаете, что другие изображения будут лучше, в этом случае лучше просто сделать:
var getvalue = $(this).find('img').attr('rel');
Или, может быть, дать ему описательный класс и заменить img
на img.someclassname
- это сделает так, что если вы позже отредактируете свой HTML, ваш Javascript не сломается из-за вашего конкретного селектора.
Кроме того, вы можете просто привязать событие click к самим изображениям:
$('div.item img').click(function() {
var getvalue = $(this).attr('rel');
});