[TASK] Fluid Viewhelpers will be declared final in v12
Communication patch for an upcoming v12 change. We've had the situation a couple of times that core changed Fluid Viewhelpers to adapt them to current needs, keeping the template arguments, but changing internal handling and method signatures. It happened that developers then stepped up with "This broke my own Viewhelper!" and we had to act upon. The ImageViewHelper for instance has been a victim of this. From a framework point of view, Viewhelpers are specific implementations tailored for specific use. In most cases, they're small codewise. Extending these classes and changing their scope should not be supported. Developers should extend a matching abstract instead of extending specific VH's. The patch brings an Important.rst to pitch an upcoming v12 change that will declare all Viewhelpers final. The core currently of course also extends a couple of VH's itself, but this is too late for v11 now to change and adapt. Instead, we'll take the time in v12 to resolve these cases and will have a look at the abstract classes as well to see which VH functionality should be included in abstracts to simplify re-use and transition for extension authors and to sharpen core API. Resolves: #95298 Releases: master Change-Id: Ia3312829fad94e7429c0989f3aa3554c3cf944ec Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/71164 Tested-by:Oliver Bartsch <bo@cedev.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Helmut Hummel <typo3@helhum.io> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Mathias Brodala <mbrodala@pagemachine.de> Reviewed-by:
Helmut Hummel <typo3@helhum.io> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
Please register or sign in to comment