Skip to content

Commit 88133cc

Browse files
Fix uopz signature
1 parent ce0aaf2 commit 88133cc

File tree

3 files changed

+36
-8
lines changed

3 files changed

+36
-8
lines changed

resources/functionMap.php

+8-8
Original file line numberDiff line numberDiff line change
@@ -12929,15 +12929,15 @@
1292912929
'unserialize' => ['mixed', 'variable_representation'=>'string', 'allowed_classes='=>'array{allowed_classes?:string[]|bool}'],
1293012930
'untaint' => ['bool', '&rw_string'=>'string', '&...rw_strings='=>'string'],
1293112931
'uopz_add_function' => ['bool', 'class'=>'string', 'function'=>'string', 'handler'=>'Closure', '$flags'=>'bool', '$all'=>'bool'],
12932-
'uopz_add_function\1' => ['bool', 'function'=>'string', 'handler'=>'Closure', '$flags'=>'bool'],
12932+
'uopz_add_function\'1' => ['bool', 'function'=>'string', 'handler'=>'Closure', '$flags'=>'bool'],
1293312933
'uopz_allow_exit' => ['void', 'allow'=>'bool'],
1293412934
'uopz_backup' => ['void', 'class'=>'string', 'function'=>'string'],
1293512935
'uopz_backup\'1' => ['void', 'function'=>'string'],
1293612936
'uopz_compose' => ['void', 'name'=>'string', 'classes'=>'array', 'methods='=>'array', 'properties='=>'array', 'flags='=>'int'],
1293712937
'uopz_copy' => ['Closure', 'class'=>'string', 'function'=>'string'],
1293812938
'uopz_copy\'1' => ['Closure', 'function'=>'string'],
1293912939
'uopz_del_function' => ['bool', 'class'=>'string', 'function'=>'string', '$all'=>'bool'],
12940-
'uopz_del_function\1' => ['bool', 'function'=>'string'],
12940+
'uopz_del_function\'1' => ['bool', 'function'=>'string'],
1294112941
'uopz_delete' => ['void', 'class'=>'string', 'function'=>'string'],
1294212942
'uopz_delete\'1' => ['void', 'function'=>'string'],
1294312943
'uopz_extend' => ['void', 'class'=>'string', 'parent'=>'string'],
@@ -12947,13 +12947,13 @@
1294712947
'uopz_function\'1' => ['void', 'function'=>'string', 'handler'=>'Closure', 'modifiers='=>'int'],
1294812948
'uopz_get_exit_status' => ['mixed'],
1294912949
'uopz_get_hook' => ['Closure', 'class'=>'string', 'function'=>'string'],
12950-
'uopz_get_hook\1' => ['Closure', 'function'=>'string'],
12950+
'uopz_get_hook\'1' => ['Closure', 'function'=>'string'],
1295112951
'uopz_get_mock' => ['mixed', 'class'=>'string'],
12952-
'uopz_get_property' => ['void', 'class'=>'string', 'property'=>'string'],
12953-
'uopz_get_property\1' => ['void', 'instance'=>'object', 'property'=>'string'],
12952+
'uopz_get_property' => ['mixed', 'class'=>'string', 'property'=>'string'],
12953+
'uopz_get_property\'1' => ['mixed', 'instance'=>'object', 'property'=>'string'],
1295412954
'uopz_get_return' => ['mixed', 'class='=>'string', 'function='=>'string'],
1295512955
'uopz_get_static' => ['array', 'class='=>'string', 'function='=>'string'],
12956-
'uopz_get_static\1' => ['array', 'function='=>'string'],
12956+
'uopz_get_static\'1' => ['array', 'function='=>'string'],
1295712957
'uopz_implement' => ['void', 'class'=>'string', 'interface'=>'string'],
1295812958
'uopz_overload' => ['void', 'opcode'=>'int', 'callable'=>'Callable'],
1295912959
'uopz_redefine' => ['void', 'class'=>'string', 'constant'=>'string', 'value'=>'mixed'],
@@ -12964,13 +12964,13 @@
1296412964
'uopz_restore\'1' => ['void', 'function'=>'string'],
1296512965
'uopz_set_mock' => ['void', 'class'=>'string', 'mock'=>'object|string'],
1296612966
'uopz_set_property' => ['void', 'class'=>'string', 'property'=>'string', 'value'=>'mixed'],
12967-
'uopz_set_property\1' => ['void', 'instance'=>'object', 'property'=>'string', 'value'=>'mixed'],
12967+
'uopz_set_property\'1' => ['void', 'instance'=>'object', 'property'=>'string', 'value'=>'mixed'],
1296812968
'uopz_set_return' => ['bool', 'class'=>'string', 'function'=>'string', 'value'=>'mixed', 'execute='=>'bool'],
1296912969
'uopz_set_return\'1' => ['bool', 'function'=>'string', 'value'=>'mixed', 'execute='=>'bool'],
1297012970
'uopz_undefine' => ['void', 'class'=>'string', 'constant'=>'string'],
1297112971
'uopz_undefine\'1' => ['void', 'constant'=>'string'],
1297212972
'uopz_set_hook' => ['bool', 'class'=>'string', 'function'=>'string', 'hook'=>'Closure'],
12973-
'uopz_set_hook\1' => ['bool', 'function'=>'string', 'hook'=>'Closure'],
12973+
'uopz_set_hook\'1' => ['bool', 'function'=>'string', 'hook'=>'Closure'],
1297412974
'uopz_unset_mock' => ['void', 'class'=>'string'],
1297512975
'uopz_unset_return' => ['bool', 'class='=>'string', 'function='=>'string'],
1297612976
'uopz_unset_return\'1' => ['bool', 'function'=>'string'],

tests/PHPStan/Rules/Functions/CallToFunctionParametersRuleTest.php

+5
Original file line numberDiff line numberDiff line change
@@ -1939,4 +1939,9 @@ public function testBug9167(): void
19391939
$this->analyse([__DIR__ . '/data/bug-9167.php'], []);
19401940
}
19411941

1942+
public function testBug12499(): void
1943+
{
1944+
$this->analyse([__DIR__ . '/data/bug-12499.php'], []);
1945+
}
1946+
19421947
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
<?php
2+
3+
namespace Bug12499;
4+
5+
class HelloWorld {
6+
public static string $testString = 'test';
7+
}
8+
9+
$property = uopz_get_property(HelloWorld::class, 'testString');
10+
11+
uopz_set_property(HelloWorld::class, 'testString', 'test2');
12+
13+
class HelloWorld2 {
14+
public function __construct(
15+
public string $testString,
16+
) {}
17+
}
18+
19+
$classInstance = new HelloWorld2('test');
20+
21+
$property = uopz_get_property($classInstance, 'testString');
22+
23+
uopz_set_property($classInstance, 'testString', 'test2');

0 commit comments

Comments
 (0)