[php][$_GET] multiple rows delete Opcje

[hettmix]

Witam.

Tym razem mam taki oto problemik. Przekazuję metodą $_GET nr id wierszy tabeli do skryptu pdf_del_all.php:

[PHP] pobierz, plaintext 
function arrayToString($array){
		$i = 0;
		$string = '';
 
		foreach ($array as $index => $value ){
		if($i != count($array)-1){
			$string .= "$value,";
		}else $string .= "$value";
		$i++;
		}
 
		$string = '['.$string.']';
		return  $string;
		}
 
while ($z = mysql_fetch_assoc($wynik)) {
 
			$ids[$z['id']] = $z['id'];
			$pdfs = arrayToString($ids, 1);
 
                        echo "<table>";
			echo "<tr style=\"font-size: 0.9em; font-weight: normal; text-align: right; resize: none;\" bgcolor=$bgcolor>";
			echo "<td colspan=\"2\"><a id=\"confirm_button\" style=\"text-decoration: none; text-align: right;\" href='pdf_del_all.php?id_pdfs=$pdfs' onclick=\"return confirmDeleteAll(this);\"><img src='/modules/mod_zbiorczy_pdf/images/x.png' border=0>wszystkie</a></td>";
			echo "<td colspan=\"4\">Wszystkich plików:&nbsp;<b>$i<b>&nbsp;&nbsp;</td>";
			echo "</tr>";		
			echo "</table>";
}
[PHP] pobierz, plaintext 

Teraz skrytp pdf_del_all.php powinien usunąć z bazy wskazane wiersze oraz pliki z serwera powiązane przez zapisaną ścieżkę:

[PHP] pobierz, plaintext 
if (isset($_GET['id_pdfs'])) {
 
			$delete_id = explode(",", $_GET['id_pdfs']);
			$id = count($delete_id);
 
			if (count($id) > 0)
			{
				foreach ($delete_id as $id_d)
					{
						$sql = "delete FROM tbl_zwykle_2str_pdf WHERE id='$id_d'";
						$delete = mysql_query($sql) or die('Błąd zapytania delete');
					}
			}
		}
[PHP] pobierz, plaintext 

Puki co skrypt usuwa z bazy wiersze lecz nie wszystkie - pozostaje jeden wiersz mimo że print_r($id_d) pokazuje wszystkie wiersze będące w tabeli.
Nie wiem też jak pobrać z bazy ścieżkę do pliku, a następnie usunąć go z serwera.

[SQL] pobierz, plaintext 
CREATE TABLE IF NOT EXISTS `tbl_zwykle_2str_pdf` (
  `id` smallint(6) UNSIGNED NOT NULL AUTO_INCREMENT,
  `pdf_uzytkownik` varchar(30) COLLATE utf8_polish_ci NOT NULL,
  `pdf_czas_dodania` time NOT NULL,
  `pdf_data_dodania` date NOT NULL,
  `pdf_nazwa_pliku` varchar(200) COLLATE utf8_polish_ci NOT NULL,
  `pdf_typ_pliku` varchar(100) CHARACTER SET utf8 NOT NULL,
  `pdf_rozmiar_pliku` int(9) NOT NULL,
  PRIMARY KEY (`id`),
  KEY `pdf_uzytkownik` (`pdf_uzytkownik`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=4 ;
[SQL] pobierz, plaintext 

Jakaś wskazówka....

[mokry]

[SQL] pobierz, plaintext 
SELECT * FROM tbl_zwykle_2str_pdf WHERE id IN($_GET['id_pdfs']);
[SQL] pobierz, plaintext 

Oczywiście zmienna GET powinna zostać zabezpieczna przez ew. atakiem.

Następnie foreach po pobranych wierszach oraz unlink() wg ścieżki z kolumny `pdf_nazwa_pliku`.

Na końcu:

[SQL] pobierz, plaintext 
DELETE FROM tbl_zwykle_2str_pdf WHERE id IN($_GET['id_pdfs']);
[SQL] pobierz, plaintext 

Edit:
Zamiast stosować jakieś dziwne funkcje do składania i rozkładania string'u do/z tablicy zastosuj implode() i explode().

Ten post edytował mokry 7.01.2013, 20:43:05

[hettmix]

[SQL] pobierz, plaintext 
SELECT * FROM tbl_zwykle_2str_pdf WHERE id IN($_GET['id_pdfs']);
[SQL] pobierz, plaintext 

Oczywiście zmienna GET powinna zostać zabezpieczna przez ew. atakiem.

Następnie foreach po pobranych wierszach oraz unlink() wg ścieżki z kolumny `pdf_nazwa_pliku`.

Na końcu:

[SQL] pobierz, plaintext 
DELETE FROM tbl_zwykle_2str_pdf WHERE id IN($_GET['id_pdfs']);
[SQL] pobierz, plaintext 

Edit:
Zamiast stosować jakieś dziwne funkcje do składania i rozkładania string'u do/z tablicy zastosuj implode() i explode().

No właśnie pętla foreach ładnie wybiera wskazane wiersze, lecz usuwane są wszystkie poza jednym - nie wiem dlaczego...

[mokry]

Zobacz jakie zapytanie do bazy generuje Ci PHP, wyświetl je i sprawdź, czy wszystie id się zgadzają z tymi, które posiadasz w bazie danych.
Możliwe, że gdzieś masz podane złe id i dlatego Ci go nie chwyta.

[hettmix]

Zobacz jakie zapytanie do bazy generuje Ci PHP, wyświetl je i sprawdź, czy wszystie id się zgadzają z tymi, które posiadasz w bazie danych.
Możliwe, że gdzieś masz podane złe id i dlatego Ci go nie chwyta.

OK - podmieniłem funkcję zamieniającą tablicę na string na implode i teraz usuwanie rekordów ładnie działa. Nie mogę sobie poradzić w dalszym ciągu z wybraniem z bazy poszczególnych ścieżek do plików, które później usunę unlink'iem.

[PHP] pobierz, plaintext 
foreach ($delete_id as $id_d)
					{
						mysql_query("delete FROM tbl_zwykle_2str_pdf WHERE id='$id_d'") or die('Błąd zapytania delete');
						$pdf_nazwa_pliku = mysql_query("select * FROM tbl_zwykle_2str_pdf WHERE id in('.implode(',',$delete_id).')") or die('Błąd zapytania select');
						$target_path = "/volume1/web/Joomla_1.5/modules/mod_zbiorczy_pdf/tmp/zwykle_1_strona/";
						$file  = $target_path . $pdf_nazwa_pliku;
						echo $file;
					}
[PHP] pobierz, plaintext 

w wyniku dostaję:
/volume1/web/Joomla_1.5/modules/mod_zbiorczy_pdf/tmp/zwykle_1_strona/Resource id #18
/volume1/web/Joomla_1.5/modules/mod_zbiorczy_pdf/tmp/zwykle_1_strona/Resource id #19
/volume1/web/Joomla_1.5/modules/mod_zbiorczy_pdf/tmp/zwykle_1_strona/Resource id #20

[mokry]

Masz:

[PHP] pobierz, plaintext 
$delete_id = '15,23,59,98,78,26,58';  // Przykladowe id
 
$target_path = "/volume1/web/Joomla_1.5/modules/mod_zbiorczy_pdf/tmp/zwykle_1_strona/";
 
//Zapytanie po rekordy do usuniecia plikow
$result = mysql_query("SELECT * FROM tbl_zwykle_2str_pdf WHERE id IN(".$delete_id.")") or die('Błąd zapytania delete');
while($row = mysql_fetch_assoc($result)) //Iteracja po wynikach z bazy danych
{
    $file = $target_path . $row['pdf_nazwa_pliku']; // Dobranie sie do kolumny aktualnie iterowanego wiersza
    echo $file;
	//unlink($file);  // Tutaj mozesz sobie usunac plik
}
//Usuwasz wszystkie rekordy	
$query = mysql_query("delete FROM tbl_zwykle_2str_pdf WHERE id IN(".$delete_id.")") or die('Błąd zapytania delete');
[PHP] pobierz, plaintext 

[hettmix]

Masz:

[PHP] pobierz, plaintext 
$delete_id = '15,23,59,98,78,26,58';  // Przykladowe id
 
$target_path = "/volume1/web/Joomla_1.5/modules/mod_zbiorczy_pdf/tmp/zwykle_1_strona/";
 
//Zapytanie po rekordy do usuniecia plikow
$result = mysql_query("SELECT * FROM tbl_zwykle_2str_pdf WHERE id IN(".$delete_id.")") or die('Błąd zapytania delete');
while($row = mysql_fetch_assoc($result)) //Iteracja po wynikach z bazy danych
{
    $file = $target_path . $row['pdf_nazwa_pliku']; // Dobranie sie do kolumny aktualnie iterowanego wiersza
    echo $file;
	//unlink($file);  // Tutaj mozesz sobie usunac plik
}
//Usuwasz wszystkie rekordy	
$query = mysql_query("delete FROM tbl_zwykle_2str_pdf WHERE id IN(".$delete_id.")") or die('Błąd zapytania delete');
[PHP] pobierz, plaintext 

No i problem rozwiązany - poniżej sprawdzony, działający kod:

[PHP] pobierz, plaintext 
if (isset($_GET['id_pdfs'])) {
 
			$id_pdfs = trim($_GET['id_pdfs']);
			$delete_id = explode(",", $_GET['id_pdfs']);
			$id = count($delete_id);
 
			if (count($id) > 0)
			{
				$result = mysql_query("SELECT * FROM tbl_zwykle_2str_pdf WHERE id IN(".$id_pdfs.")") or die('Błąd zapytania delete');
				while($row = mysql_fetch_assoc($result))
				{
					$file = $row['pdf_nazwa_pliku'];
					unlink($file); 
				}
					//Usuwasz wszystkie rekordy	
					$query = mysql_query("delete FROM tbl_zwykle_2str_pdf WHERE id IN(".$id_pdfs.")") or die('Błąd zapytania delete');
 
					echo "<BR /><div style=\"background-color: #F5F5F5; height: 45px; padding-top: 5px; text-align: center;\"><b>REKORDY <span style=\"color: blue\">\ ".$id_pdfs." \</span> ZOSTAŁY POPRAWNIE USUNIĘTE Z BAZY.<BR /><b></div>";
					echo "<div style=\"padding-top: 30px; text-align: center;\"><img height=\"50%\" width=\"20%\" align=\"middle\" src=\"images/ok_icon.png\" title=\"\" alt=\"\" /></div><BR />";
			}
		}
[PHP] pobierz, plaintext