Skip to content
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

[5.2] [PHP8.4] Update for nullable type declaration #43323

Merged

Conversation

heelc29
Copy link
Contributor

@heelc29 heelc29 commented Apr 20, 2024

Summary of Changes

updates for https://wiki.php.net/rfc/deprecate-implicitly-nullable-types

Testing Instructions

at the moment no QA release of PHP8.4 ... so only code review (and phpcs by drone) PHP 8.4.0 Alpha 1 available for testing

Actual result BEFORE applying this Pull Request

Warning:

Deprecated: Joomla\CMS\Application\AdministratorApplication::__construct(): Implicitly marking parameter $input as nullable is deprecated, the explicit nullable type must be used instead in libraries\src\Application\AdministratorApplication.php on line 74

Deprecated: Joomla\CMS\Application\AdministratorApplication::__construct(): Implicitly marking parameter $config as nullable is deprecated, the explicit nullable type must be used instead in libraries\src\Application\AdministratorApplication.php on line 74

Deprecated: Joomla\CMS\Application\AdministratorApplication::__construct(): Implicitly marking parameter $client as nullable is deprecated, the explicit nullable type must be used instead in libraries\src\Application\AdministratorApplication.php on line 74

Deprecated: Joomla\CMS\Application\AdministratorApplication::__construct(): Implicitly marking parameter $container as nullable is deprecated, the explicit nullable type must be used instead in libraries\src\Application\AdministratorApplication.php on line 74

[...]

Expected result AFTER applying this Pull Request

No new deprecated warnings (excepted composer dependencies) if update to PHP8.4

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed

@HLeithner HLeithner changed the base branch from 5.1-dev to 5.2-dev April 24, 2024 09:06
@HLeithner
Copy link
Member

This pull request has been automatically rebased to 5.2-dev.

@HLeithner HLeithner changed the title [5.1][PHP8.4][WIP] Update for nullable type declaration [5.2] [PHP8.4][WIP] Update for nullable type declaration Apr 24, 2024
@heelc29 heelc29 changed the title [5.2] [PHP8.4][WIP] Update for nullable type declaration [5.2] [PHP8.4] Update for nullable type declaration Apr 25, 2024
@heelc29 heelc29 marked this pull request as ready for review April 25, 2024 19:17
@heelc29 heelc29 force-pushed the 5.1/php84/implicitly-nullable-parameter branch from 74dad8b to 02fe995 Compare June 30, 2024 17:23
Copy link
Member

@HLeithner HLeithner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did a code review on all 286 changed files, all of them looking ok except one which is also ok but not changing implizit null
image

libraries/src/Document/Document.php Show resolved Hide resolved
@laoneo laoneo merged commit aa475c1 into joomla:5.2-dev Aug 6, 2024
3 checks passed
@laoneo
Copy link
Member

laoneo commented Aug 6, 2024

Thanks!

@laoneo laoneo added this to the Joomla! 5.2.0 milestone Aug 6, 2024
@heelc29 heelc29 deleted the 5.1/php84/implicitly-nullable-parameter branch September 1, 2024 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants