php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #55300 \DirectoryIterator, parent::__construct() and \LogicException
Submitted: 2011-07-27 15:36 UTC Modified: 2011-10-29 23:18 UTC
Votes:12
Avg. Score:4.5 ± 0.8
Reproduced:10 of 10 (100.0%)
Same Version:10 (100.0%)
Same OS:8 (80.0%)
From: frederic dot hardy at mageekbox dot net Assigned: cataphract (profile)
Status: Closed Package: SPL related
PHP Version: 5.4.0alpha2 OS: Linux
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: frederic dot hardy at mageekbox dot net
New email:
PHP Version: OS:

 

 [2011-07-27 15:36 UTC] frederic dot hardy at mageekbox dot net 
Description:
------------
In PHP 5.4, an exception was throwed if a call to parent::__construct() is not done in the constructor of a class which extends \DirectoryIterator.
Exception was not throwed in previous version of PHP, so it's a BC break.
And this BC break must be annoying in unit test, because it's impossible to mock the \DirectoryIterator class.

Test script:
---------------
<?php

class myDirectoryIterator extends \DirectoryIterator
{
	public function __construct()
	{
	}
}

$directoryIterator = new myDirectoryIterator();

?>

Expected result:
----------------
Nothing.

Actual result:
--------------
Fatal error: Uncaught exception 'LogicException' with message 'In the constructor of myDirectoryIterator, parent::__construct() must be called and its exceptions cannot be cleared' in /path/to/directoryIterator.php:10
Stack trace:
#0 /path/to/directoryIterator.php(10): myDirectoryIterator->internal_construction_wrapper()
#1 {main}
  thrown in /path/to/directoryIterator.php on line 10

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2011-07-27 23:30 UTC] cataphract@php.net 
If you don't call the parent constructor, the class has no functionality. Why would you want to do this?
 [2011-07-28 07:27 UTC] frederic dot hardy at mageekbox dot net 
This bug seems to be related to #54384.
And i want to do this to mock a \DirectoryIterator instance, to "simulate" a directory on the file system.
 [2011-07-28 08:29 UTC] cataphract@php.net 
Why can't you use a user stream wrapper then? Maybe DirectoryIterator should be an interface, but it isn't.
 [2011-07-28 08:44 UTC] pajoye@php.net 
Thing is that it is a bc break, whether it was well designed in the 1st place or 
not is sadly not relevant here.

Is there any way to restore this behavior without bc breaks?
 [2011-08-04 06:56 UTC] frederic dot hardy at mageekbox dot net 
This BC break is a problem with the symfony 2 framework.
All command throw this exception :

 [LogicException]                                                                                                                    
  In the constructor of Symfony\Component\Finder\SplFileInfo, parent::__construct() must be called and its exceptions cannot be cleared
 [2011-08-06 01:09 UTC] felipe@php.net
-Status: Open +Status: Assigned -Assigned To: +Assigned To: cataphract
 [2011-08-09 08:44 UTC] seld@php.net 
The main issue is not that it's a BC break I'd say, it's that apparently adding the call to parent::__construct does not fix the issue, the exception is always thrown if you try to have your own constructor in the extending class.

BTW: This is a duplicate of https://bugs.php.net/bug.php?id=55175 IMO.
 [2011-08-09 08:46 UTC] seld@php.net 
Actually according to my test case in https://bugs.php.net/bug.php?id=55175 - it's even happening if you don't redefine the constructor at all. It prevents extending the class altogether, which makes no sense to me.
 [2011-08-09 08:52 UTC] cataphract@php.net 
I wouldn't consider these duplicates. #55175 is an actual problem, this is simply an intended BC break that turned out to have a larger impact than I expected. I'll take care of both not after the beginning of next week.
 [2011-09-02 20:03 UTC] krossekrabbe at gmail dot com 
No, it's not only a BC break. It's exactly the same, it is not possible to extend 
the class. Not when calling parent::__construct(), nor when not overriding 
__construct() at all.
 [2011-09-14 15:38 UTC] frederic dot hardy at mageekbox dot net 
\Phar has the same problem, because this class extends \recursiveDirectoryIterator.
And stream wrapper can not be used with it because "Phar objects can only be created from local files".
 [2011-10-29 23:17 UTC] cataphract@php.net 
Automatic comment from SVN on behalf of cataphract
Revision: http://svn.php.net/viewvc/?view=revision&amp;revision=318566
Log: - Reverted changes that required constructor overrides to invoke the parent
  constructor in several SPL classes and applied 5.3 fixes instead.
  Related bugs: #54384, #55175 and #55300
 [2011-10-29 23:18 UTC] cataphract@php.net 
Replaced with 5.3 code.
 [2011-10-29 23:18 UTC] cataphract@php.net
-Status: Assigned +Status: Closed
 
PHP Copyright © 2001-2025 The PHP Group
All rights reserved. 		Last updated: Mon Apr 28 18:01:30 2025 UTC