Bug #68606
closedcopyRecords removes NULL values from original data
100%
Description
While doing a copy & paste of a record after another record the method copyRecord makes use of some values of the target record.
These values are coming from fields that can be defined in TCA so basically there can be any field of any type and configuration as a source for such a value.
This works perfectly fine unless the value is NULL, which is a rare case in TYPO3 but can happen already.
While preparing the structured content stuff I noticed that necessary NULL values in colPos fields got replaced with 0.
This is due to the fact that copyRecords uses isset(), which returns FALSE for NULL values, where array_key_exists() would be the way to go.
Since this is not just a problem of structured content, I would like to fix that within a separate patch, to make sure the method as such is working as expected.
Updated by Gerrit Code Review over 9 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42081
Updated by Gerrit Code Review over 9 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42081
Updated by Gerrit Code Review over 9 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42081
Updated by Gerrit Code Review over 9 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/42081
Updated by Susanne Moog over 9 years ago
- Target version changed from 7.4 (Backend) to 7.5
Updated by Jo Hasenau over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset a7707bcb6e9cc144a6e8dea902d9fad0aef3f063.
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed