Skip to content

Commit

Permalink
[Alerting] Fix validation support for nested IErrorObjects (#62833) (#…
Browse files Browse the repository at this point in the history
…62995)

* [Alerting] Add validation support for nested IErrorObjects

* Typecheck fix

* Fix recursion crash when errors are strings

* Typecheck fix
  • Loading branch information
Zacqary authored Apr 9, 2020
1 parent 69fc518 commit 5e6c6f0
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
* you may not use this file except in compliance with the Elastic License.
*/
import React, { useCallback, useReducer, useState } from 'react';
import { isObject } from 'lodash';
import { FormattedMessage } from '@kbn/i18n/react';
import {
EuiTitle,
Expand Down Expand Up @@ -83,7 +84,7 @@ export const AlertAdd = ({
...(alertType ? alertType.validate(alert.params).errors : []),
...validateBaseProperties(alert).errors,
} as IErrorObject;
const hasErrors = !!Object.keys(errors).find(errorKey => errors[errorKey].length >= 1);
const hasErrors = parseErrors(errors);

const actionsErrors: Array<{
errors: IErrorObject;
Expand Down Expand Up @@ -213,3 +214,9 @@ export const AlertAdd = ({
</EuiPortal>
);
};

const parseErrors: (errors: IErrorObject) => boolean = errors =>
!!Object.values(errors).find(errorList => {
if (isObject(errorList)) return parseErrors(errorList as IErrorObject);
return errorList.length >= 1;
});
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,14 @@ import {
} from '@elastic/eui';
import { builtInAggregationTypes } from '../constants';
import { AggregationType } from '../types';
import { IErrorObject } from '../../types';
import { ClosablePopoverTitle } from './components';
import './of.scss';

interface OfExpressionProps {
aggType: string;
aggField?: string;
errors: { [key: string]: string[] };
errors: IErrorObject;
onChangeSelectedAggField: (selectedAggType?: string) => void;
fields: Record<string, any>;
customAggTypesOptions?: {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,12 @@ import {
} from '@elastic/eui';
import { builtInComparators } from '../constants';
import { Comparator } from '../types';
import { IErrorObject } from '../../types';
import { ClosablePopoverTitle } from './components';

interface ThresholdExpressionProps {
thresholdComparator: string;
errors: { [key: string]: string[] };
errors: IErrorObject;
onChangeSelectedThresholdComparator: (selectedThresholdComparator?: string) => void;
onChangeSelectedThreshold: (selectedThreshold?: number[]) => void;
customComparators?: {
Expand Down
2 changes: 1 addition & 1 deletion x-pack/plugins/triggers_actions_ui/public/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -111,5 +111,5 @@ export interface AlertTypeModel {
}

export interface IErrorObject {
[key: string]: string[];
[key: string]: string | string[] | IErrorObject;
}

0 comments on commit 5e6c6f0

Please sign in to comment.