Du weisst dem onclick Handler lediglich einen String zu. So wie du es im HTML Code machen würdest (selbst dort ist das javascript: lediglich ein überflüssiges Label und gehört eigentlich auch nicht dorthin)
Ich hab hier mal versucht http://javascript.jstruebig.de/exp/events.html das erklären.
Ich habe mal versucht nach deiner Anleitung meinen Code umzuschreiben.
Das sieht jetzt so aus (ist auch auf dem Server):
var obj=document.getElementById("vorschau1");
var bild=art+"_1.jpg";
obj.onclick = wechsel(bild,2);
obj=document.getElementById("vorschau2");
bild=art+"_2.jpg";
obj.onclick = wechsel(bild,2);
Jetzt führt er die wechsel-Funktion direkt aus, obwohl sie erst ausgeführt werden soll, wenn auf das Vorschaubild geklickt wird.
Was habe ich noch falsch gemacht?