Javascript vars versturen werkt niet

[willem-alex]

Ik wil vanuit een PHP bestand een popup openen en in deze popup enkele variabelen laden.
Hieronder zie je de link:

code:

<a href=\"#\" onClick=\"return do_submit('$module','$menutitelid','$item_short')\">aanpassen</a>

En hieronder het scriptje waarnaar hij verwijst:

code:

<script>
<!-- 
function do_submit(A,B,C) {
    var page = 'menu_item_edit.php?module=' + A + '&menutitelid=' + B + '&menuitem=' + C;
    var properties = 
       'status=no,toolbar=no,menubar=no,resizable=no,scrollbars=no,top=100,left=100,width=500,height=400;
    window.open(page,'titeltje',properties);
    return false;
}

// -->
</script>

Waarom werkt dit niet?

[ Voor 14% gewijzigd door willem-alex op 05-08-2003 19:00 ]

[Bosmonster]

WAT werkt niet?

En je schrijft van die variabelen die je noemt wel de inhoud weg toch?

Oftewel, geef de gegenereerde code eens ipv de PHP code..

[willem-alex]

Gegenereerde codes:

code:

<script>
<!-- 
function do_submit(A,B,C) {
    var page = 'menu_item_edit.php?module=' + A + '&menutitelid=' + B + '&menuitem=' + C;
    var properties = 
       'status=no,toolbar=no,menubar=no,resizable=no,scrollbars=no,top=100,left=100,width=500,height=400;
    window.open(page,'titeltje',properties);
    return false;
}

// -->
</script>

code:

<a href="#" onClick="return do_submit('projecten','5','7')">aanpassen</a>

[crisp]

Je moet haast wel een script error krijgen want ik mis een single quootje aan het einde van de var properties=

en laat die hide-script-from-older-browsers dingetjes aub achterwege en geef een type declaratie op in je script-tag

[willem-alex]

Heel scherp crisp! Hier was ik dus al drie kwartier mee bezig....
Erg bedankt voor de hulp.

[whoami]

P&W -> W&G

Javascript hoort niet thuis in P&W, zie de forumdescriptions.

[Johnny]

code:

<script type="text/javascript">

het is inderaad het aanhalingsteken achter properties=

[ Voor 50% gewijzigd door Johnny op 05-08-2003 19:13 ]

[crisp]

duidelijk; weet iemand nog een tutje javascript debuggen?