.JPG files binnen 1 dag deleted

Door Perry S op 17-08-2017 10:54

7.254 views

Goedemorgen,

Ik heb gisteren een php gevonden alleen hij werkt niet echt, het is de bedoeling dat hij binnen een dag de eerst oude jpg files van de vorige dag automatisch verwijderd. Dit wordt gebruikt voor een automatische foto slider. Ik heb er een hekel aan om elke keer de map leeg te maken.

Kunnen jullie mij vertellen wat er verkeerd is aan deze file?

Code://

<?php
$expiretime=1440; //Verloopt na minuten, 1 dag = 1x 24x60= 1440

$tmpFolder="/home/vhosts/(Domein)/subdomains/(Subdomein)/httpdocs/wp-content/uploads/uploaden/uploads/";
$fileTypes=".jpg";

foreach (glob($tmpFolder . $fileTypes) as $Filename) {

$FileCreationTime = filectime($Filename);

$FileAge = time() - $FileCreationTime;

if ($FileAge > ($expiretime * 60)){

echo "The file $Filename is older than $expiretime minutes\n";

//delete bestanden
unlink($Filename);
}

}
?>

E vH

31-08-2017 11:33 gewijzigd op 31-08-2017 11:34

Probeer eens (LET OP, dit betreft de code uit je beginpost! ):


<?php
$expiretime=1440; //Verloopt na minuten, 1 dag = 1x 24x60= 1440

$tmpFolder="/home/vhosts/(Domein)/subdomains/(Subdomein)/httpdocs/wp-content/uploads/uploaden/uploads/";
$fileTypes="*.jpg"; // * toegevoegd

foreach (glob($tmpFolder . $fileTypes) as $Filename) {

$FileCreationTime = filemtime($Filename); // filemtime gebruikt ipv filectime

$FileAge = time() - $FileCreationTime;

if ($FileAge > ($expiretime * 60)){

echo "The file $Filename is older than $expiretime minutes\n";

//delete bestanden
unlink($Filename);
}

}
?>

Perry S

31-08-2017 20:01

Elmar vH op 31/08/2017 11:33:22

Probeer eens (LET OP, dit betreft de code uit je beginpost! ):


<?php
$expiretime=1440; //Verloopt na minuten, 1 dag = 1x 24x60= 1440

$tmpFolder="/home/vhosts/(Domein)/subdomains/(Subdomein)/httpdocs/wp-content/uploads/uploaden/uploads/";
$fileTypes="*.jpg"; // * toegevoegd

foreach (glob($tmpFolder . $fileTypes) as $Filename) {

$FileCreationTime = filemtime($Filename); // filemtime gebruikt ipv filectime

$FileAge = time() - $FileCreationTime;

if ($FileAge > ($expiretime * 60)){

echo "The file $Filename is older than $expiretime minutes\n";

//delete bestanden
unlink($Filename);
}

}
?>

Ik denk "glob($tmpFolder.$fileTypes)"

.$fileTypes moet er uit anders heb je achter mijn domeinpad .jpg staan ????

E vH

31-08-2017 20:16

Kijk eens goed.

Weet je nog waar het sterretje voor stond? (Lees anders even terug)

Perry S

31-08-2017 20:43

Elmar vH op 31/08/2017 20:16:47

Kijk eens goed.

Weet je nog waar het sterretje voor stond? (Lees anders even terug)

(glob($tmpFolder . *$fileTypes) ?

E vH

31-08-2017 20:46

Lees even terug en kijk heel goed wat ik verander (oranje tekst).

Ps: heb je mijn code al geprobeerd?

- Ariën -

31-08-2017 20:47

Ik weet niet wat je zit te rommelen, maar die sterretje staat daar niet goed.
Die hoort in de string van $fileTypes, en niet lukraak ergens in je PHP-code.

Perry S

31-08-2017 22:14

- Ariën - op 31/08/2017 20:47:38

Ik weet niet wat je zit te rommelen, maar die sterretje staat daar niet goed.
Die hoort in de string van $fileTypes, en niet lukraak ergens in je PHP-code.

Ik snap het niet meer..

- Ariën -

31-08-2017 22:25

Debug het anders eens, door de variabelen te echo'en. Dan weet je precies welke waarden er zijn, en kan je de fout zo vinden.

En die * hoort in je string van $fileTypes, en niet voor die variabele.

E vH

01-09-2017 08:19

Even in haken op je uitspraak:

Perry Saarloos op 31/08/2017 20:43:25

(glob($tmpFolder . *$fileTypes) ?

Zie hier: waar ik het * uitleg

Je kreeg van mij de hint om regel 5 en regel 9 aan te passen, daarmee zou je code moeten werken...

Perry S

01-09-2017 08:24 gewijzigd op 01-09-2017 08:24

Ik ga zo weer proberen :)

[size=xsmall]Toevoeging op 01/09/2017 08:59:55:[/size]

Elmar vH op 01/09/2017 08:19:34

Even in haken op je uitspraak:
[quote="Perry Saarloos op 31/08/2017 20:43:25"]
(glob($tmpFolder . *$fileTypes) ?

Zie hier: waar ik het * uitleg

Je kreeg van mij de hint om regel 5 en regel 9 aan te passen, daarmee zou je code moeten werken...
[/quote]

$fileTypes="*.jpg";

[size=xsmall]Toevoeging op 01/09/2017 09:17:49:[/size]

Elmar vH op 01/09/2017 08:19:34

Even in haken op je uitspraak:
[quote="Perry Saarloos op 31/08/2017 20:43:25"]
(glob($tmpFolder . *$fileTypes) ?

Zie hier: waar ik het * uitleg

Je kreeg van mij de hint om regel 5 en regel 9 aan te passen, daarmee zou je code moeten werken...
[/quote]

<?php
$expiretime=86400; // Veranderd

$tmpFolder="/home/vhosts/(Domein)/subdomains/(Subdomein)/httpdocs/wp-content/uploads/uploaden/uploads/";
$fileTypes="*.jpg"; // Veranderd *

foreach (glob($tmpFolder . $fileTypes) as $Filename) {

$FileCreationTime = filemtime($Filename); // Veranderd naar filemtime

$FileAge = time() - $FileCreationTime;

if ($FileCreationTime > ($expiretime * 60)){ //Veranderd

echo "The file $Filename is older than $expiretime minutes\n";

//delete bestanden
unlink($Filename);
}

}
?>

Volgensmij is dit 'm?

Reageren

Inloggen om te reageren