sabbelasichon / typo3-rector

Rector for TYPO3
MIT License
226 stars 63 forks source link

Breaking: #101471 - Introduce type declarations in AbstractDriver #3669

Closed simonschaufi closed 6 months ago

simonschaufi commented 11 months ago

Breaking: #101471 - Introduce type declarations in AbstractDriver

https://docs.typo3.org/c/typo3/cms-core/main/en-us/Changelog/13.0/Breaking-101471-IntroduceTypeDeclarationsInAbstractDriver.html .. include:: /Includes.rst.txt

.. _breaking-101471-1690531810:

================================================================= Breaking: #101471 - Introduce type declarations in AbstractDriver

See :issue:101471

Description

Return and param type declarations have been introduced for all methods and method stubs of of :php:\TYPO3\CMS\Core\Resource\Driver\AbstractDriver and :php:\TYPO3\CMS\Core\Resource\Driver\AbstractHierarchicalFilesystemDriver

Impact

In consequence, all classes, extending any of those abstract classes and overriding any of those affected methods need to reflect those changes and add the same return and param type declarations.

Affected methods are:

Affected installations

Installations that extend any of those abstract classes might be affected.

Migration

Add the same param and return type declarations the interface does.

.. index:: FAL, PHP-API, NotScanned, ext:core

sabbelasichon commented 8 months ago

Have a look at https://github.com/rectorphp/rector/blob/main/docs/rector_rules_overview.md#addreturntypedeclarationbasedonparentclassmethodrector

sabbelasichon commented 6 months ago

Feel free to reopen, if you see a need. But i guess it is very unlikely that someone is extending these classes in the wild