$_Get probleem

Door Marco Hopster op 03-04-2017 21:17 gewijzigd op 03-04-2017 21:23

1.318 views

Hallo Allemaal,

Ik ben sinds kort weer bezig om mijn oude/lang niet gebruikte site op te knappen.
Alleen loop ik vast op het volgende probleem.

Ik heb een beantwoord knop in een mail pagina, als je daar op klikt dan ga je naar een reply pagina :

<? <a style="color:white; cursor: hand; text-decoration:none;" href="javascript:ajax3.run('mail-reply.php?id=$idtje','content');" title="Beantwoorden"><img align=absmiddle src="icons1/email/reply.png" border="0">Beantwoorden</a> ?>

Ik merkte dat hij de volgende pagina niet opende, dus ben ik gaan kijken waar het aan kan liggen.
Nu haal ik met onderstaande script $idtje op, en zie dat hij er een tijdsaanduiding achter gooit. Iemand enig idee hoe dit kan?

<?php
if (!session_id()) { session_start(); };
$idtje=$_GET['id'];
$u_id=$_SESSION['is_id'];
$u_naam=$_SESSION['is_naam'];
$is_online2=$_SESSION['isonline'];
$post_datum=date("d-m-Y H:i");
include("config.php");
$db = mysqli_connect($db_host, $db_user, $db_pass, $db_name);
$lp="SELECT * FROM privemsg WHERE id='$id' limit 1";
$leespost=mysqli_query($db,$lp);
$aantal = mysqli_num_rows($leespost);
if ($aantal == 1) {
if ($myrow = mysqli_fetch_array($leespost)); {
$myonderwerp=$myrow['onderwerp'];
$aanp=$myrow['vannaam'];
}
}

echo "Ik $idtje";
?>

Het zelfde regeltje heb ik eerder gebruikt in de site, en daar werkt het perfect!

Dit is de echo wat ik krijg:

Ik $idtje?t=1491247349621

- Ariën -

03-04-2017 21:52

Ik denk dat het uit een ander stukje script komt.

Verder nog wat goede tips:

- Zorg voor goed inspringing met TAB, dit is nvoor nu behoorlijke spaghetti-code
- Escape je invoer in je queries met mysqli_real_escape_string()
- Variabelen horen buiten quotes:


<?php
echo "Ik ".$idtje;
?>

- Zorg voor duidelijke neemgeving in variabelen. Waarom geen $id i.p.v. $idtje.
- Foutafhandeling op je queries.
- Bij voorkeur geen variabelen overschrijven (lijn 3 t/m 6), waarom zou je terwijl je ie ook direct kan benoemen.

Ivo P

04-04-2017 01:03

je fout zit in die losse regel die je eerst post.

Je gebruikt daar <? ?> maar het is geen php.

Wel staat daar letterlijk $idtje in de link. Daarmee komt letterlijk die string in de url terecht. Kijk maar in de adresbalk van je browser.

En omdat je nergens valideert wat je uit $_GET ophaalt.....

[size=xsmall]Toevoeging op 04/04/2017 01:05:26:[/size]


<a style="color:white; 
   cursor: hand; text-decoration:none;" 
   href="javascript:ajax3.run('mail-reply.php?id=<?= $idtje?>','content');" 
   title="Beantwoorden">
   <img align=absmiddle 
        src="icons1/email/reply.png" 
        border="0">Beantwoorden</a>

Reageren

Inloggen om te reageren