refactor(psalm): Fully type IStorageFactory

[CarlSchwan]

Summary

TODO

• ...

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)
• Labels added where applicable (ex: bug/enhancement, 3. to review, feature component)
• Milestone added for target branch/version (ex: 32.x for stable32)

[provokateurin]

Please add all files to psalm:strict and rector:strict.

[CarlSchwan]

Please add all files to psalm:strict and rector:strict.

psalm strict really disliked the usort call for some reasons

ERROR: MixedInferredReturnType - lib/private/Files/Storage/StorageFactory.php:60:52 - Could not verify return type 'int' for /home/carl/projects/nextcloud-docker-dev/workspace/server/lib/private/files/storage/storagefactory.php:60:1946:-:closure (see https://psalm.dev/047)
                usort($wrappers, static fn (array $a, array $b): int => $b['priority'] - $a['priority']);


ERROR: MixedOperand - lib/private/Files/Storage/StorageFactory.php:60:59 - Left operand cannot be mixed (see https://psalm.dev/059)
                usort($wrappers, static fn (array $a, array $b): int => $b['priority'] - $a['priority']);


ERROR: MixedReturnStatement - lib/private/Files/Storage/StorageFactory.php:60:59 - Could not infer a return type (see https://psalm.dev/138)
                usort($wrappers, static fn (array $a, array $b): int => $b['priority'] - $a['priority']);

[provokateurin]

That's odd, I can take a look tomorrow if you want me to.