Merge pull request Esri#909 from Saulzi/setOpacity

setOpacity on RasterLayers
jgravois · Jan 4, 2017 · 5d9bd90 · 5d9bd90
2 parents 3f45418 + 9d3023f
commit 5d9bd90
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/spec/Layers/RasterLayerSpec.js b/spec/Layers/RasterLayerSpec.js
@@ -0,0 +1,9 @@
+/* eslint-env mocha */
+/* eslint-disable handle-callback-err */
+describe('L.esri.RasterLayer', function () {
+  it('should not error when calling setOpacity when _currentImage is null', function () {
+    var layer = new L.esri.RasterLayer();
+    layer._currentImage = null;
+    expect(function () { layer.setOpacity(0.5); }).to.not.throw();
+  });
+});
diff --git a/src/Layers/RasterLayer.js b/src/Layers/RasterLayer.js
@@ -122,7 +122,9 @@ export var RasterLayer = L.Layer.extend({
 
   setOpacity: function (opacity) {
     this.options.opacity = opacity;
-    this._currentImage.setOpacity(opacity);
+    if (this._currentImage) {
+      this._currentImage.setOpacity(opacity);
+    }
     return this;
   },