Skip to content
Snippets Groups Projects
Commit 5c86fdb1 authored by Owen Bush's avatar Owen Bush Committed by Owen Bush
Browse files

Issue #3085952 by owenbush, achoura: Prevent circular dependency when selecting a destination field

parent d6c8e503
No related branches found
No related tags found
No related merge requests found
...@@ -129,6 +129,12 @@ class FieldInheritanceForm extends EntityForm { ...@@ -129,6 +129,12 @@ class FieldInheritanceForm extends EntityForm {
$instance_fields = array_keys($this->entityFieldManager->getFieldDefinitions('eventinstance', 'eventinstance')); $instance_fields = array_keys($this->entityFieldManager->getFieldDefinitions('eventinstance', 'eventinstance'));
$instance_fields = array_combine($instance_fields, $instance_fields); $instance_fields = array_combine($instance_fields, $instance_fields);
// You should never be able to use the inherited field as part of an
// inheritance as that creates an infinite loop.
if (!empty($field_inheritance->id()) && !empty($instance_fields[$field_inheritance->id()])) {
unset($instance_fields[$field_inheritance->id()]);
}
$form['entityField'] = [ $form['entityField'] = [
'#type' => 'select', '#type' => 'select',
'#title' => $this->t('Entity/Instance Field'), '#title' => $this->t('Entity/Instance Field'),
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment