Да, правильно, мы возвращаем false - перехода нету.
Но: мы для всей строки уже указали переход в onclick:
<tr align=\"center\" class=\"price align=\"center\" bgcolor=\"FAF9C9\"
onclick=\"window.location.href='info.php?t=pkv&id=$idskv'\" style=\"cursor:pointer\"
onMouseOver=this.style.backgroundColor='7A9EF9' onMouseOut=this.style.backgroundColor='FAF9C9'>
И получается, что как-бы один "слой" onclick накладывается на другой, и в результате если не срабатывает второй (во время добавления в избранное), срабатывает первый...
P.S. первый убрать нельзя, т.к. там идет изменение цвета и ссылка на следующую страницу...