[stable10] owncloud coding standard

[phil-davis]

Backport #31442

The first commits backport the various interim php-cs-fixer changes from PR #31198 #31236 #31250 #31271 #31273

That was the easy way to make a list of cherry-picks that all fit nicely on top of each other. And its brings along the Makefile and drone changes etc that were already done before #31442

The last commit uses php-cs-fixer to make the automatic fixes needed for the code in stable10

lib/composer/bin/php-cs-fixer fix -v --diff --diff-format udiff --allow-risky yes

[codecov]

Codecov Report

Merging #31657 into stable10 will decrease coverage by 0.05%.
The diff coverage is 70.53%.

@@              Coverage Diff               @@
##             stable10   #31657      +/-   ##
==============================================
- Coverage       62.15%    62.1%   -0.06%     
+ Complexity      18249    18235      -14     
==============================================
  Files            1199     1199              
  Lines           71798    71882      +84     
  Branches         1248     1248              
==============================================
+ Hits            44629    44641      +12     
- Misses          26799    26871      +72     
  Partials          370      370

Flag	Coverage Δ	Complexity Δ
#javascript	52.64% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	63.13% <70.53%> (-0.07%)	18235 <660> (-14)

Impacted Files	Coverage Δ	Complexity Δ
apps/dav/lib/CardDAV/ContactsManager.php	100% <ø> (ø)	4 <0> (ø)	⬇️
...nnector/Sabre/Exception/PasswordLoginForbidden.php	0% <ø> (ø)	2 <0> (ø)	⬇️
apps/files/list.php	0% <ø> (ø)	0 <0> (ø)	⬇️
apps/files/templates/index.php	0% <ø> (ø)	0 <0> (ø)	⬇️
apps/files/simplelist.php	0% <ø> (ø)	0 <0> (ø)	⬇️
apps/files_external/lib/AppInfo/Application.php	0% <ø> (ø)	0 <0> (ø)	⬇️
apps/files/appinfo/routes.php	100% <ø> (ø)	0 <0> (ø)	⬇️
apps/dav/lib/Capabilities.php	0% <ø> (ø)	1 <0> (ø)	⬇️
apps/federation/lib/Hooks.php	100% <ø> (ø)	4 <0> (ø)	⬇️
.../federation/lib/Middleware/AddServerMiddleware.php	100% <ø> (ø)	3 <0> (ø)	⬇️
... and 578 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6f799c5...7f59cd4. Read the comment docs.

[DeepDiver1975]

Conflicting - please merge after 10.0.9 - otherwise we are lost in rebase nightmare again ....

[phil-davis]

Note: For now, this is just a place to keep a handy list of all the commits that make up implementing the code-style checks. The last commit(s) that actually apply the changes to the code can easily be dropped and then re-made by running the checker to make the fixes. That is often easier (and more reliable) than manually resolving loads of "rubbish" conflicts that happen here.

[phil-davis]

Too many conflicts due to objectstore backport merging. It is much quicker to create a new branch, cherry-pick the functional commits and run php-cs-fixer again to make the code sttyle changes.
See new PR #31722