Bug #26851

ProxyNode should not automatically persist newNode if it has been cloned beforehand.

Added by Sebastian Kurfuerst about 4 years ago. Updated about 4 years ago.

Status:Resolved Start date:2011-05-17
Priority:Must have Due date:
Assigned To:Sebastian Kurfuerst % Done:

100%
Category:Node handling
Target version:Base Distribution - Sprint 7

Description

If one has a $proxyNode, clones it, and changes a property on that node, these changes are automatically persisted. This also happens if one clones the ProxyNode. In this case, however, it needs to lose its connection with the Persistence Layer (and a call to Repository::update() is needed again).

Related issues

related to TYPO3.Neos - Bug #26849: Saving of Content Elements does not work anymore Resolved 2011-05-17

Associated revisions

Revision 1c510dfd
Added by Sebastian Kurfuerst about 4 years ago

[BUGFIX] ProxyNode only persists the newNode if it has not been cloned

If the ProxyNode has been cloned, we only add the newNode when calling
NodeRepository::update(ProxyNode), as only then the changes should be
persisted.

Change-Id: I93eaddf0131d1c7ab8ca29108d87976063f0951b
Resolves: #26851

History

#1 Updated by Sebastian Kurfuerst about 4 years ago

  • Subject changed from ProxyNode should not automatically persist newNode on creation to ProxyNode should not automatically persist newNode if it has been cloned beforehand.

#2 Updated by Mr. Hudson about 4 years ago

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

#3 Updated by Sebastian Kurfuerst about 4 years ago

  • Status changed from Accepted to Under Review
  • % Done changed from 60 to 90

#4 Updated by Mr. Hudson about 4 years ago

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

#5 Updated by Mr. Hudson about 4 years ago

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

#6 Updated by Sebastian Kurfuerst about 4 years ago

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

Also available in: Atom PDF