Bug #55701 [Com]: GlobIterator throws LogicException with message 'The parent constructor was not

From: Date: Mon, 08 Feb 2016 10:30:53 +0000
Subject: Bug #55701 [Com]: GlobIterator throws LogicException with message 'The parent constructor was not
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-199107@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=55701&edit=1 ID: 55701 Comment by: axiac dot ro at gmail dot com Reported by: bs@php.net Summary: GlobIterator throws LogicException with message 'The parent constructor was not Status: Assigned Type: Bug Package: SPL related Operating System: Linux, OSX PHP Version: 5.3.8 Assigned To: cataphract Block user comment: N Private report: N New Comment: Tested on PHP 5.3, 5.4, 5.5, 5.6 and 7.0 on OSX and Ubuntu. It fails the same on all of them. <?php error_reporting(E_ALL); ini_set('display_errors', '1'); $g = new \GlobIterator('/tmp/*.txt'); echo('Before: '.$g->count()."\n"); foreach ($g as $f) { } echo('After : '.$g->count()."\n"); If the glob expression matches at least one file it throws the exception on the second call to count() (line 9). If it doesn't match then it throws on the first call to count() (line 6). Not only count() throws the exception. All its methods do the same. Previous Comments: ------------------------------------------------------------------------ [2016-01-11 16:24:08] pheagey at gmail dot com Able to reproduce error in 5.5.6 on Ubuuntu 14.04 $iterator = new GlobIterator('./someDir/*'); $iterator->count(); Where 'someDir' has not contents. ------------------------------------------------------------------------ [2014-08-10 10:57:12] hanskrentel at yahoo dot de Additionally I can not reproduce the original report: ----------------------------------------------------- <?php $g = new \GlobIterator(__DIR__ . '/*'); do { $g->next(); } while($g->valid()); ---------------------------------------------------- Works flawlessly for me. Using 5.5.15 ------------------------------------------------------------------------ [2014-08-10 10:53:44] hanskrentel at yahoo dot de SplFileObject checks on method invocation that the state is sane - it does not care whether or not you have it extended. Whether or not this qualifies as a bug (I tend to say it isn't a flaw, because SplFileObject has it's responsibilities and if you extend from it, you're explicitly stating that your object comes with the same responsibilities, you can not lower the checks). However, if you actually want to stub the file-object first, that must not stand in your way, it's easy to work-around, for example with a fake "data://" URI: <?php /* * File: test.php * Link: https://bugs.php.net/bug.php?id=55701 */ class MyFileObject extends \SplFileObject { public function __construct($file_name) { parent::__construct("data://,"); $new_fname = $this->getRealPath(); parent::__construct($file_name); } public function getRealPath() { return '/tmp/foo.txt'; } } $f1 = new MyFileObject(__FILE__); var_dump($f1->getBasename()); # string(8) "test.php" ------------------------------------------------------------------------ [2014-02-26 14:11:20] log dot kot at gmail dot com Confirm the bug Ubuntu Linux 13.10 3.11.0-17-generic x86_64 PHP 5.5.3-1ubuntu2.1 (cli) (built: Dec 12 2013 04:24:35) Copyright (c) 1997-2013 The PHP Group Zend Engine v2.5.0, Copyright (c) 1998-2013 Zend Technologies with Zend OPcache v7.0.3-dev, Copyright (c) 1999-2013, by Zend Technologies with Xdebug v2.2.3, Copyright (c) 2002-2013, by Derick Rethans ------------------------------------------------------------------------ [2013-08-02 11:54:52] rosier at interstroom dot nl I can confirm this unexpected result also for php54 and php55 Test script: --------------- <?php $path_to_files = sys_get_temp_dir(); // Next works as expected: no xml files found = no output foreach (new GlobIterator($path_to_files . '/*.xml') as $fileinfo) { echo $fileinfo->getFilename() . "\n"; } $it = new GlobIterator($path_to_files . '/*.xml'); // Expected result: count = 0 // Instead next line will crash php if no xml files are found if ($it->count()) { // do something... } ?> ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=55701 -- Edit this bug report at https://bugs.php.net/bug.php?id=55701&edit=1

« previous php.bugs (#199107) next »