Edit report at https://bugs.php.net/bug.php?id=62343&edit=1

 ID:                 62343
 Updated by:         dmi...@php.net
 Reported by:        valentiny510 at yahoo dot es
 Summary:            Show class_alias In get_declared_classes( )
-Status:             Assigned
+Status:             Closed
 Type:               Feature/Change Request
 Package:            Class/Object related
 Operating System:   XP
 PHP Version:        5.4.4
 Assigned To:        dmitry
 Block user comment: N
 Private report:     N

 New Comment:

Automatic comment on behalf of dmi...@zend.com
Revision: 
http://git.php.net/?p=php-src.git;a=commit;h=4a6291508d69fab951562b996ed7223c052a0168
Log: Fixed bug #62343 (Show class_alias In get_declared_classes())


Previous Comments:
------------------------------------------------------------------------
[2012-08-25 13:24:12] valentiny510 at yahoo dot es

P.S. I think the "easyest" solucion can be to create a new function like 
"get_aliases( )" and assign all the aliases references to that function
and this way we can know for sure what names from the array are aliases

------------------------------------------------------------------------
[2012-08-25 13:07:12] valentiny510 at yahoo dot es

pajoye said: "Aliases are not declared classes but ... aliases :)"

With this comment I must understand that if the aliases are not declared 
classes, that aliases must NOT be included in the get_declared_classes( ) also 
..
right ? .. see u smiling :D

------------------------------------------------------------------------
[2012-06-25 08:00:27] dmi...@php.net

I would say the patch is bad, because all the returned class names will be 
lowercased. However, it's possible to improve the patch by comparison of 
lowercasesd ce->name and hash_key->arKey. In case they are equal we can return 
ce->name and hash_key->arKey otherwise.

I would also prefer not to extend functionality of internal functions without a 
big reason (user_only argument).

------------------------------------------------------------------------
[2012-06-22 16:25:50] larue...@php.net

okey, thanks :)

------------------------------------------------------------------------
[2012-06-22 09:37:10] paj...@php.net

Also bring that on internals please. There are room for discussions here.

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=62343


-- 
Edit this bug report at https://bugs.php.net/bug.php?id=62343&edit=1

Reply via email to