kleiner bug in "/includes/functions.general.php"

Gesperrt
herm
Beiträge: 1
Registriert: So 4. Dez 2005, 14:10
Kontaktdaten:

kleiner bug in "contenido/includes/functions.general.ph

Beitrag von herm » So 4. Dez 2005, 14:19

in der "function recursive_copy($from_path, $to_path)" in line 440 der "contenido/includes/functions.general.php" wird die rekursion nach verzeichniswechsel am ende mit leere variable angesprungen, und erzeugt die fehlermeldungen (Warning: copy(/www/htdocs/v119307/kunden_cms/contenido/kunden/max/images): failed to open stream: Ist ein Verzeichnis in /www/htdocs/v119307/kunden_cms/contenido/contenido/includes/functions.general.php on line 462 )

wenn man den code dahingehend ändert (else - schleife) sind die meldungen weg:




while (($myfile = readdir($myhandle)) !== false)
{
if (($myfile != ".") && ($myfile != ".."))
{
if (is_dir($myfile))
{
recursive_copy($from_path.$myfile."/", $to_path.$myfile."/");
chdir($from_path);
}
else
{
if (file_exists($myfile))
{
copy($from_path.$myfile, $to_path.$myfile);
}
}

}
}

emergence
Beiträge: 10641
Registriert: Mo 28. Jul 2003, 12:49
Wohnort: Austria
Kontaktdaten:

Beitrag von emergence » So 4. Dez 2005, 14:44

ich hab einen split of gemacht von
-> http://contenido.org/forum/viewtopic.php?t=10474
*** make your own tools (wishlist :: thx)

rethus
Beiträge: 1851
Registriert: Di 28. Mär 2006, 11:55
Wohnort: Mönchengladbach
Kontaktdaten:

Etwas sauberer....

Beitrag von rethus » Mi 29. Mär 2006, 10:40

PS: Danke für den tollen Tip, hat mir echt weitergeholfen.

Vielleicht etwas übersichtlicher und sauberer könnte man das so machen:

anstatt:
else
{
if (file_exists($myfile))
{
copy($from_path.$myfile, $to_path.$myfile);
}
}

einfach:

elseif (file_exists($myfile))
{
copy($from_path.$myfile, $to_path.$myfile);
}

HerrB
Beiträge: 6935
Registriert: Do 22. Mai 2003, 12:44
Wohnort: Berlin
Kontaktdaten:

Beitrag von HerrB » Mo 12. Jun 2006, 09:15

Done.

Gruß
HerrB
Bitte keine unaufgeforderten PMs oder E-Mails -> use da Forum!

Newsletter: V4.4.x | V4.6.0-15 (Module, Backend) | V4.6.22+
Standardartikelliste: V4.4.x | V4.6.x
http://www.contenido.org/forum/search.php | http://faq.contenido.org | http://www.communido.net

Gesperrt