You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now editable message is determined from a header instead of as a field in the entity (db). On the client side, this detail is hidden for reads only, and editableMessage is populated when messages are searched for. But, when persisting a message, the reverse is not true: clients must know to add this as a header.
I would add this logic to ConversionUtility so that going from an EsbMessage adds a header entity for the editableMessage field on the message model. Similarly, when reading, I would also move this logic to ConversionUtility from the entity to the EsbMessage.
Of course, maybe there is a better way. But for example, in EsbMessageAdminServiceImpl:
The extra setter calls are maybe pointing out that convertToEsbMessage is not doing it's job completely. Maybe it would be better if the returned EsbMessage(s) could be fully populated after that call, so this instead looked like:
Right now editable message is determined from a header instead of as a field in the entity (db). On the client side, this detail is hidden for reads only, and editableMessage is populated when messages are searched for. But, when persisting a message, the reverse is not true: clients must know to add this as a header.
I would add this logic to ConversionUtility so that going from an EsbMessage adds a header entity for the editableMessage field on the message model. Similarly, when reading, I would also move this logic to ConversionUtility from the entity to the EsbMessage.
Of course, maybe there is a better way. But for example, in EsbMessageAdminServiceImpl:
The extra setter calls are maybe pointing out that
convertToEsbMessage
is not doing it's job completely. Maybe it would be better if the returned EsbMessage(s) could be fully populated after that call, so this instead looked like:The text was updated successfully, but these errors were encountered: