From 1c33157fb81f8150b5385cbab1f5cd019cca31d0 Mon Sep 17 00:00:00 2001 From: Pete Richards Date: Thu, 6 Jul 2017 13:09:13 -0700 Subject: [PATCH] [Telem] Handle no range values Update the telemetry adapter to gracefully handle cases where a range value is not found via hints. This allows telemetry objects that don't have ranges to still work with some old style displays --- platform/telemetry/src/TelemetryCapability.js | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/platform/telemetry/src/TelemetryCapability.js b/platform/telemetry/src/TelemetryCapability.js index f784f59ffc..466609dc64 100644 --- a/platform/telemetry/src/TelemetryCapability.js +++ b/platform/telemetry/src/TelemetryCapability.js @@ -213,7 +213,8 @@ define( var metadata = telemetryAPI.getMetadata(domainObject); var defaultDomain = metadata.valuesForHints(['domain'])[0].source; - var defaultRange = metadata.valuesForHints(['range'])[0].source; + var defaultRange = metadata.valuesForHints(['range'])[0]; + defaultRange = defaultRange ? defaultRange.source : undefined; var isLegacyProvider = telemetryAPI.findRequestProvider(domainObject) === telemetryAPI.legacyProvider; @@ -275,7 +276,8 @@ define( var metadata = telemetryAPI.getMetadata(domainObject); var defaultDomain = metadata.valuesForHints(['domain'])[0].source; - var defaultRange = metadata.valuesForHints(['range'])[0].source; + var defaultRange = metadata.valuesForHints(['range'])[0]; + defaultRange = defaultRange ? defaultRange.source : undefined; var isLegacyProvider = telemetryAPI.findSubscriptionProvider(domainObject) === telemetryAPI.legacyProvider;