Project

General

Profile

Actions

Bug #83403

closed

<span></span> tag not filled in form extension for type radio

Added by Eckard Gehrke over 6 years ago. Updated over 5 years ago.

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

100%

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

Description

Hi,
when upgrading from 8.7.8 to 8.7.9 I get the following error:

In 8.7.8 my form was rendered to:

<label class="form-check-label check0">
<input class="xlarge form-check-input" type="radio" name="tx_form_formframework[anfrage][radiobutton-1]" value="Informatik" checked="checked">
<span> Informatik</span>
</label>

whereas in 8.7.9 and current 8.7 branch the result is an empty <span> tag:

<label class="form-check-label check0">
<input class="xlarge form-check-input" type="radio" name="tx_form_formframework[anfrage][radiobutton-1]" value="Informatik" checked="checked">
<span></span>
</label>

I could trace it down to changes in form -> Classes -> Service and ViewHelpers but could not break it down further to a single change. Check0 and counting the radiobuttons in the group is an extension of my own, which still works fine

Best regrads
Eckard.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #82210: ext:form multi checkbox values containing . or & result in empty frontend labelsClosed2017-08-25

Actions
Actions #1

Updated by Eckard Gehrke over 6 years ago

  • Priority changed from Should have to -- undefined --
  • Target version deleted (next-patchlevel)

Hi,

after some more search I found the issue:
If you use own partials RadioButton.html needs to be adopted before you upgrade to 8.7.9
The span tag should now read:

<span>{formvh:translateElementProperty(element: element, property: '{0: \'options\', 1: value}')}</span>

because TranslationService was changed.
Maybe you give some hints on which other own partials should be adopted before the upgrade.

Best regards
Eckard.

PS: Can't set it myelf to resolved.

Actions #2

Updated by Oliver Hader over 6 years ago

  • Priority changed from -- undefined -- to Should have
Actions #3

Updated by Ralf Zimmermann over 6 years ago

  • Related to Bug #82210: ext:form multi checkbox values containing . or & result in empty frontend labels added
Actions #4

Updated by Gerrit Code Review over 6 years ago

  • Status changed from New to Under Review

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/55306

Actions #5

Updated by Gerrit Code Review over 6 years ago

Patch set 2 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/55306

Actions #6

Updated by Gerrit Code Review about 6 years ago

Patch set 3 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/55306

Actions #7

Updated by Gerrit Code Review about 6 years ago

Patch set 4 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/55306

Actions #8

Updated by Susanne Moog about 6 years ago

  • Target version set to 8.7.10
Actions #9

Updated by Ralf Zimmermann about 6 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF