changes for timeInterval

Author: hanbollar

CHANGES.md

@@ -29,6 +29,7 @@ Change Log
 * Fixed crash bug in PolylineCollection when a polyline was updated and removed at the same time. [#6455](https://github.com/AnalyticalGraphicsInc/cesium/pull/6455)
 * Fixed Imagery Layers Texture Filters Sandcastle example. [#6472](https://github.com/AnalyticalGraphicsInc/cesium/pull/6472).
 * Fixed a bug causing Cesium 3D Tilesets to not clip properly when tiles were unloaded and reloaded. [#6484](https://github.com/AnalyticalGraphicsInc/cesium/issues/6484)
+* Fixed `TimeInterval` so now it throws if `fromIso8601` is given an iso8601 input that doesn't have '/' for proper formatting. 
 
 ### 1.44 - 2018-04-02
 

Source/Core/TimeInterval.js

@@ -3,13 +3,15 @@ define([
         './defaultValue',
         './defined',
         './defineProperties',
+        './DeveloperError',
         './freezeObject',
         './JulianDate'
     ], function(
         Check,
         defaultValue,
         defined,
         defineProperties,
+        DeveloperError,
         freezeObject,
         JulianDate) {
     'use strict';
@@ -126,10 +128,12 @@ define([
     };
 
     /**
-     * Creates a new instance from an {@link http://en.wikipedia.org/wiki/ISO_8601|ISO 8601} interval.
+     * Creates a new instance from a {@link http://en.wikipedia.org/wiki/ISO_8601|ISO 8601} interval.
+     *
+     * @throws DeveloperError if options.iso8601 does not contain a '/'.
      *
      * @param {Object} options Object with the following properties:
-     * @param {String} options.iso8601 An ISO 8601 interval.
+     * @param {String} options.iso8601 An ISO 8601 interval. Must contain a '/'.
      * @param {Boolean} [options.isStartIncluded=true] <code>true</code> if <code>options.start</code> is included in the interval, <code>false</code> otherwise.
      * @param {Boolean} [options.isStopIncluded=true] <code>true</code> if <code>options.stop</code> is included in the interval, <code>false</code> otherwise.
      * @param {Object} [options.data] Arbitrary data associated with this interval.
@@ -140,6 +144,9 @@ define([
         //>>includeStart('debug', pragmas.debug);
         Check.typeOf.object('options', options);
         Check.typeOf.string('options.iso8601', options.iso8601);
+        if (options.iso8601.indexOf('/') === -1) {
+            throw new DeveloperError('iso8601 must contain a / per proper formatting. See {@link http://en.wikipedia.org/wiki/ISO_8601|ISO 8601}.');
+        }
         //>>includeEnd('debug');
 
         var dates = options.iso8601.split('/');