Index: leahi_dialin/utils/checks.py =================================================================== diff -u -r2138d06d100fdcf23f2e9069f35ee2fdee62008f -r03dc6c3ff80d7f76a5b6438ba067e02a37e2ce93 --- leahi_dialin/utils/checks.py (.../checks.py) (revision 2138d06d100fdcf23f2e9069f35ee2fdee62008f) +++ leahi_dialin/utils/checks.py (.../checks.py) (revision 03dc6c3ff80d7f76a5b6438ba067e02a37e2ce93) @@ -14,14 +14,24 @@ # ############################################################################ -def check_broadcast_interval_override_ms(ms: int): +from logging import Logger + + +def check_broadcast_interval_override_ms(ms: int, logger: Logger): """ Checks whether a given broadcast interval override (in ms) is valid. @param ms: (int) number of ms being used to override a broadcast interval @return: True if valid, False if not """ - result = False if ms > 0 and ms % 50 == 0: - result = True - return result + return True + elif ms <= 0: + logger.debug("ms must be positive non-zero.") + return False + elif ms % 50 != 0: + logger.debug("ms must be a multiple of 50.") + return False + else: + logger.debug("ms must be an integer.") + return False