Rework required using standard functionality

[ashemedai]

SUMMARY

Ansible provides various mechanisms to describe what parameters are required. This commit converts them, reducing custom logic.

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME

community.zabbix.zabbix_script

ADDITIONAL INFORMATION

Ansible provides ways to specify dependencies between module options. This change converts this module to using them. This also reduces required parameters that the HTTP API does not mandate, e.g. deletion of a script.

Before it was necessary to specify something like this:

- name: Delete test script
  community.zabbix.zabbix_script:
    name: Test script
    script_type: webhook
    command: ""
    state: absent

Now it suffices to use:

- name: Delete test script
  community.zabbix.zabbix_script:
    name: Test script
    state: absent

Some output to show the requirements are still in place with this:

TASK [zabbix : Create a new SSH script] **************************************************************************************************
fatal: [localhost]: FAILED! => {"changed": false, "msg": "script_type is ssh but all of the following are missing: authtype, username"}

TASK [zabbix : Create a new SSH password script] *****************************************************************************************
fatal: [localhost]: FAILED! => {"changed": false, "msg": "authtype is password but all of the following are missing: password"}

TASK [zabbix : Create a new SSH key script] **********************************************************************************************
fatal: [localhost]: FAILED! => {"changed": false, "msg": "authtype is public_key but all of the following are missing: publickey, privatekey"}

TASK [zabbix : Create a new telnet script] ***********************************************************************************************
fatal: [localhost]: FAILED! => {"changed": false, "msg": "script_type is telnet but all of the following are missing: password"}