Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:8176
Return-Path: <news@eecoo.de>
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Delivered-To: mailing list internals@lists.php.net
Received: (qmail 34168 invoked by uid 1010); 26 Feb 2004 18:51:22 -0000
Delivered-To: ezmlm-scan-internals@lists.php.net
Delivered-To: ezmlm-internals@lists.php.net
Received: (qmail 34072 invoked by uid 1007); 26 Feb 2004 18:51:22 -0000
Message-ID: <20040226185122.34071.qmail@pb1.pair.com>
To: internals@lists.php.net
Reply-To: news@eecoo.de
Date: Thu, 26 Feb 2004 19:47:25 +0100
Lines: 39
User-Agent: KNode/0.7.2
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7Bit
X-Posted-By: 217.230.89.55
Subject: PHP5: Class behavior: method overloading
From: news@eecoo.de (Vivian Steller)

Hello,

in the latest snapshot i found some very important differences to the php5
beta 4 version concerning the behavior of inherited classes. please look at
the following example:

<pre>
<?php
        class MyClass extends MyParent{         
                function test($arg) {
                
                }
        }       
        
        class MyParent {
                function test($arg1, $arg2) {
                
                }
        }
        
        $obj = new MyClass();
?>
</pre>

the output is:

Fatal error:  Declaration of MyClass::test() must be compatible with that of
MyParent::test() in ... on line 3


I think that this is a nice feature in the face of "compatibility", but may
cause in a complete restructure of existing class trees. The consequence is
a bad :( backward compatibility!! Personally I think it would be a good
solution to set an option in the configuration or something like this.

other oppinions would be appreciated.

thanks,
vivi