tonyair/silverstripe-userforms
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-userforms.git synced 2024-10-22 17:05:42 +02:00
Code Issues Projects Releases Wiki Activity

FIX Ensure duplicated multiple option field is written (has an ID) before duplicating options

Browse Source
This commit is contained in:
Robbie Averill 2018-03-23 11:53:43 +13:00 committed by Daniel Hensby
parent 3e71cf7840
commit 453a35e114
No known key found for this signature in database
GPG Key ID: D8DEBC4C8E7BC8B9
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
code/Extension/UserFormFieldEditorExtension.php
View File

@ -225,6 +225,7 @@ class UserFormFieldEditorExtension extends DataExtension
// List of EditableFieldGroups, where the key of the array is the ID of the old end group // List of EditableFieldGroups, where the key of the array is the ID of the old end group
$fieldGroups = []; $fieldGroups = [];
foreach ($oldPage->Fields() as $field) { foreach ($oldPage->Fields() as $field) {
/** @var EditableFormField $newField */
$newField = $field->duplicate(false); $newField = $field->duplicate(false);
$newField->ParentID = $this->owner->ID; $newField->ParentID = $this->owner->ID;
$newField->ParentClass = $this->owner->ClassName; $newField->ParentClass = $this->owner->ClassName;

2
code/Model/EditableFormField/EditableMultipleOptionField.php
View File

@ -123,7 +123,7 @@ class EditableMultipleOptionField extends EditableFormField
$manyMany = null; $manyMany = null;
} }
$clonedNode = parent::duplicate($doWrite, $manyMany); $clonedNode = parent::duplicate(true, $manyMany);
foreach ($this->Options() as $field) { foreach ($this->Options() as $field) {
/** @var EditableOption $newField */ /** @var EditableOption $newField */