Skip to content

Commit

Permalink
Merge pull request #6318 from likangning93/clippingPlanesDeprecations
Browse files Browse the repository at this point in the history
Added deprecation warnings for ClippingPlaneCollection
  • Loading branch information
Hannah authored Mar 9, 2018
2 parents 6d55f29 + a4441a1 commit 27567dd
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions Source/Core/ClippingPlaneCollection.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ define([
'./defaultValue',
'./defined',
'./defineProperties',
'./deprecationWarning',
'./destroyObject',
'./DeveloperError',
'./FeatureDetection',
Expand All @@ -36,6 +37,7 @@ define([
defaultValue,
defined,
defineProperties,
deprecationWarning,
destroyObject,
DeveloperError,
FeatureDetection,
Expand Down Expand Up @@ -249,6 +251,7 @@ define([
};
plane.index = newPlaneIndex;
} else {
deprecationWarning('ClippingPlaneCollection.add', 'Ability to use Plane objects with ClippingPlaneCollection.add is deprecated and will be removed in Cesium 1.45. Please use ClippingPlane objects instead.');
this._containsUntrackablePlanes = true;
}
setIndexDirty(this, newPlaneIndex);
Expand Down Expand Up @@ -680,6 +683,7 @@ define([
* @deprecated
*/
ClippingPlaneCollection.isSupported = function() {
deprecationWarning('ClippingPlaneCollection.isSupported', 'ClippingPlaneCollection.isSupported is deprecated and will be removed in Cesium 1.45. Clipping Planes are now supported on all platforms capable of running Cesium.');
return true;
};

Expand Down

0 comments on commit 27567dd

Please sign in to comment.