[ben_johnson1991]

Hey,
Part of my account deletion, I obviously want all items related to that user to be removed.
This includes from all the tables and and folders and images.

this while loop is supposed to delete all 'event' folders they manage and including the photo inside of it. and then obviously being deleted from the member table and event table, where there are generaly 3 link events to one member.


This is my current script.
All important parts are in bold

PHP Code:

if(isset($_POST['delete'])){

 $del_pass = $_POST['del_pass'];

 $del_pass = md5($del_pass);

$mem_id = $_SESSION['id'];

 $get_pass = mysql_query("SELECT * FROM members WHERE mem_id='$mem_id' AND password='$del_pass'");

////////////

 $pass_num = mysql_num_rows($get_pass);

  if($pass_num == 1){

   $mem_id = $_SESSION['id'];

     

      $event_get = mysql_query("SELECT * FROM events WHERE mem_id='$mem_id'");

     if(mysql_num_rows($event_get) > 0){

      while($delrow = mysql_fetch_array($event_get)){

       $event_id = $delrow['event_id'];

      $eventfile .= 'events/$event_id/image01.jpg';

     if (file_exists($eventfile)) {

     unlink($eventfile);

     rmdir($events/$event_id);

      }

      }

     $sql_del_event = mysql_query ("DELETE FROM event WHERE mem_id='$mem_id'");

     }

  }else{

  $errorMsg = '<span style="color:#F00; font-weight:bold">Delete Error:</span> Please check errors in the Password Section.';

  $delError1 = '* <span style="color:#F00; font-weight:bold">Password</span> was not Corrent<br />';

  }

} 

I rarely use while loops apart from recieve most data from a table.

But what this is supposed to do is check through the event table and find the event_id that the member owns, and on every one that it owns, delete the image (image01.jpg in that 1 folder (events/$event_id) and then delete the folder and finally deletes all the entries from that mmber.

I hope this makes sense?
and I know it's not that hard, but I can't set it out, and a the moment, when this is submitted, nothing is deleted at all.

Thanks for any help!!
Ben.

[wirehopper]

$eventfile .= 'events/$event_id/image01.jpg';

should be

$eventfile .= "events/$event_id/image01.jpg";

The double quotes tell PHP to evaluate any variables inside the string, single quotes use the string literally.

[Jcbones]

It shouldn't have the (.) either.

$eventfile = "events/$event_id/image01.jpg";

Because if it did, you would be looking for files that looked like.

events/$event_id/image01.jpgevents/$event_id/image01.jpgevents/$event_id/image01.jpgevents/$event_id/image01.jpgevents/$event_id/image01.jpg

[wirehopper]

You can use echo to display stuff while you're working on it - makes it easier to see what's happening.