Skip to content

Conversation

graalvmbot
Copy link
Collaborator

@graalvmbot graalvmbot commented Aug 16, 2022

See the JavaDoc of the new class FieldValueTransformer for the API description.

FieldValueTransformer replaces the CustomFieldValueProvider/CustomFieldValueComputer/CustomFieldValueTransformer that we had in RecomputeValueField for the kind Custom. But more importantly from an API point of view, the transformer can now also be registered from a Feature, i.e., without the need to use the substitution mechanism.

For simplicity, the actual implementation of transformations did not change. It is still done via the substitution universe, i.e, the field value transformer are still invoked from ComputedValueField. This PR also does not remove existing RecomputeFieldValue annotations in our code base.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Aug 16, 2022
@graalvmbot graalvmbot force-pushed the cwi/GR-39852-FieldValueTransformer branch 2 times, most recently from cc4e3b3 to 8d61e31 Compare August 18, 2022 22:16
@graalvmbot graalvmbot force-pushed the cwi/GR-39852-FieldValueTransformer branch from 8d61e31 to 837030b Compare August 19, 2022 02:31
@graalvmbot graalvmbot merged commit 9971b5c into master Aug 19, 2022
@graalvmbot graalvmbot deleted the cwi/GR-39852-FieldValueTransformer branch August 19, 2022 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant