Bug #55375 [Com]: Inherited constructor signature: new behavior of PHP5.4 or bug?
| From: | dagguh at gmail dot com | Date: | Fri, 26 Oct 2012 16:29:26 +0000 |
| Subject: | Bug #55375 [Com]: Inherited constructor signature: new behavior of PHP5.4 or bug? | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-174350@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=55375&edit=1
ID: 55375
Comment by: dagguh at gmail dot com
Reported by: mikaelkael@php.net
Summary: Inherited constructor signature: new behavior of
PHP5.4 or bug?
Status: Open
Type: Bug
Package: Class/Object related
Operating System: Debian Squeeze
PHP Version: 5.4.0alpha3
Block user comment: N
Private report: N
New Comment:
Abstract constructors? What are you smoking? WONTFIX
Previous Comments:
------------------------------------------------------------------------
[2011-08-06 15:58:36] perraud dot mickael at orange dot fr
Perfect, thanks!
------------------------------------------------------------------------
[2011-08-06 15:52:07] felipe@php.net
In fact the behavior has been changed in 5.4, related to bug #51421.
I'll add the change to UPGRADING file, thanks.
------------------------------------------------------------------------
[2011-08-06 15:51:26] pajoye@php.net
It sounds to me like a bug fix. As the child declaration is more restrictive than
the parent declaration, and is against any good design or OO convention.
------------------------------------------------------------------------
[2011-08-06 11:02:30] mikaelkael@php.net
Description:
------------
I just experiment this behavior with PHP5.4a3 when running the following code:
<?php
abstract class Zend_Pdf_FileParserDataSource
{
abstract public function __construct();
}
class Zend_Pdf_FileParserDataSource_File extends Zend_Pdf_FileParserDataSource
{
public function __construct($filePath)
{}
}
$test = new Zend_Pdf_FileParserDataSource_File('path');
// Throw a Fatal Error:
/*
PHP Fatal error: Declaration of Zend_Pdf_FileParserDataSource_File::__construct() must be
compatible with that of Zend_Pdf_FileParserDataSource::__construct() in /home/mikaelkael/php54.php
on line 11
Fatal error: Declaration of Zend_Pdf_FileParserDataSource_File::__construct() must be compatible
with that of Zend_Pdf_FileParserDataSource::__construct() in /home/mikaelkael/php54.php on line 11
*/
It throws a fatal error (like a "normal" class method in 5.3). But this code was OK for
the constructor in 5.3. In case of new behavior, is it possible to document it? I can't find
any informations in NEWS or in UPGRADE files.
Test script:
---------------
<?php
abstract class Zend_Pdf_FileParserDataSource
{
abstract public function __construct();
}
class Zend_Pdf_FileParserDataSource_File extends Zend_Pdf_FileParserDataSource
{
public function __construct($filePath)
{}
}
$test = new Zend_Pdf_FileParserDataSource_File('path');
// Throw a Fatal Error:
/*
PHP Fatal error: Declaration of Zend_Pdf_FileParserDataSource_File::__construct() must be compatible
with that of Zend_Pdf_FileParserDataSource::__construct() in /home/mikaelkael/php54.php on line 11
Fatal error: Declaration of Zend_Pdf_FileParserDataSource_File::__construct() must be compatible
with that of Zend_Pdf_FileParserDataSource::__construct() in /home/mikaelkael/php54.php on line 11
*/
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=55375&edit=1