1. Alain Rivest
  2. Bugs
  3. Monday, 25 June 2012 15:58
  4.  Subscribe via email
Hi Daniel,

I tested JEDchecker with my extensions. When I click unzip, it crashed with the error "PHP Fatal error: Maximum function nesting level of '100' reached, aborting!" With xdebug I found it's stucked in an infinite loop in unzipAll (uploads.php) because it call unzipAll back with the "." directory.

I fixed it with a call to isDot() to bypass the "." and ".." directories. I never used RecursiveDirectoryIterator before, so I'm not sure if my fix has any side effects...

[code type=php]
public function unzipAll($start)
{
$iterator = new RecursiveDirectoryIterator($start);

foreach ($iterator as $file) {
if ($file->isFile()) {
$extension = $file->getExtension();
if ($extension == 'zip') {
$unzip = $file->getPath() . '/' . $file->getBasename('.' . $extension);
$result = JArchive::extract($file->getPathname(), $unzip);
// delete the archive once we extract it
if ($result) {
JFile::delete($file->getPathname());

// now check the new extracted folder for archive files
$this->unzipAll($unzip);
}
}
} else if (!$iterator->isDot()) {
$this->unzipAll($file->getPathname());

}
}
}
[/code]


There are 4 replies to this question. If you want to see them you need a valid subscription.
If you have a valid subscription, please login now.
Visit store now

Categories

General
  1. 3 subcategories
Billing
  1. 4 subcategories
Hotspots
  1. 11 subcategories
CComment
  1. 11 subcategories
Matukio
  1. 6 subcategories
CMigrator
  1. 4 subcategories
CForms
  1. 2 subcategories
International Zone
  1. 10 subcategories
Free extensions
  1. 47 subcategories
Archive
  1. 15 subcategories

Last questions

Matukio 7 versendet Stornierungsmails obwohl keine
2 Replies
Posted on Monday, 16 October 2017 21:08
COM_MATUKIO_SAVING_RECURRING_FAILED
0 Replies
Posted on Friday, 20 October 2017 20:07
Change / remove default form fields
0 Replies
Posted on Friday, 20 October 2017 19:58
Error 500 Bootstrap
0 Replies
Posted on Friday, 20 October 2017 19:47
Nach Update auf Matukio 7 geht nichts mehr
16 Replies
Posted on Thursday, 05 October 2017 13:49