From 4c22b6fcefd2045f614437693936e0337373d44f Mon Sep 17 00:00:00 2001
From: Alexander Zobnin <alexanderzobnin@gmail.com>
Date: Thu, 4 Feb 2021 13:58:25 +0300
Subject: [PATCH] Option for using zabbix mapping, closes #1128

---
 docs/sources/configuration/provisioning.md       |  2 ++
 src/datasource-zabbix/partials/query.editor.html |  5 +++++
 src/datasource-zabbix/query.controller.ts        |  4 +++-
 src/datasource-zabbix/responseHandler.ts         | 10 ++++++----
 src/datasource-zabbix/types.ts                   |  9 +++++----
 5 files changed, 21 insertions(+), 9 deletions(-)

diff --git a/docs/sources/configuration/provisioning.md b/docs/sources/configuration/provisioning.md
index b12a59b..4a32094 100644
--- a/docs/sources/configuration/provisioning.md
+++ b/docs/sources/configuration/provisioning.md
@@ -41,6 +41,8 @@ datasources:
     disableReadOnlyUsersAck: true
     # Disable time series data alignment
     disableDataAlignment: false
+    # Use value mapping from Zabbix
+    useZabbixValueMapping: true
   version: 1
   editable: false
 
diff --git a/src/datasource-zabbix/partials/query.editor.html b/src/datasource-zabbix/partials/query.editor.html
index 1679ba6..ec91acd 100644
--- a/src/datasource-zabbix/partials/query.editor.html
+++ b/src/datasource-zabbix/partials/query.editor.html
@@ -288,6 +288,11 @@
           checked="ctrl.target.options.showDisabledItems"
           on-change="ctrl.onQueryOptionChange()">
         </gf-form-switch>
+        <gf-form-switch class="gf-form" label-class="width-10"
+          label="Use Zabbix value mapping"
+          checked="ctrl.target.options.useZabbixValueMapping"
+          on-change="ctrl.onQueryOptionChange()">
+        </gf-form-switch>
         <gf-form-switch class="gf-form" label-class="width-10"
           label="Disable data alignment"
           checked="ctrl.target.options.disableDataAlignment"
diff --git a/src/datasource-zabbix/query.controller.ts b/src/datasource-zabbix/query.controller.ts
index 391b25a..afb9d08 100644
--- a/src/datasource-zabbix/query.controller.ts
+++ b/src/datasource-zabbix/query.controller.ts
@@ -4,7 +4,7 @@ import * as c from './constants';
 import * as utils from './utils';
 import * as metricFunctions from './metricFunctions';
 import * as migrations from './migrations';
-import { ShowProblemTypes } from './types';
+import { ShowProblemTypes, ZabbixMetricsQuery } from './types';
 import { CURRENT_SCHEMA_VERSION } from '../panel-triggers/migrations';
 import { getTemplateSrv, TemplateSrv } from '@grafana/runtime';
 
@@ -29,6 +29,7 @@ function getTargetDefaults() {
       showDisabledItems: false,
       skipEmptyValues: false,
       disableDataAlignment: false,
+      useZabbixValueMapping: false,
     },
     table: {
       'skipEmptyValues': false
@@ -456,6 +457,7 @@ export class ZabbixQueryController extends QueryCtrl {
     const metricOptionsMap = {
       showDisabledItems: "Show disabled items",
       disableDataAlignment: "Disable data alignment",
+      useZabbixValueMapping: "Use Zabbix value mapping",
     };
 
     const problemsOptionsMap = {
diff --git a/src/datasource-zabbix/responseHandler.ts b/src/datasource-zabbix/responseHandler.ts
index f0d9ec6..34a03bf 100644
--- a/src/datasource-zabbix/responseHandler.ts
+++ b/src/datasource-zabbix/responseHandler.ts
@@ -2,7 +2,8 @@ import _ from 'lodash';
 import TableModel from 'grafana/app/core/table_model';
 import * as c from './constants';
 import * as utils from './utils';
-import { ArrayVector, DataFrame, DataQuery, Field, FieldType, MutableDataFrame, TIME_SERIES_TIME_FIELD_NAME, TIME_SERIES_VALUE_FIELD_NAME } from '@grafana/data';
+import { ArrayVector, DataFrame, Field, FieldType, MutableDataFrame, TIME_SERIES_TIME_FIELD_NAME, TIME_SERIES_VALUE_FIELD_NAME } from '@grafana/data';
+import { ZabbixMetricsQuery } from './types';
 
 /**
  * Convert Zabbix API history.get response to Grafana format
@@ -60,7 +61,8 @@ function convertHistory(history, items, addHostName, convertPointCallback) {
   });
 }
 
-export function seriesToDataFrame(timeseries, target: DataQuery, valueMappings?: any[], fieldType?: FieldType): DataFrame {
+export function seriesToDataFrame(timeseries, target: ZabbixMetricsQuery, valueMappings?: any[], fieldType?: FieldType): DataFrame {
+  console.log(target);
   const { datapoints, scopedVars, target: seriesName, item } = timeseries;
 
   const timeFiled: Field = {
@@ -122,8 +124,8 @@ export function seriesToDataFrame(timeseries, target: DataQuery, valueMappings?:
 
     // Try to use value mapping from Zabbix
     const mappings = utils.getValueMapping(item, valueMappings);
-    if (mappings) {
-      console.log(`Datasource: value mapping detected`);
+    if (mappings && target.options?.useZabbixValueMapping) {
+      console.log(`Datasource: using Zabbix value mapping`);
       valueFiled.config.mappings = mappings;
     }
   }
diff --git a/src/datasource-zabbix/types.ts b/src/datasource-zabbix/types.ts
index da924c8..299aa9d 100644
--- a/src/datasource-zabbix/types.ts
+++ b/src/datasource-zabbix/types.ts
@@ -37,10 +37,10 @@ export interface ZabbixMetricsQuery extends DataQuery {
   triggers: { minSeverity: string; acknowledged: boolean; count: number; };
   queryType: string;
   datasourceId: number;
-  group: { filter: string; name: string; };
-  host: { filter: string; name: string; };
-  application: { filter: string; name: string; };
-  item: { filter: string; name: string; };
+  group: { filter: string; name?: string; };
+  host: { filter: string; name?: string; };
+  application: { filter: string; name?: string; };
+  item: { filter: string; name?: string; };
   textFilter: string;
   mode: number;
   itemids: number[];
@@ -62,6 +62,7 @@ export interface ZabbixQueryOptions {
   showDisabledItems?: boolean;
   skipEmptyValues?: boolean;
   disableDataAlignment?: boolean;
+  useZabbixValueMapping?: boolean;
   // Problems options
   minSeverity?: number;
   sortProblems?: string;