Hallo Roman

var popup = open(url, popup, 'width=299, height=222');

1. Ich würde die Variable nicht wie die Funktion benennen. Funktionen in JS sind afaik selbst nur Referenzen auf Funktionsobjekte und es gibt somit Namensraumkonflikte
2. Wieso übergibst du open() die Variable popup? Das macht kein SInn und geht nicht, da popup zu dem Zeitpunkt noch nicht initialisiert ist. Wenn du popup als Namen übergeben willst, stelle es in Anführungszeichen.

Grüße

Daniel