Skip to content

Commit ce771c6

Browse files
lolli42fe-hicking
authored andcommitted
[!!!][TASK] Remove registered 'class' handling in abstract form VH
Do not handle manually registered 'class' attribute in setErrorClassAttribute() anymore. Resolves: #105485 Related: #105377 Related: #104412 Releases: main Change-Id: Id688b797b2342daa950eaa4aa4a44d5b152020a9 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/86801 Reviewed-by: Garvin Hicking <[email protected]> Tested-by: Garvin Hicking <[email protected]> Tested-by: core-ci <[email protected]> Tested-by: Anja Leichsenring <[email protected]> Reviewed-by: Anja Leichsenring <[email protected]> Reviewed-by: Nikita Hovratov <[email protected]> Tested-by: Nikita Hovratov <[email protected]>
1 parent 51dc4c0 commit ce771c6

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

typo3/sysext/core/Documentation/Changelog/14.0/Breaking-105377-DeprecatedFunctionalityRemoved.rst

+3
Original file line numberDiff line numberDiff line change
@@ -236,6 +236,9 @@ The following fallbacks have been removed:
236236
- Allowing instantiation of :php:`\TYPO3\CMS\Core\Imaging\IconRegistry` in ext_localconf.php
237237
- Accepting a comma-separated list of fields as value for the `columnsOnly` parameter
238238
- Support for extbase repository magic :php:`findByX()`, :php:`findOneByX()` and :php:`countByX()` methods
239+
- Fluid view helpers that extend :php:`\TYPO3\CMS\Fluid\ViewHelpers\Form\AbstractFormFieldViewHelper`
240+
should no longer register :html:`class` attribute and should rely on attribute auto registration
241+
for the error class to be added correctly.
239242

240243
The following upgrade wizards have been removed:
241244

typo3/sysext/fluid/Classes/ViewHelpers/Form/AbstractFormFieldViewHelper.php

+1-6
Original file line numberDiff line numberDiff line change
@@ -325,12 +325,7 @@ protected function isObjectAccessorMode(): bool
325325
*/
326326
protected function setErrorClassAttribute(): void
327327
{
328-
if ($this->hasArgument('class')) {
329-
// @deprecated: Fallback layer for VH's that register 'class' as argument
330-
// via registerUniversalTagAttributes(). Remove in v14. Make
331-
// elseif() below if().
332-
$cssClass = $this->arguments['class'] . ' ';
333-
} elseif (isset($this->additionalArguments['class'])) {
328+
if (isset($this->additionalArguments['class'])) {
334329
$cssClass = $this->additionalArguments['class'] . ' ';
335330
} else {
336331
$cssClass = '';

0 commit comments

Comments
 (0)