Automation raises errors when default snmp options is set to none

[bmfmancini]

Hey Guys I have been playing around with the latest release and found a issue with autom8

If you do a network scan with the below settings

You will net the below errors

2020-12-06  21:29:58 - CMDPHP SQL Backtrace:   (/poller_automation.php[358]:discoverDevices(),  /poller_automation.php[790]:db_execute(),  /lib/database.php[213]:db_execute_prepared())
--
2020-12-06 21:29:58 - CMDPHP ERROR: A DB Exec Failed!, Error:  Incorrect integer value: '' for column  `cacti`.`automation_devices`.`snmp_version` at row 1
2020-12-06 21:29:58 - CMDPHP SQL Backtrace:   (/poller_automation.php[358]:discoverDevices(),  /poller_automation.php[790]:db_execute(),  /lib/database.php[213]:db_execute_prepared())
2020-12-06 21:29:58 - CMDPHP ERROR: A DB Exec Failed!, Error:  Incorrect integer value: '' for column  `cacti`.`automation_devices`.`snmp_version` at row 1
2020-12-06 21:29:58 - CMDPHP SQL Backtrace:   (/poller_automation.php[358]:discoverDevices(),  /poller_automation.php[790]:db_execute(),  /lib/database.php[213]:db_execute_prepared())
2020-12-06 21:29:58 - CMDPHP ERROR: A DB Exec Failed!, Error:  Incorrect integer value: '' for column  `cacti`.`automation_devices`.`snmp_version` at row 1
2020-12-06 21:29:58 - CMDPHP SQL Backtrace:   (/poller_automation.php[358]:discoverDevices(),  /poller_automation.php[790]:db_execute(),  /lib/database.php[213]:db_execute_prepared())
2020-12-06 21:29:58 - CMDPHP ERROR: A DB Exec Failed!, Error:  Incorrect integer value: '' for column  `cacti`.`automation_devices`.`snmp_version` at row 1
2020-12-06 21:29:58 - CMDPHP SQL Backtrace:   (/poller_automation.php[358]:discoverDevices(),  /poller_automation.php[790]:db_execute(),  /lib/database.php[213]:db_execute_prepared())
2020-12-06 21:29:58 - CMDPHP SQL Backtrace:   (/poller_automation.php[358]:discoverDevices(),  /poller_automation.php[790]:db_execute(),  /lib/database.php[213]:db_execute_prepared())
2020-12-06 21:29:58 - CMDPHP ERROR: A DB Exec Failed!, Error:  Incorrect integer value: '' for column  `cacti`.`automation_devices`.`snmp_version` at row 1
2020-12-06 21:29:58 - CMDPHP ERROR: A DB Exec Failed!, Error:  Incorrect integer value: '' for column  `cacti`.`automation_devices`.`snmp_version` at row 1

[TheWitness]

Thanks for testing Sean. I'm not surprised in that it's totally biased towards snmp today. Good catch.

[netniV]

This shouldn't occur if the snmp version is being read from an existing device, which is what the code suggests. The snmp_version in the db for both the host table and the automation table is a tinyint. This would suggest that the blank value is being sent in code.