lib389 is designed to have objects encapsulate behaviour, and to move data to locations that determine how to handle that data. The get_status function of the Monitor types doesn't do this correctly opting to print directly. The issue is if someone used get_status and was expecting a dictionary as a return, and wanted to format the output in a certain way, consume the output for a lookup, or wanted to write their own scripts to extend this functionality.
get_status on monitor types (and it's callers) need to returne a dict, and then the caller should be responsible for the display of the data.
Metadata Update from @mreynolds:
- Custom field origin adjusted to None
- Custom field reviewstatus adjusted to None
- Issue set to the milestone: 1.4.1
to comment on this ticket.