Jump to content

Recommended Posts

I have this page that allows me to delete a picture from a folder. When i press delete the error message

 

Error deleting photo!

Please try again.

 

This is an echo message from my code. I dont know what to change to get the file to delete. Heres the code below

<?php
include_once("config.php");
// No album id has been selected
if( !$_GET['photo_id'] ){
// Display error message to user
$msg .= "Photo not selected. Please choose a photo you wish to delete!";
$msg .= "<br /><a href=\"edit_photos.php\">Edit photos</a>";
displayPage($msg, "Error Selecting Photo");
} else {
con;
// Retrieve image and thumbnail path information
$sql = "SELECT photo_location, thumbnail_location FROM photos WHERE photo_id = " . addslashes($_GET['photo_id']);
$result = @mysql_query($sql) or die("Error retrieving records: " . mysql_error());
while ($row = mysql_fetch_array($result)){ 
$photo_location = $row['photo_location'];
$thumb_location = $row['thumbnail_location'];
}
if (@unlink("../photos/" . $photo_location)){
} else {
die("Error deleting photo!<br /><a href=\"index.php\">Please try again</a>.");
}
if (@unlink("../thumbs/" . $thumb_location)){ 
} else {
die("Error deleting thumbnail!<br /><a href=\"index.php\">Please try again</a>.");
} 
// Delete specified album 
$sql = "DELETE FROM photos WHERE photo_id = " . addslashes($_GET['photo_id']);
$result = @mysql_query($sql) or die("Error deleting record: " . mysql_error());
// Display success to user
$msg .= "Photo has been successfully deleted!<br /><a href='index.html'>Return to Admin Menu</a>";
displayPage($msg, "Photo Deleted!");
}
?>

 

Sorry about the layout, im using a works computer without an editor.

It doesnt like this part of the code

 

 

if (@unlink("../" . $photo_location)){

} else {

die("Error deleting photo!<br /><a href=\"index.php\">Please try again</a>.");

}

 

 

In other words, it cant delete ../$photo_location

 

try adding echo $photo_location; to your script to make sure its a valid image name

 

Also, is the image dir a level below the folder your script is in, hence the ../

This thread is more than a year old. Please don't revive it unless you have something important to add.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.