zabbix_action.py: mediatypeid must not exist in update_operations, an…

…d if operationtype=notify_all_involved then opmessage_usr and opmessage_grp must not be used (Zabbix 6.0).

plugins/modules/zabbix_action.py

@@ -900,6 +900,7 @@ def _construct_parameters(self, **kwargs):
             _params.pop('ack_shortdata', None)
 
         if LooseVersion(self._zbx_api_version) >= LooseVersion('6.0'):
+            _params['update_operations'] = kwargs.get('update_operations')
             if 'update_operations' in _params and not isinstance(_params.get('update_operations', None), type(None)):
                 _params.pop('acknowledge_operations', None)
             elif isinstance(_params.get('acknowledge_operations', None), list):
@@ -1392,10 +1393,17 @@ def construct_the_data(self, operations):
             }
 
             # Send Message type
-            if constructed_operation['operationtype'] in ('0', '11'):
+            if constructed_operation['operationtype'] == '0':
                 constructed_operation['opmessage'] = self._construct_opmessage(op)
                 constructed_operation['opmessage_usr'] = self._construct_opmessage_usr(op)
                 constructed_operation['opmessage_grp'] = self._construct_opmessage_grp(op)
+                if LooseVersion(self._zbx_api_version) >= LooseVersion('6.0'):
+                    constructed_operation['opmessage'].pop('mediatypeid')
+
+            if constructed_operation['operationtype'] == '12':
+                constructed_operation['opmessage'] = self._construct_opmessage(op)
+                if LooseVersion(self._zbx_api_version) >= LooseVersion('6.0'):
+                    constructed_operation['opmessage'].pop('mediatypeid')
 
             # Send Command type
             if constructed_operation['operationtype'] == '1':