remove crufty method

elastic · Mar 19, 2020 · aa49420 · aa49420
1 parent 7c94a0d
commit aa49420
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 9 deletions.
diff --git a/x-pack/legacy/plugins/maps/public/layers/styles/vector/properties/dynamic_size_property.js b/x-pack/legacy/plugins/maps/public/layers/styles/vector/properties/dynamic_size_property.js
@@ -77,7 +77,7 @@ export class DynamicSizeProperty extends DynamicStyleProperty {
     const rangeFieldMeta = this.getRangeFieldMeta();
     if (this._isSizeDynamicConfigComplete(this._options) && rangeFieldMeta) {
       const halfIconPixels = this.getIconPixelSize() / 2;
-      const targetName = this.getCompletedFieldName();
+      const targetName = this.getFieldName();
       // Using property state instead of feature-state because layout properties do not support feature-state
       mbMap.setLayoutProperty(symbolLayerId, 'icon-size', [
         'interpolate',
@@ -126,7 +126,7 @@ export class DynamicSizeProperty extends DynamicStyleProperty {
     }
 
     return this._getMbDataDrivenSize({
-      targetName: this.getCompletedFieldName(),
+      targetName: this.getFieldName(),
       minSize: this._options.minSize,
       maxSize: this._options.maxSize,
       minValue: rangeFieldMeta.min,

diff --git a/x-pack/legacy/plugins/maps/public/layers/styles/vector/properties/dynamic_style_property.js b/x-pack/legacy/plugins/maps/public/layers/styles/vector/properties/dynamic_style_property.js
@@ -108,13 +108,6 @@ export class DynamicStyleProperty extends AbstractStyleProperty {
     return this._field ? this._field.getName() : '';
   }
 
-  getCompletedFieldName() {
-    if (!this.isComplete()) {
-      return null;
-    }
-    return this.getField().getName();
-  }
-
   isDynamic() {
     return true;
   }