Consistency: final, readonly, and minor cleanup#196

Merged

yann-eugone merged 10 commits into1.xfrom

refactor-code-consistency

Apr 17, 2026

Commits on Apr 17, 2026

Fix @phstan-var typo to @phpstan-var in TriggerScheduledJobsJob
yann-eugone
committed
Fix duplicate @param/@return tags in docblocks (ItemWriterInterface, FinderInterface, SchedulerInterface)
yann-eugone
committed
Fix typo: rename $contraints to $constraints in SkipInvalidItemProcessor
yann-eugone
committed
Add missing final keyword on JobWithChildJobs (was only @final in docblock)
yann-eugone
committed
Make getVariables() private in ReplaceWithVariablesParameterAccessor: protected on a final class is a contradiction
yann-eugone
committed
Make CallbackScheduler final with readonly property, consistent with other concrete classes in the codebase
yann-eugone
committed
Make MessengerJobsConfiguration readonly
yann-eugone
committed
Replace mb_strpos with str_contains in DoctrineDBALQueryOffsetReader for consistency with DoctrineDBALQueryCursorReader
yann-eugone
committed
Use constructor property promotion for $sql and $batch in DoctrineDBALQueryOffsetReader
yann-eugone
committed
Add instructions to upgrade guide
yann-eugone
committed