[Pixelshaped]

Fixes #8769

Not entirely sure why priorities were changed in #7739 @alexisLefebvre?

[alexisLefebvre]

Good catch, thanks. The #7739 commit says "Adjust Doctrine metadata factory decoration priorities to ensure correct execution order", but looking at it again the move wasn't needed. setParametersFilterClass() only needs the parameters built by ParameterResourceMetadataCollectionFactory, which was already decorated at 1000, so it ran before the Doctrine factory at 40. Going below -10 only moved the Doctrine factory outside CachedResourceMetadataCollectionFactory, which wasn't intended. -5 keeps it inside the cache and still after every other factory, so 👍 for this fix.

Could you check that tests/Functional/Parameters/DoctrineTest.php (the stateOptions test from #7739) still passes with -5? CI seems broken for unrelated reasons right now.

[Pixelshaped]

I ran DoctrineTest.php and it passes at -5. That includes State options and State options and no api filter, the two tests tied to #7739’s stateOptions behaviour.

I pushed on 4.2 because the bug was introduced on 4.2 but the CI seems otherwise broken on 4.2. I can target another branch if needed.