mattcasters opened a new pull request, #7956: URL: https://github.com/apache/hop/pull/7956
## What Fixes the duplicate Metadata Injection keys on Simple Mapping (issue #3025). Input and output mappings share `MappingIODefinition`, so after the `HopMetadataProperty` migration the dialog showed two `input_transform` / `output_transform` rows and assigning one overwrote the other. ## How - Add opt-in `injectionKeyPrefix` on `@HopMetadataProperty`. Serialization `key()` / `groupKey()` are unchanged. - Both injection engines (`BeanInjectionInfo` for the dialog, `HopMetadataInjector` for runtime) apply the prefix to descendant keys and group keys. - Simple Mapping uses `INPUT_`, `OUTPUT_`, and `PARAMETERS_` prefixes so keys are unique. - Field mappings and parameter mappings are now injectable as list groups. ## Compatibility - Pipeline XML/JSON is unchanged. - Other transforms keep their published MDI keys (`injectionKeyPrefix` defaults to empty). - Simple Mapping MDI was not usable before, so the new key names are not a regression. ## Tests - Unit tests for prefixing and Simple Mapping injection. - MDI integration test `0061` injects from `files/simple-mapping.xml` and compares the generated pipeline to golden datasets. Fixes #3025 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
