changes for timeInterval

CHANGES.md

@@ -52,6 +52,7 @@ Change Log
 * Fixed crash when animating a glTF model with a single keyframe. [#6422](https://github.com/AnalyticalGraphicsInc/cesium/pull/6422)
 * 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 ISO 8601 string with improper formatting. [#6164](https://github.com/AnalyticalGraphicsInc/cesium/issues/6164)
 * Improved rendering of glTF models that don't contain normals with a temporary unlit shader workaround. [#6501](https://github.com/AnalyticalGraphicsInc/cesium/pull/6501)
 * Fixed rendering of glTF models with emissive-only materials. [#6501](https://github.com/AnalyticalGraphicsInc/cesium/pull/6501)
 

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,7 +128,9 @@ 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 match proper formatting.
      *
      * @param {Object} options Object with the following properties:
      * @param {String} options.iso8601 An ISO 8601 interval.
@@ -143,6 +147,9 @@ define([
         //>>includeEnd('debug');
 
         var dates = options.iso8601.split('/');
+        if (dates.length !== 2) {
+            throw new DeveloperError('options.iso8601 is an invalid ISO 8601 interval.');
+        }
         var start = JulianDate.fromIso8601(dates[0]);
         var stop = JulianDate.fromIso8601(dates[1]);
         var isStartIncluded = defaultValue(options.isStartIncluded, true);

Specs/Core/TimeIntervalSpec.js

@@ -100,6 +100,12 @@ defineSuite([
         expect(interval.data).toEqual(data);
     });
 
+    it('fromIso8601 throws error when given Invalid ISO 8601 date.', function() {
+        expect(function() {
+            TimeInterval.fromIso8601({ iso8601 : '2020-08-29T00:00:00+00:00' });
+        }).toThrowDeveloperError();
+    });
+
     it('toIso8601 works', function() {
         var isoDate1 = '0950-01-02T03:04:05Z';
         var isoDate2 = '0950-01-03T03:04:05Z';