Skip to content

IBX-8471: Upgraded codebase to Symfony 7 #501

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 89 commits into
base: main
Choose a base branch
from
Draft

Conversation

alongosz
Copy link
Member

@alongosz alongosz commented Mar 3, 2025

Caution

Contains other PRs

Merge together with other IBX-8471 Pull Requests

🎫 Issue IBX-8471

Description:

Upgrading Symfony dependencies to version 7 along with necessary 3rd party packages and codebase changes.

Key changes:

  • [Composer] Dropped obsolete symfony/proxy-manager-bridge
  • [Composer] Bumped Symfony packages requirements to ^7.2
  • [Composer] Bumped 3rd party packages compatible with for Symfony 7

For QA:

Sanity tests and/or regression builds.

@alongosz alongosz force-pushed the ibx-8471-symfony-7 branch 5 times, most recently from b62a943 to cc1494b Compare April 7, 2025 13:59
alongosz and others added 21 commits April 10, 2025 16:47
Applied Rectors:
  * Rector\TypeDeclaration\Rector\ArrowFunction\AddArrowFunctionReturnTypeRector
  * Rector\TypeDeclaration\Rector\Class_\AddTestsVoidReturnTypeWhereNoReturnRector
  * Rector\TypeDeclaration\Rector\ClassMethod\AddMethodCallBasedStrictParamTypeRector
  * Rector\TypeDeclaration\Rector\ClassMethod\AddParamTypeBasedOnPHPUnitDataProviderRector
  * Rector\TypeDeclaration\Rector\ClassMethod\AddParamTypeFromPropertyTypeRector
  * Rector\TypeDeclaration\Rector\ClassMethod\AddReturnTypeDeclarationBasedOnParentClassMethodRector
  * Rector\TypeDeclaration\Rector\ClassMethod\AddVoidReturnTypeWhereNoReturnRector
  * Rector\TypeDeclaration\Rector\ClassMethod\BoolReturnTypeFromBooleanConstReturnsRector
  * Rector\TypeDeclaration\Rector\ClassMethod\BoolReturnTypeFromBooleanStrictReturnsRector
  * Rector\TypeDeclaration\Rector\ClassMethod\NumericReturnTypeFromStrictReturnsRector
  * Rector\TypeDeclaration\Rector\ClassMethod\NumericReturnTypeFromStrictScalarReturnsRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ParamTypeByMethodCallTypeRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ParamTypeByParentCallTypeRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnNeverTypeRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnNullableTypeRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnTypeFromMockObjectRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnTypeFromReturnDirectArrayRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnTypeFromReturnNewRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnTypeFromStrictConstantReturnRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnTypeFromStrictNativeCallRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnTypeFromStrictNewArrayRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnTypeFromStrictParamRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnTypeFromStrictTypedCallRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnTypeFromStrictTypedPropertyRector
  * Rector\TypeDeclaration\Rector\ClassMethod\ReturnUnionTypeRector
  * Rector\TypeDeclaration\Rector\ClassMethod\StrictArrayParamDimFetchRector
  * Rector\TypeDeclaration\Rector\ClassMethod\StrictStringParamConcatRector
  * Rector\TypeDeclaration\Rector\ClassMethod\StringReturnTypeFromStrictScalarReturnsRector
  * Rector\TypeDeclaration\Rector\ClassMethod\StringReturnTypeFromStrictStringReturnsRector
  * Rector\TypeDeclaration\Rector\Class_\PropertyTypeFromStrictSetterGetterRector
  * Rector\TypeDeclaration\Rector\Class_\TypedPropertyFromCreateMockAssignRector
  * Rector\TypeDeclaration\Rector\Closure\AddClosureVoidReturnTypeWhereNoReturnRector
  * Rector\TypeDeclaration\Rector\Closure\ClosureReturnTypeRector
  * Rector\TypeDeclaration\Rector\Empty_\EmptyOnNullableObjectToInstanceOfRector
  * Rector\TypeDeclaration\Rector\Property\TypedPropertyFromAssignsRector
  * Rector\TypeDeclaration\Rector\Property\TypedPropertyFromStrictConstructorRector
  * Rector\TypeDeclaration\Rector\Property\TypedPropertyFromStrictSetUpRector
Doctrine DBAL upgrade was a combination of:
 * applying `DoctrineSetList` Rector sets,
 * manual code style fixes of Rector changes,
 * manual fixes of issues found by SonarQube PHPStorm plugin,
 * manual fixes of issues found by PHPStan.
Doctrine DBAL upgrade was a combination of:
 * applying `DoctrineSetList` Rector sets,
 * manual code style fixes of Rector changes,
 * manual fixes of issues found by SonarQube PHPStorm plugin,
 * manual fixes of issues found by PHPStan,
 * manual code quality improvements.
…ve PDO::FETCH_GROUP usage

Extracted getSubtreeContent `$onlyIds` behavior to a separate method as it applies to a different use case on a similar query.
@alongosz alongosz force-pushed the ibx-8471-symfony-7 branch from cc1494b to c142542 Compare April 10, 2025 14:47
alongosz added 28 commits April 15, 2025 15:35
@alongosz alongosz force-pushed the ibx-8471-symfony-7 branch from f4b1909 to 84347ec Compare April 16, 2025 15:44
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
5 Security Hotspots
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants