Project

General

Profile

Actions

Bug #94307

closed

f:form.hidden inconsistency for disabled attribute

Added by Aimeos no-lastname-given about 3 years ago. Updated 8 days ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Fluid
Target version:
-
Start date:
2021-06-11
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
easy
Is Regression:
Sprint Focus:
On Location Sprint

Description

These tags will all throw an exception:

<f:form.hidden name="test" value="1" disabled />
<f:form.hidden name="test" value="1" disabled="disabled" />
Exception:

Undeclared arguments passed to ViewHelper TYPO3\CMS\Fluid\ViewHelpers\Form\HiddenViewHelper: disabled. Valid arguments are: additionalAttributes, data, aria, name, value, property, class, dir, id, lang, style, title, accesskey, tabindex, onclick

Other f:form.* tags allow disabled="disabled" so the behavior is inconsistent .

Actions #1

Updated by Georg Ringer about 1 month ago

  • Status changed from New to Needs Feedback

thanks for creating the issue.
can you tell if this feature request/bug report is still valid and if so what is a use case for that? hidden fields are not shown to the user, why defining as disabled?

Actions #2

Updated by Aimeos no-lastname-given about 1 month ago

Georg Ringer wrote in #note-1:

can you tell if this feature request/bug report is still valid and if so what is a use case for that? hidden fields are not shown to the user, why defining as disabled?

Sometimes, what should be sent to the server depends on what the user selects in the frontend. For example, there are two options and both require sending additional data to the server in a hidden field and by default, one or both must be disabled to avoid wrong processing at the server side.

Actions #3

Updated by Garvin Hicking about 1 month ago

  • Complexity set to easy
  • Sprint Focus set to On Location Sprint
Actions #4

Updated by Garvin Hicking about 1 month ago

  • Status changed from Needs Feedback to Accepted
Actions #5

Updated by Garvin Hicking about 1 month ago

Should be adressed via https://github.com/TYPO3/Fluid/pull/859

Hope we'll remember to close this ticket when it gets merged 😬

Actions #6

Updated by Simon Praetorius about 1 month ago

  • Status changed from Accepted to Resolved

This has been fixed in Fluid Standalone and will be included in the next minor release of Fluid.

Actions #7

Updated by Benni Mack 8 days ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF