yesamer opened a new issue, #1150: URL: https://github.com/apache/incubator-kie-issues/issues/1150
The `ImportDMNResolverUtil` class holds the logic that resolves a DMN import. The current logic has several issues that need to be addressed: - An old custom `drools:modelName` is used in the logic. That attribute is a custom definition (i.e. not present in the DMN specs) no longer used in our implementation since DMN 1.1 version. As the XSD that defines that attribute is no longer present in our codebase, the only reference of that attribute is present in the 1.1 `TImport` class, and our editors are not managing it at all, we agreed to remove any reference to that attribute in the logic - The new logic, should rely on the `namespace` only to resolve the imported DMN file. The `namespace` of the Definition metamodel doesn't ensure this attribute is unique over all DMNs of the entire project, so a collision may happen. In addition, the Import metamodel paragraph says: `The namespace value should be globally unique`, which in our interpretation doesn't force all the `namespace` to be unique but the required conditions to have a correct import resolution, without saying how to manage a possible collision. In case of collisions (i.e. multiple DMN files with the same namespace in the Definition metamodel), the idea is to rely on the `locationURI` attribute to correctly resolve the DMN model to be imported. The `locationURI` is optional, so the logic should manage such a scenario as well. - Improve the reported messages in case is not possible to resolve the DMN Import @baldimir @gitgabrio -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
