Feature #85033

Form plugin - Finishers : Hardcoded label for empty default value

Added by RVVN no-lastname-given about 1 year ago. Updated 3 months ago.

Status:
Resolved
Priority:
Should have
Assignee:
-
Category:
Form Framework
Target version:
-
Start date:
2018-05-17
Due date:
% Done:

100%

PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Hi,

In the backend form for finishers overriding, the part of labels displaying the default value is hardcoded with "[Empty]" in case of this value is empty (file : /typo3/sysext/form/Classes/Hooks/DataStructureIdentifierHook.php).

It would be helpful if this part of labels could use translation ;)

Associated revisions

Revision 54851545 (diff)
Added by Ralf Zimmermann 3 months ago

[BUGFIX] Change finisher override display behavior

Show all possible finisher options which can be overridden within the
form plugin even if if they are not part of a form definition.

Resolves: #86635
Resolves: #85033
Releases: master, 9.5
Change-Id: Ie5ce8e2fb97e2c3fde92cbcb405d77818d1c7bda
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/60577
Tested-by: TYPO3com <>
Tested-by: Mathias Brodala <>
Tested-by: Susanne Moog <>
Reviewed-by: Mathias Brodala <>
Reviewed-by: Susanne Moog <>

Revision 85d7c84b (diff)
Added by Ralf Zimmermann 3 months ago

[BUGFIX] Change finisher override display behavior

Show all possible finisher options which can be overridden within the
form plugin even if if they are not part of a form definition.

Resolves: #86635
Resolves: #85033
Releases: master, 9.5
Change-Id: Ie5ce8e2fb97e2c3fde92cbcb405d77818d1c7bda
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/60597
Tested-by: TYPO3com <>
Tested-by: Ralf Zimmermann <>
Tested-by: Susanne Moog <>
Reviewed-by: Ralf Zimmermann <>
Reviewed-by: Susanne Moog <>

History

#1 Updated by Ralf Zimmermann 12 months ago

  • Tracker changed from Bug to Feature

#2 Updated by Riccardo De Contardi 9 months ago

still present on latest master (line 251 of /typo3/sysext/form/Classes/Hooks/DataStructureIdentifierHook.php)

#3 Updated by Gerrit Code Review 3 months ago

  • Status changed from New to Under Review

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

#4 Updated by Gerrit Code Review 3 months ago

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

#5 Updated by Gerrit Code Review 3 months ago

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

#6 Updated by Gerrit Code Review 3 months ago

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

#7 Updated by Gerrit Code Review 3 months ago

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

#8 Updated by Gerrit Code Review 3 months ago

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

#9 Updated by Gerrit Code Review 3 months ago

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

#10 Updated by Anonymous 3 months ago

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

Also available in: Atom PDF