1. Alain Rivest
  2. Bugs
  3. Monday, 25 June 2012
  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 Multilingual Use & Constants
Hello Yves, Thanks for the Matukio 7.0.10 latest release. I am testing the co...
7 Replies
Posted on Tuesday, 26 June 2018
Matukio Filter Geolocation not work
Hello, I have a problem with Matukio Filter Geolocation. When I type in the na...
8 Replies
Posted on Friday, 13 July 2018
  • New
  • GDPR
    Hi, does CComment Pro comply with the GDPR? Kind Regards Holger...
    1 Replies
    Posted on Monday, 16 July 2018
  • New
    • #GDPR
    Can't upgrade to Pro version using cinstaller
    Hi, I tried out the core version of Hotspots and liked it, so I purchased a su...
    1 Replies
    Posted on Sunday, 15 July 2018
  • New
  • Edit Event Frontend Dates
    Hello - When editing an event from the frontend, I have noticed that the start a...
    51 Replies
    Posted on Monday, 12 March 2018