Bug #12190
Aggregate root detection does not work for non-standard repositories
Status: | Resolved | Start date: | 2011-01-13 | |
---|---|---|---|---|
Priority: | Must have | Due date: | ||
Assigned To: | Karsten Dambekalns | % Done: | 100% |
|
Category: | Persistence | |||
Target version: | TYPO3 Flow Base Distribution - 1.0 beta 1 | Estimated time: | 2.00 hours | |
PHP Version: | Complexity: | |||
Has patch: | Affected Flow version: |
Description
If a repository and the model it holds don't match via the naming conventions, the ReflectionService does not detect that the model is an aggregate root.
Associated revisions
[!!!][BUGFIX] Resolve non-standard model/repository bindings
If a repository and the model it holds don't match via the naming
conventions, the ReflectionService did not detect that the model is
an aggregate root.
This involves a breaking change for those that are using the property
$objectType in their repositories. That needs to be changed to a
constant named ENTITY_CLASSNAME:
protected $objectType = 'FooBar';
needs to become
const ENTITY_CLASSNAME = 'FooBar';
As a bonus this change adds a new method getEntityClassName() to the
base repositories which can be used to find out which entity is managed
by a repository instance.
Change-Id: I7920587c4060f6583276b05e8686a997e113b7b7
Fixes: #12190
History
#1 Updated by Karsten Dambekalns over 4 years ago
- Priority changed from Should have to Must have
#2 Updated by Mr. Hudson about 4 years ago
- Status changed from Accepted to Under Review
Patch set 1 of change I7920587c4060f6583276b05e8686a997e113b7b7 has been pushed to the review server.
It is available at http://review.typo3.org/2968
#3 Updated by Mr. Hudson about 4 years ago
Patch set 2 of change I7920587c4060f6583276b05e8686a997e113b7b7 has been pushed to the review server.
It is available at http://review.typo3.org/2968
#4 Updated by Mr. Hudson about 4 years ago
Patch set 3 of change I7920587c4060f6583276b05e8686a997e113b7b7 has been pushed to the review server.
It is available at http://review.typo3.org/2968
#5 Updated by Karsten Dambekalns about 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset ebcddb263e3d0766d6e351ffabbb906d76469abb.