Invalid argument supplied for foreach()

Beitrag von **Dungeonwatcher** » Di 13.Jan, 2009 11:40

Moin!

Gerade sehe ich in meiner phperror.log diese PHP Warnung, welche beim löschen von ungenutzten Bildern über das ACP auftritt:

PHP Warning: Invalid argument supplied for foreach() in ...\Forum\admin\admin_albumpic_delete.php on line 55

Das betrifft Zeile 4 in diesem Block:

Code: Alles auswählen

// Delete files START
if($_POST[submit_update1])
{
	foreach($_POST[check] as $file)
	{
		$base_dir = $album_root_path . "upload/";
		$path = $base_dir;
		unlink($path.$file);
	}
}
// Delete files END

Wie müsste das korrekt auusehen?

Bye

Beitrag von **oxpus** » Di 13.Jan, 2009 18:59

Wie müsste das korrekt auusehen?

Keine Ahnung. Ich weiß ja nicht, wie und wo $_POST[check] gefüllt werden soll.

Beitrag von **Dungeonwatcher** » Di 13.Jan, 2009 19:09

Hi!

Das ganze gehört zu einer Albumerweiterung. Als Autor sind darin Max, Oxpus und SuesseMaus28884 genannt.

Die kplt. admin_albumpic_delete.php sieht so aus.

Beitrag von **oxpus** » Di 13.Jan, 2009 22:10

?
An dieses Script kann ich mich überhaupt nicht mehr erinnern, zumal ich es selber hier auch installiert habe

Nun denn:
Hattest Du überhaupt ein Bild zum Löschen ausgewählt?
Denn ohne dieses kommt auch bei mir die Fehlermeldung...

Mit Auswahl gehts aber fehlerfrei.

Man könnte jetzt aber einen kleinen "Schutz" für den Fehler einbauen.
Einfach in dieser Datei die Zeile

Code: Alles auswählen

if($_POST[submit_update1])

gegen diese tauschen:

Code: Alles auswählen

if($_POST[submit_update1] && isset($_POST['check']))

Beitrag von **Dungeonwatcher** » Mi 14.Jan, 2009 04:22

Moin!

[quote="oxpus";p="88108"]An dieses Script kann ich mich überhaupt nicht mehr erinnern, zumal ich es selber hier auch installiert habe

[/quote]

Ist ja auch schon ein paar Jährchen her.

Nun denn:
Hattest Du überhaupt ein Bild zum Löschen ausgewählt?
Denn ohne dieses kommt auch bei mir die Fehlermeldung...

Schnell nochmal getestet...

Mit Auswahl gehts aber fehlerfrei.

Stimmt genau.

Man könnte jetzt aber einen kleinen "Schutz" für den Fehler einbauen.

Klappt bestens, THNX

Bye