Skip to content

deprecation(php): Fix PHP8.4 deprecations#135

Open
jerome-fix wants to merge 1 commit into
getbrevo:mainfrom
jerome-fix:fix-php-84-deprecations
Open

deprecation(php): Fix PHP8.4 deprecations#135
jerome-fix wants to merge 1 commit into
getbrevo:mainfrom
jerome-fix:fix-php-84-deprecations

Conversation

@jerome-fix
Copy link
Copy Markdown

  • Implicitly marking parameter $data as nullable is deprecated, the explicit nullable type must be used instead
  • Return type should either be compatible with declaration => add #[\ReturnTypeWillChange] attribute to temporarily suppress the notice

PHP 5.6 compatibility has already been broken several times:

Nullable types only appeared with PHP 7.1, but they are already used in several classes (03dddf3, for example).

It may be necessary to modify the composer.json, since as it stands, version 2.0.14 is not compatible with PHP 5.6.

- Implicitly marking parameter $data as nullable is deprecated, the explicit nullable type must be used instead
- Return type should either be compatible with declaration => add #[\ReturnTypeWillChange] attribute to temporarily suppress the notice
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
81.8% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

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.

1 participant