[php] unexpected $?

[fromalk]

Lo, voor het eerst krijg ik de volgende error:
Parse error: parse error, unexpected $ in /home/uds/public_html/resources/activate.inc.php on line 46
Nu regel 46 bevat ?>, en ik zie nergens een verkeerd geplaatst $ in de code:

code:

<?php
$smarty -> assign('display', 'false');
$smarty -> assign('page', $_SERVER[PHP_SELF]);
$smarty -> assign('myidexpl', 'There is an activation ID in the mail you have receaved when registering. Please fill that ID in here.');
$smarty -> assign('mykeyexpl', 'There is an activation key in the mail we\'ve send you after registering, please fill it in.
<br /><b>Note: the key field is case-sensitive!</b>');
$smarty -> assign('header', '» Me.UnDreamed / Activate');
$smarty -> assign('abb', '» Activating account');

$g = 1;
if(isset($_GET['type']))
{
    if($_GET['type']="auto" || $_GET['type']="manual")
    {
        //
        // Controling if request for registering has been made
        //
        if(isset($_GET['filled']))
        {
            $g = 0;
            $myid = strip_tags($_GET['id']);
            $mykey = strip_tags($_GET['key']);
            
            $smarty->assign('did', $myid);
            $smarty->assign('dkey', $mykey);
            
            $result_id = mysql_query("SELECT user_ID FROM share_users WHERE user_ID='".addslashes($myid)."'"); 
            if(mysql_num_rows($result_id)>0)
            { 
                $results[$g++] = 'This id dosn\'t exist'; 
            }
            $smarty->assign('error', $results);
        }
}
else
{
    //
    // User wants to manualy activate his account
    //
    $smarty -> assign('display', 'true');
    $smarty -> assign('id', 'Activation ID:');
    $smarty -> assign('key', 'Activation key:');
    
}
$smarty -> display('meuds_activate.tpl');
?>

[Peedy]

Je mist een }. Zet een } onder die op regel 44 en het zou moeten werken.

offtopic:
GoT is btw geen helpdesk voor zulke dingen, zulke dingetjes kan je snel zelf debuggen door beter naar je eigen code te loeren.

[ Voor 54% gewijzigd door Peedy op 04-02-2005 18:46 ]

[GlowMouse]

pEeDy16 schreef op vrijdag 04 februari 2005 @ 18:45:
Je mist een }. Zet een } onder die op regel 44 en het zou moeten werken.

Het werkt dan wel, maar niet zoals bedoeld.
Hij kan beter

PHP:

    if($_GET['type']="auto" || $_GET['type']="manual")
    {

afsluiten op regel 33

[Skaah]

Er mist een accolade-sluiten tussen regel 33 en 34.

edit:

* traag is

[ Voor 23% gewijzigd door Skaah op 04-02-2005 18:49 ]

[Peedy]

GlowMouse schreef op vrijdag 04 februari 2005 @ 18:47:
[...]

Het werkt dan wel, maar niet zoals bedoeld.
Hij kan beter

PHP:

    if($_GET['type']="auto" || $_GET['type']="manual")
    {

afsluiten op regel 33

Jah..doh... ff te snel gekeken haha... zulke foutjes heb ik zelf héél vaak vandaar...

[fromalk]

hartelijk dank! Vreemd dat ik daar telkens maar heb over gekeken :-s

[Verwijderd]

Ik denk sowieso dat dit niet klopt:

PHP:

if($_GET['type']="auto" || $_GET['type']="manual")

Hier controleer je namelijk de variabele niet maar je set ze.
Dit is denk wat je bedoeld:

PHP:

if($_GET['type']=="auto" || $_GET['type']=="manual")

[Soultaker]

Even voor de duidelijkheid: met dat dollarteken wordt dus het einde van het bestand bedoelt. Beetje onhandige foutmelding voor een PHP-parser (omdat het 'echte' dollarteken ook vaak genoeg gebruikt wordt). De parser meldt dus dat het einde van het bestand onverwacht was (omdat er nog een code block open stond), niet dat er ergens een dollarteken verkeerd staat.

[Creepy]

Daarnaast is dit typisch iets waarvan we hier verwachten dat je zelf in staat bent om dit op te lossen. Parse error e.d. mogen niet echt een probleem voor je zijn