[opgelost] unexpected script
Groetjes Rick,
Dit is de error:
---------------------------------
Parse error: syntax error, unexpected $end in /storage/mijndomein/users/000000/public/sites/www.mijnsite.nl/nieuws/delete.php on line 22
---------------------------------
En dit is het 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
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<?php
$index = "adminalleberichten.php";
//Kijken naar wat na "?id=" is ingevuld
$id = $_GET['id'];
if (is_dir("alle/".$id))
{
if (isset($_POST['submit']) AND !empty($_POST['submit']))
{
//Delete function
if($_GET['id'] == 'delete')
{
if(ctype_digit($_GET["id"])) //is_numeric kan ook een float zijn
{
$sql = "DELETE FROM messages WHERE id = '".$_GET['id']."'";
$res = mysql_query($sql);
header("Location: ".$index); //dit is de manier om te redirecten, hoe jij het had werkt volgens mij niet eens en is raar..
}
}
?>
$index = "adminalleberichten.php";
//Kijken naar wat na "?id=" is ingevuld
$id = $_GET['id'];
if (is_dir("alle/".$id))
{
if (isset($_POST['submit']) AND !empty($_POST['submit']))
{
//Delete function
if($_GET['id'] == 'delete')
{
if(ctype_digit($_GET["id"])) //is_numeric kan ook een float zijn
{
$sql = "DELETE FROM messages WHERE id = '".$_GET['id']."'";
$res = mysql_query($sql);
header("Location: ".$index); //dit is de manier om te redirecten, hoe jij het had werkt volgens mij niet eens en is raar..
}
}
?>
Gewijzigd op 01/01/1970 01:00:00 door Rick van Hout
Je sluit niet alle if'jes af.
Kan iemand ze er misschien inzetten? :-)
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
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
<?php
$index = "adminalleberichten.php";
//Kijken naar wat na "?id=" is ingevuld
$id = $_GET['id'];
if (is_dir("alle/".$id))
{
if (isset($_POST['submit']) AND !empty($_POST['submit']))
{
//Delete function
if($_GET['id'] == 'delete')
{
if(ctype_digit($_GET["id"])) //is_numeric kan ook een float zijn
{
$sql = "DELETE FROM messages WHERE id = '".$_GET['id']."'";
$res = mysql_query($sql);
header("Location: ".$index); //dit is de manier om te redirecten, hoe jij het had werkt volgens mij niet eens en is raar..
}
}
}
}
?>
$index = "adminalleberichten.php";
//Kijken naar wat na "?id=" is ingevuld
$id = $_GET['id'];
if (is_dir("alle/".$id))
{
if (isset($_POST['submit']) AND !empty($_POST['submit']))
{
//Delete function
if($_GET['id'] == 'delete')
{
if(ctype_digit($_GET["id"])) //is_numeric kan ook een float zijn
{
$sql = "DELETE FROM messages WHERE id = '".$_GET['id']."'";
$res = mysql_query($sql);
header("Location: ".$index); //dit is de manier om te redirecten, hoe jij het had werkt volgens mij niet eens en is raar..
}
}
}
}
?>
hij verwijdert hem niet, en hij gaat niet terug naar de $index... :(
delete is niet numeriek dus zodoende verwijderd hij niets.
Je moet het dan op de volgende manier doen:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
<?php
$index = "adminalleberichten.php";
//Kijken naar wat na "?id=" is ingevuld
$id = $_GET['id'];
if (is_dir("alle/".$id))
{
if (isset($_POST['submit']) AND !empty($_POST['submit']))
{
//Delete function
if($_GET['id'] == 'delete')
{
$sql = "DELETE FROM messages WHERE id = '".$_GET['id']."'";
$res = mysql_query($sql);
header("Location: ".$index); //dit is de manier om te redirecten, hoe jij het had werkt volgens mij niet eens en is raar..
}
}
}
?>
$index = "adminalleberichten.php";
//Kijken naar wat na "?id=" is ingevuld
$id = $_GET['id'];
if (is_dir("alle/".$id))
{
if (isset($_POST['submit']) AND !empty($_POST['submit']))
{
//Delete function
if($_GET['id'] == 'delete')
{
$sql = "DELETE FROM messages WHERE id = '".$_GET['id']."'";
$res = mysql_query($sql);
header("Location: ".$index); //dit is de manier om te redirecten, hoe jij het had werkt volgens mij niet eens en is raar..
}
}
}
?>
edit** het is een MAP/FOLDER die hij moet verwijderen.
Gewijzigd op 01/01/1970 01:00:00 door Rick van Hout
Dan zul je je script toch moeten aanpassen, je laat hem nu een record uit de database verwijderen.
Rick
Je controleert dus of $_GET['id'] een map is... en later of die 'delete' is.
Je hebt dus een map genaamd 'delete'?
Volgens mij moet je die eens controleren (regel 12).
Ga maar eens kijken naar de .htaccess mod_rewrite, dan hoef je niet meer al die mappen aan te maken, en kun je via simpele $_GET-variabelen al mooie URLs uitgekleed krijgen.
Dit stukje script gaat nergens over, je kijkt eerst of $_GET['id'] een string is, en dan wil je $_GET['id'] alszijnde het id van een bericht gaan gebruiken. Dat werkt natuurlijk niet. Bekijk je logica opnieuw en begin bij het begin en denk goed na.
heeft er toevallig iemand zo'n script?
-moet een VARIABLE map verwijderen
Groetjes,,
Die map bevat een id? Iets als delete=27? Waarbij 27 het id is? Ja. Blijkbaar heb je geen kaas gegeten van SEO of .htaccess. Kijk nou eens naar mijn post: gebruik .htaccess en je bent van al je mappen en problemen af.
het is opgelost, bedankt :-)