Project

General

Profile

Actions

Bug #82243

closed

Adding properties to "fluidAdditionalAttributes" in form definition leads to lost properties

Added by Denis Mir about 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Form Framework
Target version:
-
Start date:
2017-08-30
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.1
Tags:
Complexity:
Is Regression:
Sprint Focus:
Remote Sprint

Description

I got the following problem.

I have a custom form element definition in my form configuration:

            CustomEmail:
              __inheritances:
                10: 'TYPO3.CMS.Form.prototypes.standard.formElementsDefinition.Text'
              renderingOptions:
                templateName: 'Text'
              label: 'Email'
              defaultValue: ''
              properties:
                fluidAdditionalAttributes:
                  placeholder: Email
                elementClassAttribute: 'email'
              validators:
                -
                  identifier: NotEmpty
                -
                  identifier: EmailAddress

Normally I'm using it like this in one of my form definitions:

      -
        type: CustomEmail
        identifier: email

This does work fine.

Then I'm trying to add a property to "fluidAdditionalAttributes" in another form definition like so (in this case the email field should be required):

      -
        type: CustomEmail
        identifier: email
        properties:
          fluidAdditionalAttributes:
            required: required

This leads me to an added "required" property in the frontend rendering but the "placeholder" property defined in the form configuration for my "CustomEmail" type is gone.

Actions #1

Updated by Björn Jacob about 7 years ago

  • Sprint Focus set to Remote Sprint
Actions #2

Updated by Gerrit Code Review about 7 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54124

Actions #3

Updated by Gerrit Code Review almost 7 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54124

Actions #4

Updated by Gerrit Code Review almost 7 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54124

Actions #5

Updated by Gerrit Code Review almost 7 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54964

Actions #6

Updated by Ralf Zimmermann almost 7 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #7

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF