[PHP] usuwanie pliku po kliknięciu w link Opcje

[felixk]

Witam, próbuję napisać skrypt usuwający plik po kliknięciu w link. Udało mi się napisać coś takiego

[PHP] pobierz, plaintext 
<?
    $dir = "../data/files/Pliki";
    $folder = dir($dir);
 
    while($plik = $folder->read()) {
 
    if (($plik != '.') AND ($plik != '..')) {
 
    echo '<li><a href="index.php?plik='.unlink($dir.'/'.$_GET['plik']).'">usuń</a></li>'; //line 110
    }
    }
    $folder->close();
?>
[PHP] pobierz, plaintext 

Listowanie może zbędne, ale o dziwo ono działa tzn. pliki ze zmiennej $dir wyświetlają się, ale już przy linku otrzymuję poniży komunikat

Kod

Warning: unlink(../data/files/Pliki/) [function.unlink]: Jest katalogiem in index.php on line 110

Jak zrobić aby to usuwanie działało prawidłowo?

Z góry dziękuję za pomoc

[rocktech.pl]

Witam.

W poniższej linijce wykonujesz instrukcję echo. Oraz funkcję unlink.

[PHP] pobierz, plaintext 
  echo '<li><a href="index.php?plik='.unlink($dir.'/'.$_GET['plik']).'">usuń</a></li>'; //line 110
[PHP] pobierz, plaintext 

Aby utworzyć listing.

[PHP] pobierz, plaintext 
foreach (glob($dir."/*") as $plik) {
      echo '<li><a href="index.php?plik='.$plik.'">usuń</a></li>'; 
}
[PHP] pobierz, plaintext 

I dopiero potem odpowiednio przetwórz $_GET['plik'].

[PHP] pobierz, plaintext 
$dir = "/SCIEZKA_BEZWGLEDNA/data/files/Pliki";
//zwróć uwagę na potencjalne zagrożenia wynikające z poniższego kodu funkcję realpath i basename ...
if (isset($_GET['plik'])) {
    if (file_exist($dir.$_GET['plik']))
        unlink ($dir.$GET['plik']);
    }
}
[PHP] pobierz, plaintext 

Ten post edytował rocktech.pl 23.09.2012, 14:35:43

[felixk]

Witam.

W poniższej linijce wykonujesz instrukcję echo. Oraz funkcję unlink.

[PHP] pobierz, plaintext 
  echo '<li><a href="index.php?plik='.unlink($dir.'/'.$_GET['plik']).'">usuń</a></li>'; //line 110
[PHP] pobierz, plaintext 

Aby utworzyć listing.

[PHP] pobierz, plaintext 
foreach (glob($dir"/*") as $plik) {
      echo '<li><a href="index.php?plik='.$plik.'">usuń</a></li>'; 
}
[PHP] pobierz, plaintext 

I dopiero potem odpowiednio przetwórz $_GET['plik'].

[PHP] pobierz, plaintext 
$dir = "/SCIEZKA_BEZWGLEDNA/data/files/Pliki";
//zwróć uwagę na potencjalne zagrożenia wynikające z poniższego kodu funkcję realpath i basename ...
if (isset($_GET['plik'])) {
    if (file_exist($dir.$_GET['plik']))
        unlink ($dir.$GET['plik']);
    }
}
[PHP] pobierz, plaintext 

Twój kod wyrzuca mi
Parse error: syntax error, unexpected T_CONSTANT_ENCAPSED_STRING w linii 105, czyli w foreach (IMG:style_emoticons/default/smile.gif)

Ten post edytował felixk 23.09.2012, 14:30:56

[rocktech.pl]

Zgadza się poprawiłem. Pisane z palca.