#50189 get_status in monitor directly prints
Closed: duplicate 3 months ago by mreynolds. Opened a year ago by firstyear.

Issue Description

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

a year ago

Metadata Update from @mreynolds:
- Issue priority set to: minor
- Issue set to the milestone: 1.4.3 (was: 1.4.1)

5 months ago

I accidentally fixed this in #50545 :-) Closing this as a duplicate...

Metadata Update from @mreynolds:
- Issue close_status updated to: duplicate
- Issue status updated to: Closed (was: Open)

3 months ago

Login to comment on this ticket.