Coding-standard: UnusedPrivateElementsSniff.UnusedMethod - false positive with Closure::fromCallable()

[enumag]

Closure::fromCallable is very useful when you want to pass a private method as callback. The sniff should recognize such method as used.

class Foo
{
    public function test()
    {
        array_map(\Closure::fromCallable([$this, 'map']), []);
    }

    private function map($item)
    {
        // ...
    }
}

[kukulich]

If you prepare PR, I'm willing to merge it. However I don't want to invest my time for supporting the old way [$this, 'map'].

You should really rewrite the code in the example to:

array_map(function ($item) {
    return $this->map($item);
}, []);

[enumag]

That's the syntax I used before. It's too much writing and forces you to repeat all the parameters. No thanks.

[TomasVotruba]

@enumag Yet it's cleaner for most IDEs, static analysis tools and refactoring tools

[github-actions[bot]]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.