Bug #39445
PropertyMapper chokes on Objects that are already converted inside an collection
| Status: | Resolved | Start date: | 2012-12-25 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assigned To: | Marc Neuhaus | % Done: | 100% |
|
| Category: | Property | |||
| Target version: | TYPO3 Flow Base Distribution - 2.0 beta 1 | |||
| PHP Version: | Complexity: | |||
| Has patch: | No | Affected Flow version: | Git 1.2 (master) |
Description
The PropertyMapper still chokes on objects which are already converted if it's inside an collection/array.
The PropertyMapper should skip the mapping if the source is already an instance of the target.
Subtasks
Related issues
Associated revisions
[BUGFIX] PropertyMapper should be able to work with recursive already-converted objects
If f.e. an array of already converted objects is passed into the PropertyMapper, a fatal error
will be thrown.
Thus, we move the check from convert() to doMapping() such that it works recursively.
Change-Id: I67fe649c1ee15d38a277227dc353844c0fb17b4b
Resolves: #39445
Releases: 1.2
History
#1 Updated by Gerrit Code Review about 3 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 http://review.typo3.org/13404
#2 Updated by Gerrit Code Review almost 3 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13404
#3 Updated by Gerrit Code Review almost 3 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13404
#4 Updated by Marc Neuhaus almost 3 years ago
- Status changed from Under Review to Resolved
Applied in changeset 8ac03660b9c5835d67f9f514b3035fa2e9eb2dd5.