De PHPhulp kroeg
Action niet benoemd
fout:
Notice: Undefined variable: action in C:\server\Apache2\htdocs\beheer\index.php on line 60
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
<?php
//parameters lezen
if(isset($_GET['action'])) $action=$_GET['action'];
if(isset($_POST['action'])) $action=$_POST['action'];
//Opdrachten voor het wissen van een record
if($action=="delete"){
//parameters lezen
$news_ID=$_GET['news_ID'];
//Record wissen
$news_SQL_del="DELETE FROM news WHERE news_ID=$news_ID";
$bool=mysql_query($news_SQL_del);
if($bool==1) echo "<SCRIPT LANGUAGE=JavaScript>window.alert('Bericht is gewist')</SCRIPT>";
if($bool<>1) echo "<SCRIPT LANGUAGE=JavaScript>window.alert('Bij het wissen is een fout opgetreden')</SCRIPT>";
}
?>
//parameters lezen
if(isset($_GET['action'])) $action=$_GET['action'];
if(isset($_POST['action'])) $action=$_POST['action'];
//Opdrachten voor het wissen van een record
if($action=="delete"){
//parameters lezen
$news_ID=$_GET['news_ID'];
//Record wissen
$news_SQL_del="DELETE FROM news WHERE news_ID=$news_ID";
$bool=mysql_query($news_SQL_del);
if($bool==1) echo "<SCRIPT LANGUAGE=JavaScript>window.alert('Bericht is gewist')</SCRIPT>";
if($bool<>1) echo "<SCRIPT LANGUAGE=JavaScript>window.alert('Bij het wissen is een fout opgetreden')</SCRIPT>";
}
?>
Gewijzigd op 01/01/1970 01:00:00 door Jeffrey boud
Overigens zou ik voor de duidelijkheid { en } gebruiken. Leest wat makkelijker en is overzichtelijker
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
<?php
//parameters lezen
if(isset($_GET['action'])) $action=$_GET['action'];
if(isset($_POST['action'])) $action=$_POST['action'];
//Opdrachten voor het wissen van een record
if(isset($action) && $action=="delete"){
//parameters lezen
$news_ID=$_GET['news_ID'];
//Record wissen
$news_SQL_del="DELETE FROM news WHERE news_ID=$news_ID";
$bool=mysql_query($news_SQL_del);
if($bool==1) echo "<SCRIPT LANGUAGE=JavaScript>window.alert('Bericht is gewist')</SCRIPT>";
if($bool<>1) echo "<SCRIPT LANGUAGE=JavaScript>window.alert('Bij het wissen is een fout opgetreden')</SCRIPT>";
}
?>
//parameters lezen
if(isset($_GET['action'])) $action=$_GET['action'];
if(isset($_POST['action'])) $action=$_POST['action'];
//Opdrachten voor het wissen van een record
if(isset($action) && $action=="delete"){
//parameters lezen
$news_ID=$_GET['news_ID'];
//Record wissen
$news_SQL_del="DELETE FROM news WHERE news_ID=$news_ID";
$bool=mysql_query($news_SQL_del);
if($bool==1) echo "<SCRIPT LANGUAGE=JavaScript>window.alert('Bericht is gewist')</SCRIPT>";
if($bool<>1) echo "<SCRIPT LANGUAGE=JavaScript>window.alert('Bij het wissen is een fout opgetreden')</SCRIPT>";
}
?>
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
<?php
//parameters lezen
if(isset($_GET['action'])){ // Waarom zowel een GET als een POST? Vreemde constructie
$action = $_GET['action'];
}
elseif (isset($_POST['action'])){
$action = $_POST['action'];
}
else {
$action = ''; // Nu zal $action altijd bestaan, wel zo handig
}
//Opdrachten voor het wissen van een record
if($action == "delete" && isset($_GET['news_ID']) && ctype_digit($_GET['news_ID'])){
//parameters lezen
$news_ID = mysql_real_escape_string(); // Beveiliging!!!!
//Record wissen
$news_SQL_del = "DELETE FROM news WHERE news_ID=".$news_ID;
$bool = mysql_query($news_SQL_del);
if($bool){ // Query is gleukt, zegt niets over verwijderde berichten! Zie de WHERE in jouw query...
$count = mysql_affected_rows();
if ($count > 0) {
echo "<script language=JavaScript>window.alert('Bericht is gewist')</script>";
}
else {
echo "<script language=JavaScript>window.alert('Er zijn geen berichten gewist')</script>";
}
}
else { // Geen foutmelding over een mislukte query? Mag je zelf weten
echo "<script language=JavaScript>window.alert('Bij het wissen is een fout opgetreden')</script>";
}
}
else {
echo "<script language=JavaScript>window.alert('Ongeldige waarde opgegeven')</script>";
}
?>
//parameters lezen
if(isset($_GET['action'])){ // Waarom zowel een GET als een POST? Vreemde constructie
$action = $_GET['action'];
}
elseif (isset($_POST['action'])){
$action = $_POST['action'];
}
else {
$action = ''; // Nu zal $action altijd bestaan, wel zo handig
}
//Opdrachten voor het wissen van een record
if($action == "delete" && isset($_GET['news_ID']) && ctype_digit($_GET['news_ID'])){
//parameters lezen
$news_ID = mysql_real_escape_string(); // Beveiliging!!!!
//Record wissen
$news_SQL_del = "DELETE FROM news WHERE news_ID=".$news_ID;
$bool = mysql_query($news_SQL_del);
if($bool){ // Query is gleukt, zegt niets over verwijderde berichten! Zie de WHERE in jouw query...
$count = mysql_affected_rows();
if ($count > 0) {
echo "<script language=JavaScript>window.alert('Bericht is gewist')</script>";
}
else {
echo "<script language=JavaScript>window.alert('Er zijn geen berichten gewist')</script>";
}
}
else { // Geen foutmelding over een mislukte query? Mag je zelf weten
echo "<script language=JavaScript>window.alert('Bij het wissen is een fout opgetreden')</script>";
}
}
else {
echo "<script language=JavaScript>window.alert('Ongeldige waarde opgegeven')</script>";
}
?>