Hallo,

Nu ben ik echt het spoor bijster.
Ik heb nu een stukje code, die de UPDATE niet uitvoerd.

<?php
echo "Updating mail info<br>";

$user = $_SESSION['Name'];
$id = $_GET['id'];

echo $user."<br>".$id."<br>";

$sql = 'UPDATE Mail SET By="'.$user.'" WHERE ID="'.$id.'"';

// if(mysqli_query($connect, $sql)) {
// echo "<br>Record Updated Succesfully";
// } else {
// echo "<br>Error Updating record";
// }
echo $sql;
// header('Location: index.php?menu=contact');
?>

Haal ik de // voor de onderste code weg, doet hij ook niks en de echo output is wel okay.
Maar hij zet niks in de Dbase weg terwijl de DBase wel de juiste velden heeft

output van de echo

Updating mail info
Arno van Zanten
1
UPDATE Mail SET By="Arno van Zanten" WHERE ID="1"
Scripts zijn niet veilig.
Mysql injection.
Altijd inkomende gegevens checken.
Of mysqli_real_escape() gebruiken.

[size=xsmall]Toevoeging op 04/06/2022 21:05:09:[/size]

Arno van Zanten op 04/06/2022 21:00:52

Gedaan maar blijft het zelfe doen


En dat is???

[size=xsmall]Toevoeging op 04/06/2022 21:07:04:[/size]

- SanThe - op 04/06/2022 20:59:46

Bij beiden echo's <td> gebruiken anders valt de een buiten je tabel.
<?php
if(empty($mail['User'])){
echo "<td>Nobody</td>";
} else {
echo "<td>".$mail['User']."</td>";
}
?>


NJormaal kun je heel gemakkelijk aangeven als een veld leeg is, dat hij een process moet uitvoeren, maar bij deze wordt ik er een beetje gek van

[size=xsmall]Toevoeging op 04/06/2022 21:14:04:[/size]

- SanThe - op 04/06/2022 21:04:18

Scripts zijn niet veilig.
Mysql injection.
Altijd inkomende gegevens checken.
Of mysqli_real_escape() gebruiken.

[size=xsmall]Toevoeging op 04/06/2022 21:05:09:[/size]

[quote="Arno van Zanten op 04/06/2022 21:00:52"]
Gedaan maar blijft het zelfe doen

Ik zag wel idd dat ik daar de <td></td vergeten was, maar alsnog, werkt het niet...


En dat is???

[size=xsmall]Toevoeging op 04/06/2022 21:07:04:[/size]

- SanThe - op 04/06/2022 20:59:46

Bij beiden echo's <td> gebruiken anders valt de een buiten je tabel.
<?php
if(empty($mail['User'])){
echo "<td>Nobody</td>";
} else {
echo "<td>".$mail['User']."</td>";
}
?>



[/quote]

Je geeft niet echt duidelijk aan wat er wel en wat er niet gebeurt.
Zo wordt het gissen.
Werkt, bedankt voor jullie hulp alweer
Ok.
Succes verder.
empty() laat meer door dan je denkt. Kijk maar op http://php.net/empty.

Ikzelf haal eerst trim() over de string om de eventuele spaties voor en achter weg te halen. En daarna vergelijk ik het met een lege string "".
Ik heb de code aangepast

<?php
if(isset($mail['User']) && empty($mail['User'])){
echo "<td>Nobody</td>";
} else {
echo "<td>".$mail['User']."</td>";
}
?>
Deze moest het zijn
Mijn reactie nog gezien?

Reageren