Task #48148
Allow PropertyMapping of strings to arrays
| Status: | Resolved | Start date: | 2013-05-13 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Christian Müller | % Done: | 100% |
|
| Category: | - | |||
| Target version: | - |
Description
Primary usecase is empty string to empty array for ajax generated options where "nothing" is also possible, but comma delimited strings could also happen with certain UI elements.
Related issues
Associated revisions
[FEATURE] Allow String source for ArrayConverter
ArrayConverter can now convert strings to arrays. It uses
explode internally with a default delimiter of ",".
The delimiter is configurable with the key "stringDelimiter" in
propertyMapping configuration for the ArrayConverter.
Change-Id: I376b8c3027095a88dc5ade094198fef04dd50174
Resolves: #48148
Releases: master
History
#1 Updated by Christian Müller about 2 years ago
- Tracker changed from Bug to Task
#2 Updated by Gerrit Code Review about 2 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20737
#3 Updated by Gerrit Code Review about 2 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20737
#4 Updated by Gerrit Code Review about 2 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20737
#5 Updated by Helmut Hummel about 2 years ago
Christian Mueller wrote:
Primary usecase is empty string to empty array for ajax generated options where "nothing" is also possible, but comma delimited strings could also happen with certain UI elements.
I suggest to split this in two patch sets. One that can only convert empty strings to empty arrays (which fixes a bug with the checkbox viewhelper in older branches, see #47832) and in a second iteration add the feature to be able to explode strings by delimiter to an array. The latter can then only go into the master branch.
#6 Updated by Gerrit Code Review about 2 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20737
#7 Updated by Gerrit Code Review almost 2 years ago
Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20737
#8 Updated by Gerrit Code Review almost 2 years ago
Patch set 6 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/20737
#9 Updated by Christian Müller almost 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset commit:af810292032f7adf7437bc9fd385a8dc2ec04339.