-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Add ReflectionProperty::getMangledName()
#18980
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
alexandre-daubois
wants to merge
1
commit into
php:master
Choose a base branch
from
alexandre-daubois:props-mangling
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
43 changes: 43 additions & 0 deletions
43
ext/reflection/tests/ReflectionProperty_getMangledName_basic.phpt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,43 @@ | ||
--TEST-- | ||
Test ReflectionProperty::getMangledName() method | ||
--FILE-- | ||
<?php | ||
|
||
class TestClass { | ||
public $publicProp = 'public'; | ||
protected $protectedProp = 'protected'; | ||
private $privateProp = 'private'; | ||
} | ||
|
||
function testMangledName($class, $property) { | ||
$reflection = new ReflectionProperty($class, $property); | ||
echo "Property: $property\n"; | ||
echo "getName(): " . $reflection->getName() . "\n"; | ||
echo "getMangledName(): " . $reflection->getMangledName() . "\n"; | ||
|
||
$obj = new $class(); | ||
$array = (array) $obj; | ||
echo "In array cast: " . (array_key_exists($reflection->getMangledName(), $array) ? "found" : "not found") . "\n"; | ||
echo "\n"; | ||
} | ||
|
||
testMangledName('TestClass', 'publicProp'); | ||
testMangledName('TestClass', 'protectedProp'); | ||
testMangledName('TestClass', 'privateProp'); | ||
|
||
?> | ||
--EXPECTF-- | ||
Property: publicProp | ||
getName(): publicProp | ||
getMangledName(): publicProp | ||
In array cast: found | ||
|
||
Property: protectedProp | ||
getName(): protectedProp | ||
getMangledName(): %0*%0protectedProp | ||
In array cast: found | ||
|
||
Property: privateProp | ||
getName(): privateProp | ||
getMangledName(): %0TestClass%0privateProp | ||
In array cast: found |
100 changes: 100 additions & 0 deletions
100
ext/reflection/tests/ReflectionProperty_getMangledName_dynamic.phpt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,100 @@ | ||
--TEST-- | ||
Test ReflectionProperty::getMangledName() with dynamic properties | ||
--FILE-- | ||
<?php | ||
|
||
echo "=== Testing stdClass with dynamic properties ===\n"; | ||
$stdObj = new stdClass(); | ||
$stdObj->prop1 = 'value1'; | ||
$stdObj->prop2 = 'value2'; | ||
$stdObj->{'special-name'} = 'special value'; | ||
$stdObj->{'123numeric'} = 'numeric start'; | ||
|
||
function testDynamicProperty($obj, $property, $description) { | ||
try { | ||
$reflection = new ReflectionProperty($obj, $property); | ||
echo "$description:\n"; | ||
echo " getName(): " . $reflection->getName() . "\n"; | ||
echo " getMangledName(): " . $reflection->getMangledName() . "\n"; | ||
|
||
$array = (array) $obj; | ||
echo " Found in array cast: " . (array_key_exists($reflection->getMangledName(), $array) ? "yes" : "no") . "\n"; | ||
echo "\n"; | ||
} catch (ReflectionException $e) { | ||
echo "$description: EXCEPTION - " . $e->getMessage() . "\n\n"; | ||
} | ||
} | ||
|
||
testDynamicProperty($stdObj, 'prop1', 'stdClass property prop1'); | ||
testDynamicProperty($stdObj, 'special-name', 'stdClass property with special name'); | ||
testDynamicProperty($stdObj, '123numeric', 'stdClass property starting with number'); | ||
|
||
echo "=== Testing regular class with dynamic properties ===\n"; | ||
#[AllowDynamicProperties] | ||
class TestClass { | ||
public $existing = 'existing'; | ||
} | ||
|
||
$obj = new TestClass(); | ||
$obj->dynamic = 'dynamic value'; | ||
$obj->anotherDynamic = 'another dynamic'; | ||
|
||
testDynamicProperty($obj, 'dynamic', 'Regular class dynamic property'); | ||
testDynamicProperty($obj, 'anotherDynamic', 'Regular class another dynamic property'); | ||
|
||
$reflection = new ReflectionProperty($obj, 'existing'); | ||
echo "Regular property:\n"; | ||
echo " getName(): " . $reflection->getName() . "\n"; | ||
echo " getMangledName(): " . $reflection->getMangledName() . "\n"; | ||
|
||
echo "\n=== Testing ReflectionProperty from class vs instance ===\n"; | ||
try { | ||
$reflection = new ReflectionProperty('TestClass', 'dynamic'); | ||
echo "This should not be reached\n"; | ||
} catch (ReflectionException $e) { | ||
echo "Expected exception for class-based reflection: " . $e->getMessage() . "\n"; | ||
} | ||
|
||
try { | ||
$reflection = new ReflectionProperty($obj, 'dynamic'); | ||
echo "Instance-based reflection works: " . $reflection->getMangledName() . "\n"; | ||
} catch (ReflectionException $e) { | ||
echo "Unexpected exception: " . $e->getMessage() . "\n"; | ||
} | ||
|
||
?> | ||
--EXPECTF-- | ||
=== Testing stdClass with dynamic properties === | ||
stdClass property prop1: | ||
getName(): prop1 | ||
getMangledName(): prop1 | ||
Found in array cast: yes | ||
|
||
stdClass property with special name: | ||
getName(): special-name | ||
getMangledName(): special-name | ||
Found in array cast: yes | ||
|
||
stdClass property starting with number: | ||
getName(): 123numeric | ||
getMangledName(): 123numeric | ||
Found in array cast: yes | ||
|
||
=== Testing regular class with dynamic properties === | ||
Regular class dynamic property: | ||
getName(): dynamic | ||
getMangledName(): dynamic | ||
Found in array cast: yes | ||
|
||
Regular class another dynamic property: | ||
getName(): anotherDynamic | ||
getMangledName(): anotherDynamic | ||
Found in array cast: yes | ||
|
||
Regular property: | ||
getName(): existing | ||
getMangledName(): existing | ||
|
||
=== Testing ReflectionProperty from class vs instance === | ||
Expected exception for class-based reflection: Property TestClass::$dynamic does not exist | ||
Instance-based reflection works: dynamic |
65 changes: 65 additions & 0 deletions
65
ext/reflection/tests/ReflectionProperty_getMangledName_inheritance.phpt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,65 @@ | ||
--TEST-- | ||
Test ReflectionProperty::getMangledName() with inheritance | ||
--FILE-- | ||
<?php | ||
|
||
class ParentClass { | ||
public $public = 'parent_public'; | ||
protected $protected = 'parent_protected'; | ||
private $private = 'parent_private'; | ||
} | ||
|
||
class ChildClass extends ParentClass { | ||
private $private = 'child_private'; | ||
protected $childProp = 'child_protected'; | ||
} | ||
|
||
function testProperty($class, $property) { | ||
$reflection = new ReflectionProperty($class, $property); | ||
$obj = new $class(); | ||
$array = (array) $obj; | ||
|
||
echo "Class: $class, Property: $property\n"; | ||
echo "Mangled name: '" . $reflection->getMangledName() . "'\n"; | ||
echo "Key exists in array cast: " . (array_key_exists($reflection->getMangledName(), $array) ? "yes" : "no") . "\n"; | ||
echo "\n"; | ||
} | ||
|
||
testProperty('ParentClass', 'public'); | ||
testProperty('ParentClass', 'protected'); | ||
testProperty('ParentClass', 'private'); | ||
|
||
testProperty('ChildClass', 'public'); | ||
testProperty('ChildClass', 'protected'); | ||
testProperty('ChildClass', 'childProp'); | ||
testProperty('ChildClass', 'private'); | ||
|
||
?> | ||
--EXPECTF-- | ||
Class: ParentClass, Property: public | ||
Mangled name: 'public' | ||
Key exists in array cast: yes | ||
|
||
Class: ParentClass, Property: protected | ||
Mangled name: '%0*%0protected' | ||
Key exists in array cast: yes | ||
|
||
Class: ParentClass, Property: private | ||
Mangled name: '%0ParentClass%0private' | ||
Key exists in array cast: yes | ||
|
||
Class: ChildClass, Property: public | ||
Mangled name: 'public' | ||
Key exists in array cast: yes | ||
|
||
Class: ChildClass, Property: protected | ||
Mangled name: '%0*%0protected' | ||
Key exists in array cast: yes | ||
|
||
Class: ChildClass, Property: childProp | ||
Mangled name: '%0*%0childProp' | ||
Key exists in array cast: yes | ||
|
||
Class: ChildClass, Property: private | ||
Mangled name: '%0ChildClass%0private' | ||
Key exists in array cast: yes |
99 changes: 99 additions & 0 deletions
99
ext/reflection/tests/ReflectionProperty_getMangledName_instance.phpt
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,99 @@ | ||
--TEST-- | ||
Test ReflectionProperty::getMangledName() from instance vs class | ||
--FILE-- | ||
<?php | ||
|
||
#[AllowDynamicProperties] | ||
class TestClass { | ||
public $public = 'public'; | ||
protected $protected = 'protected'; | ||
private $private = 'private'; | ||
} | ||
|
||
$obj = new TestClass(); | ||
$obj->dynamic = 'dynamic'; | ||
|
||
echo "=== Testing ReflectionProperty from CLASS ===\n"; | ||
|
||
function testFromClass($property) { | ||
try { | ||
$reflection = new ReflectionProperty('TestClass', $property); | ||
echo "Property $property from class:\n"; | ||
echo " getName(): " . $reflection->getName() . "\n"; | ||
echo " getMangledName(): " . $reflection->getMangledName() . "\n"; | ||
echo "\n"; | ||
} catch (ReflectionException $e) { | ||
echo "Property $property from class: EXCEPTION - " . $e->getMessage() . "\n\n"; | ||
} | ||
} | ||
|
||
testFromClass('public'); | ||
testFromClass('protected'); | ||
testFromClass('private'); | ||
testFromClass('dynamic'); | ||
|
||
echo "=== Testing ReflectionProperty from INSTANCE ===\n"; | ||
|
||
function testFromInstance($obj, $property) { | ||
try { | ||
$reflection = new ReflectionProperty($obj, $property); | ||
echo "Property $property from instance:\n"; | ||
echo " getName(): " . $reflection->getName() . "\n"; | ||
echo " getMangledName(): " . $reflection->getMangledName() . "\n"; | ||
|
||
$array = (array) $obj; | ||
echo " Found in array cast: " . (array_key_exists($reflection->getMangledName(), $array) ? "yes" : "no") . "\n"; | ||
echo "\n"; | ||
} catch (ReflectionException $e) { | ||
echo "Property $property from instance: EXCEPTION - " . $e->getMessage() . "\n\n"; | ||
} | ||
} | ||
|
||
testFromInstance($obj, 'public'); | ||
testFromInstance($obj, 'protected'); | ||
testFromInstance($obj, 'private'); | ||
|
||
echo "=== Instance array keys ===\n"; | ||
$array = (array) $obj; | ||
foreach (array_keys($array) as $key) { | ||
echo "Key: '$key'\n"; | ||
} | ||
|
||
?> | ||
--EXPECTF-- | ||
=== Testing ReflectionProperty from CLASS === | ||
Property public from class: | ||
getName(): public | ||
getMangledName(): public | ||
|
||
Property protected from class: | ||
getName(): protected | ||
getMangledName(): %0*%0protected | ||
|
||
Property private from class: | ||
getName(): private | ||
getMangledName(): %0TestClass%0private | ||
|
||
Property dynamic from class: EXCEPTION - Property TestClass::$dynamic does not exist | ||
|
||
=== Testing ReflectionProperty from INSTANCE === | ||
Property public from instance: | ||
getName(): public | ||
getMangledName(): public | ||
Found in array cast: yes | ||
|
||
Property protected from instance: | ||
getName(): protected | ||
getMangledName(): %0*%0protected | ||
Found in array cast: yes | ||
|
||
Property private from instance: | ||
getName(): private | ||
getMangledName(): %0TestClass%0private | ||
Found in array cast: yes | ||
|
||
=== Instance array keys === | ||
Key: 'public' | ||
Key: '%0*%0protected' | ||
Key: '%0TestClass%0private' | ||
Key: 'dynamic' |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.