The type here is incorrect. In _build_monitor_data(), the notificationIDList parameter is a list, which then gets converted to a dict in _convert_monitor_input().
The wrong type here doesn't actually affect anything as it is an empty dict and thus never processed in _convert_monitor_input(), as an empty list would have been as well.
dict_notification_ids = {}
if kwargs["notificationIDList"]:
for notification_id in kwargs["notificationIDList"]:
dict_notification_ids[notification_id] = True
kwargs["notificationIDList"] = dict_notification_ids
But it's still technically the wrong type in that part of the code, and changing other parts of the code could result in that causing a bug at some point.
The type here is incorrect. In
_build_monitor_data(), thenotificationIDListparameter is alist, which then gets converted to adictin_convert_monitor_input().The wrong type here doesn't actually affect anything as it is an empty
dictand thus never processed in_convert_monitor_input(), as an emptylistwould have been as well.But it's still technically the wrong type in that part of the code, and changing other parts of the code could result in that causing a bug at some point.
NB: I haven't tested this change yet.