From 98025026f7e4eaaae618e5b775910b96b46ec57c Mon Sep 17 00:00:00 2001
From: Alexander Zobnin <alexanderzobnin@gmail.com>
Date: Wed, 23 Dec 2020 12:48:46 +0300
Subject: [PATCH] Filter problems by time range, fix #1094

---
 src/datasource-zabbix/datasource.ts              | 4 ++--
 src/datasource-zabbix/partials/query.editor.html | 6 ++++++
 src/datasource-zabbix/query.controller.ts        | 7 +++----
 src/datasource-zabbix/types.ts                   | 4 ++++
 4 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/src/datasource-zabbix/datasource.ts b/src/datasource-zabbix/datasource.ts
index 640c810..2d6cb08 100644
--- a/src/datasource-zabbix/datasource.ts
+++ b/src/datasource-zabbix/datasource.ts
@@ -453,7 +453,7 @@ export class ZabbixDatasource extends DataSourceApi<ZabbixMetricsQuery, ZabbixDS
     });
   }
 
-  queryProblems(target, timeRange, options) {
+  queryProblems(target: ZabbixMetricsQuery, timeRange, options) {
     const [timeFrom, timeTo] = timeRange;
     const userIsEditor = contextSrv.isEditor || contextSrv.isGrafanaAdmin;
 
@@ -512,7 +512,7 @@ export class ZabbixDatasource extends DataSourceApi<ZabbixMetricsQuery, ZabbixDS
     }
 
     let getProblemsPromise: Promise<ProblemDTO[]>;
-    if (showProblems === ShowProblemTypes.History) {
+    if (showProblems === ShowProblemTypes.History || target.options?.useTimeRange) {
       problemsOptions.timeFrom = timeFrom;
       problemsOptions.timeTo = timeTo;
       getProblemsPromise = this.zabbix.getProblemsHistory(groupFilter, hostFilter, appFilter, proxyFilter, problemsOptions);
diff --git a/src/datasource-zabbix/partials/query.editor.html b/src/datasource-zabbix/partials/query.editor.html
index f01affb..1679ba6 100644
--- a/src/datasource-zabbix/partials/query.editor.html
+++ b/src/datasource-zabbix/partials/query.editor.html
@@ -334,6 +334,12 @@
             </select>
           </div>
         </div>
+        <gf-form-switch class="gf-form"
+          label-class="width-9"
+          label="Use time range"
+          checked="ctrl.target.options.useTimeRange"
+          on-change="ctrl.onQueryOptionChange()">
+        </gf-form-switch>
         <gf-form-switch class="gf-form"
           label-class="width-9"
           label="Hosts in maintenance"
diff --git a/src/datasource-zabbix/query.controller.ts b/src/datasource-zabbix/query.controller.ts
index a8c1b57..f3a3a0e 100644
--- a/src/datasource-zabbix/query.controller.ts
+++ b/src/datasource-zabbix/query.controller.ts
@@ -101,6 +101,7 @@ export class ZabbixQueryController extends QueryCtrl {
   queryOptionsText: string;
   metric: any;
   showQueryOptions: boolean;
+  oldTarget: any;
 
   /** @ngInject */
   constructor($scope, $injector, $rootScope, $sce, templateSrv) {
@@ -370,9 +371,6 @@ export class ZabbixQueryController extends QueryCtrl {
       this.targetChanged();
     }
   }
-  oldTarget(oldTarget: any, target: any) {
-    throw new Error("Method not implemented.");
-  }
 
   onVariableChange() {
     if (this.isContainsVariables()) {
@@ -405,7 +403,7 @@ export class ZabbixQueryController extends QueryCtrl {
   targetChanged() {
     this.initFilters();
     this.parseTarget();
-    this.panelCtrl.refresh();
+    this.refresh();
   }
 
   addFunction(funcDef) {
@@ -466,6 +464,7 @@ export class ZabbixQueryController extends QueryCtrl {
       hostsInMaintenance: "Show hosts in maintenance",
       limit: "Limit problems",
       hostProxy: "Show proxy",
+      useTimeRange: "Use time range",
     };
 
     let optionsMap = {};
diff --git a/src/datasource-zabbix/types.ts b/src/datasource-zabbix/types.ts
index 8c43ff6..da924c8 100644
--- a/src/datasource-zabbix/types.ts
+++ b/src/datasource-zabbix/types.ts
@@ -51,6 +51,8 @@ export interface ZabbixMetricsQuery extends DataQuery {
   tags?: { filter: string; };
   functions: ZabbixMetricFunction[];
   options: ZabbixQueryOptions;
+  // Problems
+  showProblems?: ShowProblemTypes;
   // Deprecated
   hostFilter?: string;
   itemFilter?: string;
@@ -67,6 +69,8 @@ export interface ZabbixQueryOptions {
   hostsInMaintenance?: boolean;
   hostProxy?: boolean;
   limit?: number;
+  useTimeRange?: boolean;
+  severities?: number[];
 }
 
 export interface ZabbixMetricFunction {