gnodet commented on PR #22932: URL: https://github.com/apache/camel/pull/22932#issuecomment-4377241041
_Claude Code on behalf of Guillaume Nodet_ @davsclaus — one question for you in this PR: `NamedNode.getLevel()` and `getParentId()` treat when/otherwise as "shallow" (they don't contribute a level). This means: ``` choice → level 1 when → level 1 (same as choice — shallow) to → level 2 ``` But for try/catch, doCatch is a regular ProcessorDefinition and properly increments the level: ``` doTry → level 1 doCatch → level 2 to → level 3 ``` Now that `getChildren()` makes when/otherwise proper children in the model tree, this flattening seems inconsistent. Should we remove the shallow behavior to make choice/when consistent with try/catch? See the TODO comment in `NamedNode.java` for details. -- 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]
