gitgabrio commented on PR #6105: URL: https://github.com/apache/incubator-kie-drools/pull/6105#issuecomment-2413793666
Hi @samuel-beniamin thanks for the PR. I tried to understand better the overall context, and I'm sharing my thoughts: please correct me if/where I'm wrong: I may misread something. 1. by itself, I think the fix could be right 2. but those AfterProcessDrgElements / addCallback / addCallbackForModel are used/needed only by the signavio module 3. but being defined/ implemented in the "core" DMNCompilerImpl, those somehow "pollute" it with something that, IMO, has no reason to be there So, I was thinking to 1. change `DMNCompilerImpl.processDrgElements` from private to protected, to be overridable 2. get rid of all signavio-specific stuff from it (double check also the tests) 4. create a signavio-specific subclass of `DMNCompilerImpl` that add the behavior required by signavio 5. put your modifications in that subclass The idea would be to have `DMNCompilerImpl` dealing only with "core" responsibilities, and move implementation-specific ones (e.g. signavio) to dedicated subclasses Wdyt ? Does this make sense ? Do you think you could do that ? @baldimir @yesamer -- 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]
