In ExternalEntityType class, the public method setFieldMapperId() (currently only used in tests in this module) changes the field mapper plugin id while it does not reset the fieldMapperPlugin member which may have been initialized with a different field mapper id:
/**
* {@inheritdoc}
*/
public function setFieldMapperId($field_mapper_id) {
$this->field_mapper_id = $field_mapper_id;
}
Therefore, a developer may think calling setFieldMapperId() changes the field mapper but it does not and there can be a discrepancy between what is returned by ->getFieldMapperId() and ->getFieldMapper()->getPluginId() which could lead to unexpected behaviors. As well, the field mapper config is not changed and may not correspond to the field mapper.
/**
* {@inheritdoc}
*/
public function setFieldMapperId($field_mapper_id) {
$this->field_mapper_id = $field_mapper_id;
$this->fieldMapperPlugin = NULL;
$this->field_mapper_config = [];
}
Review the logic and merge.
None.
None.
None.
Note: I'll prepare a PR.
Needs review
2.0
Code
Issue #3425428: setFieldMapperId should reset fieldMapperPlugin member
Automatically closed - issue fixed for 2 weeks with no activity.