How to alter the code so it's in one page delete-pictures or redirect via $_GET

The_Dude_1978 · August 8, 2011

Hi guy's,

I've figure out most of the code (propably simple to you) for my admin page. But i'm stuck with the deleting of pictures because when it goes to deletefile.php it say's it can't find the path. The pictures are stored in a folder and mysql is refering to it. It seems the $_GET['username'] is not coming along to the deletefile.php. Echoing it results in blank page.

So what I'm trying to do is, get the form to process within the admin page or pass the username variable to the deletefile.php with $_GET or $row['username']

Can you help me out? I've tried allot, but it's complicated for me.

Oh, almost forgot: The !profile_id in deletefile.php is just how the code was, but not needed, because i want to delete the pics as administrator. (it will redirect me to the index page). This was al bases on sessions, which I don't want to handle users.

// link in the admin page:
// | <a href='administrator.php?delpic&id=<?php echo $row['id'];?>'>Delete Pics)</a> |
<?php 
}
//action: delete pictures -----------------------------------------------------------------------------
if (isset($_GET['delpic']) && isset($_GET['id'])) {
$id = (int) $_GET['id'];
if ($id == 0) {
	die("Invalid ID provided.");
}

if($_GET['id'])
{

$sql="SELECT `id`, `first`, `last`, `username`, `email`, `about`, `level` from `users` WHERE `id`='".$_GET['id']."'";
$res=mysql_query($sql);
$row=mysql_fetch_assoc($res);

if (!isset($_POST['deletePic']))
{
?>

<!--------------------DELETE PICTURES---------------------//-->

<html>
<head>
</head>

<body>   
<div class="dividerp">

<?php

$sql2 = "SELECT `profile_id`, `title`, `size`, `type`, `reference` FROM user_photos WHERE profile_id=".$_GET['id']."";
$res2 = mysql_query($sql2) or die(mysql_error());
            
if(mysql_num_rows($res2) > 0)
	{

	echo "<strong>Delete Pictures for</strong><br/><br/>";
	echo "<form name=\"deletefile\" method=\"post\" action=\"deletefile.php\">";

	while($file = mysql_fetch_array($res2))
		{
			echo "<br/><br/><input name=\"files[]\" type=\"checkbox\" value=\"".$file['reference']."\">";
			echo "<a href=\"".$row['username']."/pics/".$file['reference']."\"/>
			<img src=\"".$row['username']."/pics/thumbs/".$file['reference']."\"/></a>";
		}
	echo "<br/><br/><input type=\"submit\" name=\"deletePic\" value=\"Delete Files\">";

	echo "</form>";

	}
	else
	{
		echo "Don't forget to add pictures<br/>";
	}
    }
else echo header("location:index.php"); 
}
?>
</div>


<?php
// deletefile.php
session_start();
include "db_connect.php";

$profile_id = $_SESSION['id'];

if(!$profile_id) 
{	
echo "<script language=\"Javascript\" type=\"text/javascript\">
alert(\"You are not logged in!\");document.location.href='index.php';
</script>";
}
else
{            
foreach($_POST['files'] as $num => $id)
{
	//delete reference in database
	@mysql_query("DELETE FROM user_photos WHERE profile_id='$profile_id' AND reference='$id'"); 
	unlink($_SESSION['username']."/pics/".$id); //delete pic in directory
	unlink($_SESSION['username']."/pics/thumbs/".$id); //delete thumbnail
}
}
?>

I hope you understand what i'm trying to do.

Kind regards,

Martijn

The_Dude_1978 · August 8, 2011

Maybe it's more clear this way:

What I want to do is to add the code:

foreach($_POST['files'] as $num => $id)
{
	//delete reference in database
	@mysql_query("DELETE FROM user_photos WHERE profile_id='$profile_id' AND reference='$id'"); 
	unlink($_SESSION['username']."/pics/".$id); //delete pic in directory
	unlink($_SESSION['username']."/pics/thumbs/".$id); //delete thumbnail
}

to my admin page but not with the $_SESSION, but with $_GET or $row variable

But i Need to alter the foreach $_POST variable to deletePic ?

And what do I need to do with the method post action field?

WebStyles · August 8, 2011

if your script can't find the path, I expect these two lines are incomplete: (this is just a guess, but makes sense, read on.)

unlink($_SESSION['username']."/pics/".$id); //delete pic in directory
unlink($_SESSION['username']."/pics/thumbs/".$id); //delete thumbnail

in the sense that an id is normally just a number, and a picture also needs to have an extension. I'm guessing they're jpeg files, so if I'm right you would need something like: (added extensions)

unlink($_SESSION['username']."/pics/".$id.".jpg"); //delete pic in directory
unlink($_SESSION['username']."/pics/thumbs/".$id.".jpg"); //delete thumbnail

The_Dude_1978 · August 9, 2011

Hi,

No, that's not the problem. When the $_SESSION in deletefile.php is set it deletes the picture of the administrator session, because the admin has been granted the session.

What i've tried was:

foreach($_POST['files'] as $num => $id)
{
	//delete reference in database
	@mysql_query("DELETE FROM user_photos WHERE profile_id='$profile_id' AND reference='$id'"); 
	unlink($_GET['username']."/pics/".$id); //delete pic in directory
	unlink($_GET['username']."/pics/thumbs/".$id); //delete thumbnail
}

But it then say's the picture path can't be found. The add picture script makes a folder from the username and that's the one that seems to only be found if the $_SESSION is used.

That's my guess, because the path is there. The reference has the picture and extension.

I've also tried it with the form page to POST the usernames variable and with $_GET POST in deletefile.php but it just does'nt get where to get the username from.

The other thing i've tried was:

foreach($_POST['files'] as $num => $id)
{
	//delete reference in database
	@mysql_query("DELETE FROM user_photos WHERE profile_id='$profile_id' AND reference='$id'"); 
	unlink($row['username']."/pics/".$id); //delete pic in directory
	unlink($row['username']."/pics/thumbs/".$id); //delete thumbnail
}

But then again, the variable is'nt getting to the deletefile.php

With that i'm stuck, so i though if you could help me with the foreach to get it in (or maybe after) the form without causing a invalid argument foreach error, I would be very glad.

Any solution i'm happy with.

Kind regards,

Martijn

The_Dude_1978 · August 9, 2011

I've got the error out of the foreach (see code below)

But now i'm having problems with pushing the delete button. It's not executing the deleting of pictures, but only get's to the Invalid ID provided in the first part of the code.

Do I have it in the wrong order? Is the $post action wrong?

Please do help me out!

//action: delete pictures -----------------------------------------------------------------------------
if (isset($_GET['delpic']) && isset($_GET['id'])) {
$id = (int) $_GET['id'];
if ($id == 0) {
	die("Invalid ID provided.");
}

if($_GET['id'])
{

$sql="SELECT `id`, `first`, `last`, `username`, `email`, `about`, `level` from `users` WHERE `id`='".$_GET['id']."'";
$res=mysql_query($sql);
$row=mysql_fetch_assoc($res);

if (!isset($_POST['deletePic']))
{
?>
<html>
<head>
</head>

<body>

<!--------------------DELETE PICTURES---------------------//-->
    
<div class="dividerp">

<?php

$sql2 = "SELECT `profile_id`, `title`, `size`, `type`, `reference` FROM user_photos WHERE profile_id=".$_GET['id']."";
$res2 = mysql_query($sql2) or die(mysql_error());
            
if(mysql_num_rows($res2) > 0)
	{

	echo "<strong>Delete Pictures for account: ".$row['username']." </strong><br/><br/>";
	echo "<form name=\"deletefile\" method=\"post\" action=\"administrator.php?delpic&id=".$id."\">";

	while($file = mysql_fetch_array($res2))
		{
			echo "<br/><br/><input name=\"deletePic[]\" type=\"checkbox\" value=\"".$file['reference']."\">";
			echo "<a href=\"".$row['username']."/pics/".$file['reference']."\"/>
			<img src=\"".$row['username']."/pics/thumbs/".$file['reference']."\"/></a>";
		}
	echo "<br/><br/><input type=\"submit\" name=\"deletePic\" value=\"Delete Files\">";

	echo "</form>";

if(is_array($_POST['deletePic']))
foreach($_POST['deletePic'] as $num => $id)
{
	//delete reference in database
	@mysql_query("DELETE FROM user_photos WHERE profile_id='$profile_id' AND reference='$id'"); 
	unlink($row['username']."/pics/".$id); //delete pic in directory
	unlink($row['username']."/pics/thumbs/".$id); //delete thumbnail
}	

	}
	else
	{
		echo "Vergeet niet om foto's toe te voegen<br/>";
	}
    }
else echo header("location:index.php"); 
}
?>


</div>

<?php
}

The_Dude_1978 · August 9, 2011

Because of a totally different problem starting new thread in:

http://www.phpfreaks.com/forums/index.php?topic=341064.0

Sign In

How to alter the code so it's in one page delete-pictures or redirect via $_GET

Recommended Posts

The_Dude_1978

Link to comment

Share on other sites

The_Dude_1978

Link to comment

Share on other sites

WebStyles

Link to comment

Share on other sites

The_Dude_1978

Link to comment

Share on other sites

The_Dude_1978

Link to comment

Share on other sites

The_Dude_1978

Link to comment

Share on other sites

Join the conversation

Browse

Activity

Important Information