Côme Chilliet
Côme Chilliet
It’s the first time I hear about `OCP\Collaboration\Collaborators\ISearch`. Maybe the other apps use the IUserManager instead directly to get the users. But an instance where user ids are messed like...
Duplicate of #446 ?
Which branch? master uses psalm/phar already since https://github.com/nextcloud/photos/pull/2189 . Regarding symfony/console and sabre/dav, not sure how to avoid pulling them to get correct psalm validation?
/backport to stable30
https://docs.nextcloud.com/server/latest/developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_30.html is the official list for changes in 30 to take into account for applications.
https://github.com/nextcloud/server/pull/52538/commits/98b2cfc4162115e7295f683d2810e53df93010b5 This was intended to fix it. But Core application changed in https://github.com/nextcloud/server/pull/53157 since then, so maybe something different is happening. @kesselb no trace?
@icewind1991 @artonge I’m interested in your input on this. The most clear explanation is in https://github.com/nextcloud/forms/issues/2067#issuecomment-2366985992 Part of the problem is that files_version needs a path to work with, and...
> In quite a few places it is nice to use the fqn to avoid mistakes. Do you have examples in mind so that we can look if there is...
> Could we maybe limit this to a few classes like \OCP\Server and \OCP\Util for now? I don't know how many other cases we have where this happens a lot....
So, the approach in https://github.com/nextcloud-libraries/rector/pull/11/files failed. @provokateurin What do you think about using the strategy in this PR (auto import from OC/OCA/OCP), but with a manually maitained list of exempted...