Fixed PermissionResolver::canUser examples

[adriendupuis]

Question	Answer
JIRA issue	N/A
Type	improvement
Target Ibexa version	v4.5+
BC breaks	no

The 4th parameter of PermissionResolver::canUse must be an array according to its signature, an array of ValueObject according to its phpDoc.

Tested with a controller extending Ibexa\Core\MVC\Symfony\Controller\Controller:

$this->getRepository()->getPermissionResolver()->canUser('content', 'read', $content, $location);

It ends with error "Ibexa\Core\Repository\Permission\CachedPermissionService::canUser(): Argument #4 ($targets) must be of type array, Ibexa\Core\Repository\Values\Content\Location given"

The following quick fix seems to work but I didn't have a proper limitation to go with it as I could just pass an empty array or random strings in the array:

$this->getRepository()->getPermissionResolver()->canUser('content', 'read', $content, [$location]);

It needs a more proper/advanced test by someone else than me 😓

See ibexa/documentation-developer#2132 for doc counter-part.

Checklist:

• Provided PR description.
• Tested the solution manually.
• Provided automated test coverage.
• Checked that target branch is set correctly (main for features, the oldest supported for bugs).
• Ran PHP CS Fixer for new PHP code (use $ composer fix-cs).
• Asked for a review (ping @ibexa/engineering).

[adamwojs]

Could you please change branch to 4.5? In opposite to documentation repositories we are open PR against the lowest version when the given change is applicable

[alongosz]

Target Ibexa version | v4.5+

@adriendupuis could you rebase it against 4.5 then please? :)

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[adriendupuis]

@adamwojs @alongosz git rebase main --onto 4.5 done and PR target edited.

[alongosz]

Thank you @adriendupuis 🎉