ammachado commented on PR #24545: URL: https://github.com/apache/camel/pull/24545#issuecomment-4921549949
Correction / course change to my previous comment: to keep this PR self-contained and mergeable independently of #24541, I've **kept** the existing `getProperty`-based local-key check in `TuiUserConfig.livesInLocal()` rather than switching it to `containsKey()`. The `getProperty(key) != null` check already handles the empty-valued-key case correctly (an empty property line loads as `""`, not `null`), so there is no functional gap on this branch. The switch to `OrderedProperties.containsKey()` will come in a **follow-up PR once #24541 lands**. This PR now has no dependency on #24541. Still included here: the `FolderInputPopup` folder-precedence fix (`lastFolder` > `defaultFolder` > `user.dir`) with `FolderInputPopupTest`, plus a `TuiUserConfig` regression test for the empty-valued local override. Full TUI module test suite is green. _Claude Code on behalf of Adriano Machado (ammachado)_ -- 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]
