Ваш атрибут onClick пропускает завершающую одинарную кавычку. Вот что я думаю должно быть:
printf("<a href='/r?%s' target=_blank onClick='window.open(location.href=$siteurl/%s-%s.html)'>", $wurl, $myrow["wallpaperid"], $myrow["wallpapername"]);
Кстати: мне было трудно читать даже эту единственную строку. Подумайте о том, чтобы сделать код более понятным для человека. Кроме того, я рекомендую не использовать подстановку переменных (переменная $ siteurl в вашей строке), когда вы все равно используете printf. Я имею в виду, использовать тот же механизм, чтобы сделать то же самое. Либо используйте только подстановку var, либо используйте только printf formattng, а не оба. Пример:
$html = '
<a href="/r?%s"
target=_blank
onClick="window.open(location.href=%s/%s-%s.html)"
>
';
printf($html, $wurl, $siteurl, $myrow["wallpaperid"], $myrow["wallpapername"]);