Task #29039

Feature #28486: Don't clone, use explicit change tracking instead

Change default policy for Doctrine to DEFERRED_EXPLICIT

Added by Karsten Dambekalns almost 4 years ago. Updated almost 4 years ago.

Status:Resolved Start date:2011-08-17
Priority:Should have Due date:
Assigned To:Karsten Dambekalns % Done:

100%

Category:Persistence
Target version:TYPO3 Flow Base Distribution - 1.0 beta 1
Sprint: Has patch:
PHP Version: Complexity:

Description

  • change tracking policy -> explicit
  • no more clone PersistentObjectConverter
  • Repository adjustments
    • add($obj) -> throw exception when object is not new
    • remove($obj) -> throw exception when object is not known in persistence BE
    • update($obj) -> throw exception when object is not known in persistence BE, call PersistenceBackend::persist()

Biggest change: without update() call no changes will be persisted, no more automatic persistence.


Related issues

duplicated by TYPO3.Flow - Feature #28137: Do not use clone in PersistentObject ParamConverter Closed 2011-07-13 2011-07-13
blocked by TYPO3.Flow - Feature #29038: Set useful defaults for cascade attributes in Doctrine Resolved 2011-08-17
blocks TYPO3.Flow - Task #29040: Adjust documentation to new persistence behavior Resolved 2011-08-17

Associated revisions

Revision 49f4d8f4
Added by Karsten Dambekalns almost 4 years ago

[!!!][TASK] Change default policy for Doctrine to DEFERRED_EXPLICIT

This makes Doctrine use deferred explicit change tracking by default.
The PersistentObjectConverter does no longer clone objects and any
changed objects that should be persisted must be handed to update()
on their repository.

Change-Id: I05a07f9cbb4cadba138ad6774a309b8f627e22d2
Resolves: #29039
Related: #28486

History

#1 Updated by Mr. Hudson almost 4 years ago

  • Status changed from Accepted to Under Review

Patch set 1 of change I05a07f9cbb4cadba138ad6774a309b8f627e22d2 has been pushed to the review server.
It is available at http://review.typo3.org/4470

#2 Updated by Mr. Hudson almost 4 years ago

Patch set 2 of change I05a07f9cbb4cadba138ad6774a309b8f627e22d2 has been pushed to the review server.
It is available at http://review.typo3.org/4470

#3 Updated by Mr. Hudson almost 4 years ago

Patch set 3 of change I05a07f9cbb4cadba138ad6774a309b8f627e22d2 has been pushed to the review server.
It is available at http://review.typo3.org/4470

#4 Updated by Mr. Hudson almost 4 years ago

Patch set 4 of change I05a07f9cbb4cadba138ad6774a309b8f627e22d2 has been pushed to the review server.
It is available at http://review.typo3.org/4470

#5 Updated by Karsten Dambekalns almost 4 years ago

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

Also available in: Atom PDF