Skip to content

Commit 8a115a5

Browse files
committed
fix unknown types and remove lodash.get
1 parent 00b3fc8 commit 8a115a5

File tree

6 files changed

+10
-9
lines changed

6 files changed

+10
-9
lines changed

src/plugins/vis_type_vislib/public/pie_fn.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ interface Arguments {
3333
}
3434

3535
interface RenderValue {
36-
visData: any;
36+
visData: unknown;
3737
visType: string;
3838
visConfig: PieVisParams;
3939
}

src/plugins/vis_type_vislib/public/to_ast.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@
1818
*/
1919

2020
import moment from 'moment';
21-
import { get } from 'lodash';
2221

2322
import { VisToExpressionAst, getVisSchemas } from '../../visualizations/public';
2423
import { buildExpression, buildExpressionFunction } from '../../expressions/public';
@@ -78,11 +77,11 @@ export const toExpressionAst: VisToExpressionAst<BasicVislibParams> = async (vis
7877
const usedValueAxis = (visConfig.valueAxes || []).find(
7978
(valueAxis) => valueAxis.id === seriesParam.valueAxis
8079
);
81-
if (get(usedValueAxis, 'scale.mode') === 'percentage') {
80+
if (usedValueAxis?.scale.mode === 'percentage') {
8281
yDimension.format = { id: 'percent' };
8382
}
8483
}
85-
if (get(visConfig, 'gauge.percentageMode') === true) {
84+
if (visConfig?.gauge.percentageMode === true) {
8685
yDimension.format = { id: 'percent' };
8786
}
8887
});

src/plugins/vis_type_vislib/public/types.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -90,6 +90,9 @@ export interface BasicVislibParams extends CommonVislibParams {
9090
labels: Labels;
9191
thresholdLine: ThresholdLine;
9292
valueAxes: ValueAxis[];
93+
gauge: {
94+
percentageMode: boolean;
95+
};
9396
grid: {
9497
categoryLines: boolean;
9598
valueAxis?: string;

src/plugins/vis_type_vislib/public/vis_controller.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ export const createVislibVisController = (
134134
}
135135

136136
mountLegend(
137-
visData: any,
137+
visData: unknown,
138138
{ legendPosition, addLegend }: BasicVislibParams | PieVisParams,
139139
fireEvent: IInterpreterRenderHandlers['event'],
140140
uiState?: PersistedState

src/plugins/vis_type_vislib/public/vis_renderer.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
* under the License.
1818
*/
1919

20-
import { get } from 'lodash';
2120
import React, { lazy } from 'react';
2221
import { render, unmountComponentAtNode } from 'react-dom';
2322

@@ -33,8 +32,8 @@ function shouldShowNoResultsMessage(visData: any, visType: string): boolean {
3332
return false;
3433
}
3534

36-
const rows: object[] | undefined = get(visData, 'rows');
37-
const isZeroHits = get(visData, 'hits') === 0 || (rows && !rows.length);
35+
const rows: object[] = visData?.rows ?? [];
36+
const isZeroHits = visData?.hits === 0 || (rows && !rows.length);
3837

3938
return Boolean(isZeroHits);
4039
}

src/plugins/vis_type_vislib/public/vislib/components/legend/legend.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ import { BasicVislibParams } from '../../../types';
3434

3535
export interface VisLegendProps {
3636
vislibVis: any;
37-
visData: any;
37+
visData: unknown;
3838
uiState?: PersistedState;
3939
fireEvent: IInterpreterRenderHandlers['event'];
4040
addLegend: BasicVislibParams['addLegend'];

0 commit comments

Comments
 (0)