Doc #55375 [Opn->Csd]: Inherited constructor signature: new behavior of PHP5.4 or bug?

From: Date: Sun, 30 Dec 2018 16:55:37 +0000
Subject: Doc #55375 [Opn->Csd]: Inherited constructor signature: new behavior of PHP5.4 or bug?
References: 1  Groups: php.doc.bugs 
Request: Send a blank email to doc-bugs+get-16259@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 Updated by: girgias@php.net Reported by: mikaelkael@php.net Summary: Inherited constructor signature: new behavior of PHP5.4 or bug? -Status: Open +Status: Closed Type: Documentation Problem Package: Class/Object related -Operating System: Debian Squeeze +Operating System: N/A -PHP Version: 5.4.0alpha3 +PHP Version: >= 5.4.0 -Assigned To: +Assigned To: girgias Block user comment: N Private report: N New Comment: Seems more like bad code design than a documentation error. Previous Comments: ------------------------------------------------------------------------ [2012-10-26 16:29:26] dagguh at gmail dot com Abstract constructors? What are you smoking? WONTFIX ------------------------------------------------------------------------ [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

« previous php.doc.bugs (#16259) next »