populate_hostid_cache mixes up host's name and visible name

[blu3bird]

populate_hostid_cache uses the visible name of the host while everything else uses the real name.
It should also not exclude not monitored(disabled) host, otherwise define_host_monitoring_status is useless in bulk operations.

Here's my quick and dirty fix which seems to work just fine:

index fd32527..0fa9976 100644
--- a/zabbix_cli/cli.py
+++ b/zabbix_cli/cli.py
@@ -7672,11 +7672,10 @@ class zabbixcli(cmd.Cmd):
         try:
             temp_dict = {}
             
-            data = self.zapi.host.get(output=['hostid','name'],
-                                      monitored_hosts=True)
+            data = self.zapi.host.get(output=['hostid','host'])
 
             for host in data:
-                temp_dict[host['hostid']] = host['name'] 
+                temp_dict[host['hostid']] = host['host'] 
 
             return temp_dict
          ```

[usit-gd]

Thanks for the feedback.
+1. This is a confirmed bug and will be fixed with the next release.

The problem is the monitored_hosts=True attribute. name == host if it is not defined according to the API doc: https://www.zabbix.com/documentation/3.0/manual/api/reference/host/object#host