Bug #79294: Persisting Traversable DomainObjectInterface broken - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #79294

closed

Persisting Traversable DomainObjectInterface broken

Added by Mathias Brodala over 7 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Mathias Brodala

Category:

Extbase

Target version:

next-patchlevel

Start date:

2017-01-12

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

Currently objects which implement the Traversable interface are always persisted as CSV of their values after iterating. This change introduced this change of behavior with TYPO3 7.0.0. This is broken in various ways:

The type of the relation is ignored which could be e.g. ColumnMap::RELATION_HAS_ONE, thus multivalue does not make sense.
If an object implements DomainObjectInterface it must always be persisted using its identifier.

While I'm not sure how to handle the first case the second one should be fixed right away since it breaks persistence of objects which implement both Traversable as well as DomainObjectInterface

Files

test_traversable.zip (6.24 KB) test_traversable.zip

Test extension

Mathias Brodala, 2017-01-23 10:15

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Gerrit Code Review over 7 years ago

Status changed from In Progress 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 https://review.typo3.org/51287

Actions

Copy link

Updated by Gerrit Code Review over 7 years ago

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

Actions

Copy link

Updated by Mathias Brodala about 7 years ago

File test_traversable.zip test_traversable.zip added

Here's a test extension to demonstrate the issue. It contains a model Main and has one property pointing to a Related model which implements Traversable (through IteratorAggregate and a Generator).

When running the typo3/cli_dispatch.phpsh extbase test:persisttraversabledomainobject command, a row is added to each table, but the related field of tx_testtraversable_domain_model_main contains 0 instead of the UID of the newly inserted tx_testtraversable_domain_model_related row.

Actions

Copy link

Updated by Gerrit Code Review about 7 years ago

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51446

Actions

Copy link