Req #50296 [Opn->Wfx]: Automatically translate "self" in method_exists

From: Date: Thu, 18 Feb 2021 11:33:52 +0000
Subject: Req #50296 [Opn->Wfx]: Automatically translate "self" in method_exists
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-232242@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=50296&edit=1 ID: 50296 Updated by: nikic@php.net Reported by: soapergem at gmail dot com Summary: Automatically translate "self" in method_exists -Status: Open +Status: Wont fix Type: Feature/Change Request Package: Scripting Engine problem Operating System: Windows XP PHP Version: 5.3.1 Block user comment: N Private report: N New Comment: Support for self/parent/static in callables is nowadays considered to have been a mistake, as it makes them context-dependent. I think I can safely say that this will not be supported for method_exists(). I think that (with the exception of callables) PHP fairly consistenty rejects "self" etc for dynamic class names. The class name can be obtained using self::class, or __CLASS__. Previous Comments: ------------------------------------------------------------------------ [2011-01-12 20:04:10] m dot grundner at delegate dot at Why don't you use method_exists(__CLASS__, "method")? ------------------------------------------------------------------------ [2009-11-25 17:24:05] soapergem at gmail dot com Description: ------------ In reference to bug #50289: If I call method_exists("self", "method") from within a class, it should automatically translate "self" to the classname. Right now the behavior is not obvious; the behavior does not adhere to the KISS principle. 1. method_exists already can check for static methods. 2. "self" can already automatically translate to the current class name, for instance in call_user_func. 3. The most logical, simple and obvious expectation is that if you combine these two--you call method_exists with "self"--that it will automatically translate "self" to the current class name, and check for the method. Currently it does not. The current behavior is counterintuitive and confusing. Making this change is not dangerous. Making this change will not cause any backwards-compatibility issues. 1. "self" is a reserved word. You cannot declare a class named "self"- -this will issue a fatal error. So there is no possibility of any ambiguity. You cannot confuse "self" with any other class, because no other class can be named "self". It does not make sense to have "self" work in some contexts but not in others. It would be better if it were consistent. Reproduce code: --------------- <?php class Test { public static function check() { var_dump(method_exists("self", "method")); } public static function method() { } } Test::check(); ?> Expected result: ---------------- bool(true) Actual result: -------------- bool(false) ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=50296&edit=1

« previous php.bugs (#232242) next »