processMultiple in SaveHandlerImpl should not delete existing contentNode!
--------------------------------------------------------------------------
Key: MAGNOLIA-3066
URL: http://jira.magnolia-cms.com/browse/MAGNOLIA-3066
Project: Magnolia
Issue Type: Bug
Components: admininterface
Affects Versions: 4.2.3
Reporter: Will Scheidegger
Assignee: Philipp Bärfuss
When the data of a VALUETYPE_MULTIPLE control is saved, the data is handled by
the processMultiple() method in the SaveHandlerImpl class. This class first
deletes the contentNode with the name of the control, then creates a new one
and fills in the node data from the submitted dialog form.
The problem here is that by creating a new contentNode we also get a new
UUID... and when activating the modified node containing this contentNode you
will end up with a node containing multiple contentNodes with the name of the
control.
The correct way of handling this would be (if I'm not totally mistaken):
1. leave the contentNode as it is
2. remove all its nodeDatas
3. add the new nodeDatas from the submitted dialog form
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.magnolia-cms.com/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
----------------------------------------------------------------
For list details see
http://www.magnolia-cms.com/home/community/mailing-lists.html
To unsubscribe, E-mail to: <[email protected]>
----------------------------------------------------------------
