Project

General

Profile

Actions

Task #104442

open

Improve ViewHelper tests for "required"

Added by Stephan Großberndt 8 days ago. Updated 8 days ago.

Status:
Under Review
Priority:
Should have
Category:
Fluid
Target version:
-
Start date:
2024-07-19
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
13
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

ViewHelpers that do not register "required" as an argument (e.g. `PasswordViewHelper`) pass any value since Fluid v2.12.0 (see https://github.com/TYPO3/Fluid/pull/859).

ViewHelpers that register "required" as an argument (e.g. `TextfieldViewHelper`) interpret the specified value as a bool and if true always render `required="required"` and omit the attribute if false.

Once this registration will be removed, `required="false"` will be passed through as an arbitrary value and rendered in the browser.

While `required="false"` is technically incorrect, browsers correctly interpret the sole existence of the attribute as "is required" no matter which value is set.

Improve the tests to make sure this case is handled correctly once the registration for "required" in the ViewHelpers is removed:
Usage of `required="false"` should be logged as deprecated and simply be removed from the ViewHelper usage.


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Task #104436: Use standard attributes in EXT:feloginResolvedStephan Großberndt2024-07-19

Actions
Related to TYPO3 Core - Bug #104441: Correct values for attribute "required" in felogin templatesResolvedStephan Großberndt2024-07-19

Actions
Actions

Also available in: Atom PDF