Task #48148

Allow PropertyMapping of strings to arrays

Added by Christian Müller over 8 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Should have
Category:
-
Target version:
-
Start date:
2013-05-13
Due date:
% Done:

100%

Estimated time:

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

Has duplicate TYPO3.Flow - Bug #52068: ArrayConverter should accept empty stringsClosed2013-09-17

Actions
#1

Updated by Christian Müller over 8 years ago

  • Tracker changed from Bug to Task
#2

Updated by Gerrit Code Review over 8 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 over 8 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 over 8 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 over 8 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 over 8 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 about 8 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 about 8 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 about 8 years ago

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

Applied in changeset commit:af810292032f7adf7437bc9fd385a8dc2ec04339.

Also available in: Atom PDF