Hi, I'm not sure whether the current handling of log entries and status between the result object and the log entries is a good thing.
For one, all log entries added to the result are logged to a logger - I think this can easily result in having the statements logged out more than once and it breaks the contract of having result a simple data object. In addition, when a log message is added, it's level is evaluated to automatically set the status - if a HC check is just doing log statements for informational purpuse with level that leads to automatic status setting, the HC can't set the status to a lower level afterwards. I think there shouldn't be any additional logic in the result class except from just adding the log entries to a collection. Regards Carsten -- Carsten Ziegeler [email protected]
