Controleert niet op grootte

Door Marijn op 06-09-2006 22:22

1.356 views

Hey,

Ik heb hier een stukje code, waarbij die eigenlijk de grootte van de ingevoerde avatar moet controleren. Maar ook als ik een te groot plaatje invoer, dan nog zegt hij dat het profiel succesvol is gewijzigd...

Ziet iemand de fout?


<?php
if (loggedin())
{
    $list = mysql_fetch_Assoc(mysql_query("select * from leden where id = '".$_SESSION['userid']."'"));
    if (isset($_POST['submit']))
    {
        $mail = (isset($_POST['mail'])) ? $_POST['mail'] : '';
        $msn = (isset($_POST['msn'])) ? $_POST['msn'] : '';
        $icq = (isset($_POST['icq'])) ? $_POST['icq'] : '';
        $wplaats = (isset($_POST['woonplaats'])) ? $_POST['woonplaats'] : '';
        $gday = (isset($_POST['geboortedag'])) ? $_POST['geboortedag'] : '';
        $gmonth = (isset($_POST['geboortemaand'])) ? $_POST['geboortemaand'] : '';
        $gyear = (isset($_POST['geboortejaar'])) ? $_POST['geboortejaar'] : '';
        $avatar = (isset($_POST['avatar'])) ? $_POST['avatar'] : '';
        $gebdate = $gday."-".$gmonth."-".$gyear;
        $locatie = $_POST['avatar'];
        $grootte = getimagesize($locatie);
        $breedte = $grootte[0];        /*Breedte van het plaatje*/
        $hoogte = $grootte[1];         /*Hoogte van het plaatje*/
        if (empty($mail))
        {
            echo "<BR>Je hebt niet alle verplichte velden ingevuld.";
        }
        elseif ($breedte > '60' OR $hoogte > '60')
        {
            echo "<BR>De afmetingen van je avatar zijn te groot, maximale afmetingen zijn: <b>60</b> bij <b>60</b>";
        }
        else
        {
            mysql_query("update leden set mail = '".$mail."', msn = '".$msn."', icq = '".$icq."', woonplaats = '".$wplaats."', geboortedatum = '".$gebdate."', avatar = '".$avatar."' where id = '".$_SESSION['userid']."'") or die (mysql_error());
            echo "<BR>Je profiel is succesvol gewijzigd.";
        }
    }
    else
    {
        list($dag, $maand, $jaar) = explode('-', $list['geboortedatum']);
        echo "<BR>
            <table cellspacing=0 cellpadding=0>
            <form method=\"post\" action=\"\">
            <tr>
            <td width=\"193\">E-mail adres *:</td>
            <td><input type=\"text\" name=\"mail\" value=\"".$list['mail']."\" class=\"fill\"></td></tr>
            <tr>
            <td>MSN:</td>
            <td><input type=\"text\" name=\"msn\" value=\"".$list['msn']."\" class=\"fill\"></td></tr>
            <tr>
            <td>ICQ:</td>
            <td><input type=\"text\" name=\"icq\" value=\"".$list['icq']."\" class=\"fill\"></td></tr>
            <tr>
            <td>Woonplaats:</td>
            <td><input type=\"text\" name=\"woonplaats\" value=\"".$list['woonplaats']."\" class=\"fill\"></td></tr>
            <tr>
            <td>Geboortedatum <i>dd-mm-yyyy</i>:</td>
            <td><input type=\"text\" name=\"geboortedag\" style=\"width: 20px\" value=\"".$dag."\" class=\"fill\"> - <input type=\"text\" name=\"geboortemaand\" style=\"width: 20px\" value=\"".$maand."\" class=\"fill\"> - <input type=\"text\" name=\"geboortejaar\" style=\"width: 35px\" value=\"".$jaar."\" class=\"fill\"></td></tr>
            <tr>
            <td>Avatar-link (60 x 60):</td>
            <td><input type=\"text\" name=\"avatar\" value=\"".$list['avatar']."\" class=\"fill\"></td></tr>
            <tr>
            <tr><td></td><td><BR><input type=\"submit\" name=\"submit\" value=\"Wijzigen\" class=\"button\"></td></tr>
            </form>
            </table><BR>
            Velden met een <b>*</b> zijn verplicht om in te vullen.
        ";
    }
}
else
{
    echo "<BR>Je bent niet ingelogd en hebt dus geen toegang tot deze pagina.";
}
?>

- SanThe -

08-09-2006 13:12

Verder doe je
$avatar = (isset($_POST['avatar'])) ? $_POST['avatar'] : '';

en ga je toch weer verder met
$locatie = $_POST['avatar'];
$grootte = getimagesize($locatie);

Dat moet zijn (Waarom trouwens een nieuwe $var?)
$locatie = $avatar;
$grootte = getimagesize($locatie);

Marijn

08-09-2006 13:24


<?
        $locatie = $avatar;
        $grootte = getimagesize($locatie);
        $breedte = $grootte[0];        /*Breedte van het plaatje*/
        $hoogte = $grootte[1];         /*Hoogte van het plaatje*/
		if (($breedte > 60) OR ($hoogte > 60)) 
		{
			echo "De afmetingen van je avatar zijn te groot, maximale afmetingen zijn: <b>60</b> bij <b>60</b>";
		}
		elseif (empty($mail))
		{
			echo "<BR>Je hebt niet alle verplichte velden ingevuld.";
		}
		else
		{
			mysql_query("update leden set mail = '".$mail."', msn = '".$msn."', icq = '".$icq."', woonplaats = '".$wplaats."', geboortedatum = '".$gebdate."', avatar = '".$avatar."' where id = '".$_SESSION['userid']."'") or die (mysql_error());
			echo "<BR>Je profiel is succesvol gewijzigd.";
		}
?>

Nog steeds zegt ie bij een te groot plaatje dat het profiel succesvol is gewijzigd.. :(

- SanThe -

08-09-2006 14:50

<?
$locatie = $avatar;
$grootte = getimagesize($locatie);
if($grootte)
{
$breedte = $grootte[0]; /*Breedte van het plaatje*/
$hoogte = $grootte[1]; /*Hoogte van het plaatje*/
if (($breedte > 60) OR ($hoogte > 60))
{
echo "De afmetingen van je avatar zijn te groot, maximale afmetingen zijn: 60 bij 60";
}
}
else
{
echo 'Geen avatar gevonden.';
}

if(empty($mail))
{
echo " Je hebt niet alle verplichte velden ingevuld.";
}
else
{
mysql_query("update leden set mail = '".$mail."', msn = '".$msn."', icq = '".$icq."', woonplaats = '".$wplaats."', geboortedatum = '".$gebdate."', avatar = '".$avatar."' where id = '".$_SESSION['userid']."'") or die (mysql_error());
echo " Je profiel is succesvol gewijzigd.";
}
?>

Maar al met al heeft het niet veel zin om de grootte van de avatar te checken voordat ie in de database wordt gezet. Stel dat men de avatar later vervangt door een superavatar? Je moet de grootte checken op het moment dat je de avatar wilt gebruiken.

Marijn

08-09-2006 15:09

de output is nu (bij een groot en een klein plaatje):
"Geen avatar gevonden.
Je profiel is succesvol gewijzigd."

- SanThe -

08-09-2006 15:13

Dan zal ie het plaatje niet gevonden hebben.

Reageren

Inloggen om te reageren