mattcasters commented on PR #8247: URL: https://github.com/apache/hop/pull/8247#issuecomment-5551808061
The issue really is that SWT widgets never accept null as a possible string. I do somewhat disagree with @hansva in [this comment](https://github.com/apache/hop/pull/8173#issuecomment-5463492494) that there ever is a scenario where null isn't equivalent to empty string ""... in Hop metadata. In the cases where we WANT to make the distinction, we have explicit options and parameters (set empty string). In other words, serialization of `<tag/>` for empty strings where we wouldn't serialize the value when null is IMO a bug. My recommendation would be to stop `GuiCompositeWidgets` from turning `null` into `""` when the user doesn't touch the widget, since that seems to be the source of this whole ticket. -- 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]
