Req #30793 [Asn->Fbk]: Abstract functions and default parameter
| From: | stas@php.net | Date: | Sat, 25 Feb 2012 00:35:19 +0000 |
| Subject: | Req #30793 [Asn->Fbk]: Abstract functions and default parameter | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-167807@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=30793&edit=1
ID: 30793
Updated by: stas@php.net
Reported by: jonas at datatal dot se
Summary: Abstract functions and default parameter
-Status: Assigned
+Status: Feedback
Type: Feature/Change Request
Package: Class/Object related
Operating System: *
PHP Version: *
Assigned To: andi
Block user comment: N
Private report: N
New Comment:
Please try using this snapshot:
http://snaps.php.net/php5.3-latest.tar.gz
For Windows:
http://windows.php.net/snapshots/
Seems to be working fine in 5.3.
Previous Comments:
------------------------------------------------------------------------
[2004-11-29 15:57:16] tony2001@php.net
>default parameter is still a parameter
right.
but providing default value for a parameter you're changing the number of required arguments.
------------------------------------------------------------------------
[2004-11-29 15:54:36] jonas at datatal dot se
I can agree on the second problem. But the first one should be valid (default parameter is still a
parameter)
------------------------------------------------------------------------
[2004-11-27 11:40:03] tony2001@php.net
Well, personally I do not think such features could be accepted.
You get these errors because method signatures are different (number of required arguments and class
hint differ), so this is well expected behaviour.
I'm leaving this report as feature request for Andi.
------------------------------------------------------------------------
[2004-11-15 13:59:28] jonas at datatal dot se
Another related problem.
have this function in the same abstract class:
/** Fill the object with the information fetched from the db */
abstract public static function Fill(ADORecordset $rs, DbClass $db);
Since all db classes extends the DbClass this should be valid:
(in the user class)
public static function Fill(ADORecordset $rs, User $usr)
{
[...]
}
but it isnt. I get the following error:
Fatal error: Declaration of User::Fill() must be compatible with that of DbClass::Fill() in
d:\projects\inetpub\WebUser\Classes\User.php on line 83
------------------------------------------------------------------------
[2004-11-15 13:48:28] jonas at datatal dot se
Description:
------------
Don't know if this is a bug or a feature request. Any case, it's a Zend 2 problem..
Reproduce code:
---------------
<?php
abstract class DbClass
{
protected $m_con;
public function __construct(ADOConnection $con)
{
$this->m_con = $con;
}
/** Factory method to create an object from the database */
abstract public static function Get(ADOConnection $con, $id, $localeId);
}
class User extends dbClass
{
public static function Get(ADOConnection $con, $id, $localeId = 1053)
{
}
}
?>
Expected result:
----------------
Should be possible to assign default values to parameters in overloaded abstract functions.
Actual result:
--------------
Fatal error: Declaration of User::Get() must be compatible with that of DbClass::Get() in
d:\projects\inetpub\WebUser\test.php on line 23
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=30793&edit=1