moodle-cs icon indicating copy to clipboard operation
moodle-cs copied to clipboard

Analyse if we should start checking (4.4 and up) for setAccessible() reflection uses

Open stronk7 opened this issue 10 months ago • 2 comments

The setAccessible() methods are, since PHP 8.1 no-op. That implies that, since Moodle 4.4.0 (that requires PHP 8.1), there shouldn't be any use in core.

This issue is about to check if PHPCompatibility already has that and, depending on that, decide how we incorporate that check for us (importing if PHPCompatibility has it, implementing ir here, maybe as part of some deprecated processor, or just manually, we already have some similar sniffs looking for forbidden and other functions...)

Ciao :-)

stronk7 avatar Apr 23 '24 09:04 stronk7