Browse Source

Revert "Timerange: Fixes a bug where custom time ranges didn't respect UTC (#18217)" (#18246)

This reverts commit 0752a09f92b995a1e90f7bda526d634a2992b6ab.
kay delaney 6 years ago
parent
commit
6144c0f51d

+ 0 - 12
packages/grafana-data/src/utils/datemath.ts

@@ -5,18 +5,6 @@ import { TimeZone } from '../types';
 
 const units: DurationUnit[] = ['y', 'M', 'w', 'd', 'h', 'm', 's'];
 
-export function isMathString(text: string | DateTime | Date): boolean {
-  if (!text) {
-    return false;
-  }
-
-  if (typeof text === 'string' && (text.substring(0, 3) === 'now' || text.includes('||'))) {
-    return true;
-  } else {
-    return false;
-  }
-}
-
 /**
  * Parses different types input to a moment instance. There is a specific formatting language that can be used
  * if text arg is string. See unit tests for examples.

+ 2 - 11
packages/grafana-ui/src/components/TimePicker/TimePicker.tsx

@@ -14,7 +14,6 @@ import { rawToTimeRange } from './time';
 
 // Types
 import { TimeRange, TimeOption, TimeZone, TIME_FORMAT, SelectableValue } from '@grafana/data';
-import { isMathString } from '@grafana/data/src/utils/datemath';
 
 export interface Props {
   value: TimeRange;
@@ -124,21 +123,13 @@ export class TimePicker extends PureComponent<Props, State> {
     const { isCustomOpen } = this.state;
     const options = this.mapTimeOptionsToSelectableValues(selectTimeOptions);
     const currentOption = options.find(item => isTimeOptionEqualToTimeRange(item.value, value));
-
-    const isUTC = timeZone === 'utc';
-    const adjustedTo = isUTC ? value.to.utc() : value.to.local();
-    const adjustedFrom = isUTC ? value.from.utc() : value.from.local();
-    const adjustedTimeRange = {
-      to: isMathString(value.raw.to) ? value.raw.to : adjustedTo,
-      from: isMathString(value.raw.from) ? value.raw.from : adjustedFrom,
-    };
-    const rangeString = rangeUtil.describeTimeRange(adjustedTimeRange);
+    const rangeString = rangeUtil.describeTimeRange(value.raw);
 
     const label = (
       <>
         {isCustomOpen && <span>Custom time range</span>}
         {!isCustomOpen && <span>{rangeString}</span>}
-        {isUTC && <span className="time-picker-utc">UTC</span>}
+        {timeZone === 'utc' && <span className="time-picker-utc">UTC</span>}
       </>
     );
     const isAbsolute = isDateTime(value.raw.to);

+ 3 - 5
public/app/features/dashboard/components/DashNav/DashNavTimeControls.tsx

@@ -1,6 +1,6 @@
 // Libaries
 import React, { Component } from 'react';
-import { toUtc, dateMath } from '@grafana/data';
+import { toUtc } from '@grafana/data';
 
 // Types
 import { DashboardModel } from '../../state';
@@ -61,11 +61,9 @@ export class DashNavTimeControls extends Component<Props> {
     const panel = dashboard.timepicker;
     const hasDelay = panel.nowDelay && timeRange.raw.to === 'now';
 
-    const adjustedFrom = dateMath.isMathString(timeRange.raw.from) ? timeRange.raw.from : timeRange.from;
-    const adjustedTo = dateMath.isMathString(timeRange.raw.to) ? timeRange.raw.to : timeRange.to;
     const nextRange = {
-      from: adjustedFrom,
-      to: hasDelay ? 'now-' + panel.nowDelay : adjustedTo,
+      from: timeRange.raw.from,
+      to: hasDelay ? 'now-' + panel.nowDelay : timeRange.raw.to,
     };
 
     this.timeSrv.setTime(nextRange);