Skip to content

Commit

Permalink
In scripted fields, unable to switch the Type - getting a console e…
Browse files Browse the repository at this point in the history
…rror which says - Class constructor DecoratedFieldFormat cannot be invoked without 'new' (#59285)

Closes: #58763

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
  • Loading branch information
alexwizp and elasticmachine authored Mar 9, 2020
1 parent 1d0697a commit f0e063c
Show file tree
Hide file tree
Showing 4 changed files with 33 additions and 96 deletions.
56 changes: 33 additions & 23 deletions src/legacy/ui/public/field_editor/field_editor.js
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ import { ScriptingHelpFlyout } from './components/scripting_help';
import { FieldFormatEditor } from './components/field_format_editor';

import { FIELD_TYPES_BY_LANG, DEFAULT_FIELD_TYPES } from './constants';
import { copyField, getDefaultFormat, executeScript, isScriptValid } from './lib';
import { copyField, executeScript, isScriptValid } from './lib';

import { i18n } from '@kbn/i18n';
import { FormattedMessage } from '@kbn/i18n/react';
Expand All @@ -76,6 +76,25 @@ import 'brace/mode/groovy';

const getFieldFormats = () => npStart.plugins.data.fieldFormats;

const getFieldTypeFormatsList = (field, defaultFieldFormat) => {
const fieldFormats = getFieldFormats();
const formatsByType = fieldFormats.getByFieldType(field.type).map(({ id, title }) => ({
id,
title,
}));

return [
{
id: '',
defaultFieldFormat,
title: i18n.translate('common.ui.fieldEditor.defaultFormatDropDown', {
defaultMessage: '- Default -',
}),
},
...formatsByType,
];
};

export class FieldEditor extends PureComponent {
static propTypes = {
indexPattern: PropTypes.object.isRequired,
Expand Down Expand Up @@ -137,11 +156,7 @@ export class FieldEditor extends PureComponent {
field.type = fieldTypes.includes(field.type) ? field.type : fieldTypes[0];

const fieldFormats = getFieldFormats();

const fieldTypeFormats = [
getDefaultFormat(fieldFormats.getDefaultType(field.type, field.esTypes)),
...fieldFormats.getByFieldType(field.type),
];
const DefaultFieldFormat = fieldFormats.getDefaultType(field.type, field.esTypes);

this.setState({
isReady: true,
Expand All @@ -150,14 +165,14 @@ export class FieldEditor extends PureComponent {
errors: [],
scriptingLangs,
fieldTypes,
fieldTypeFormats,
fieldTypeFormats: getFieldTypeFormatsList(field, DefaultFieldFormat),
fieldFormatId: get(indexPattern, ['fieldFormatMap', field.name, 'type', 'id']),
fieldFormatParams: field.format.params(),
});
}

onFieldChange = (fieldName, value) => {
const field = this.state.field;
const { field } = this.state;
field[fieldName] = value;
this.forceUpdate();
};
Expand All @@ -169,18 +184,11 @@ export class FieldEditor extends PureComponent {
const DefaultFieldFormat = fieldFormats.getDefaultType(type);

field.type = type;

const fieldTypeFormats = [
getDefaultFormat(DefaultFieldFormat),
...getFieldFormats().getByFieldType(field.type),
];

const FieldFormat = fieldTypeFormats[0];
field.format = new FieldFormat(null, getConfig);
field.format = new DefaultFieldFormat(null, getConfig);

this.setState({
fieldTypeFormats,
fieldFormatId: FieldFormat.id,
fieldTypeFormats: getFieldTypeFormatsList(field, DefaultFieldFormat),
fieldFormatId: DefaultFieldFormat.id,
fieldFormatParams: field.format.params(),
});
};
Expand All @@ -197,12 +205,13 @@ export class FieldEditor extends PureComponent {
};

onFormatChange = (formatId, params) => {
const { getConfig } = this.props.helpers;
const fieldFormats = getFieldFormats();
const { field, fieldTypeFormats } = this.state;
const FieldFormat =
fieldTypeFormats.find(format => format.id === formatId) || fieldTypeFormats[0];
const FieldFormat = fieldFormats.getType(
formatId || fieldTypeFormats[0]?.defaultFieldFormat.id
);

field.format = new FieldFormat(params, getConfig);
field.format = new FieldFormat(params, this.props.helpers.getConfig);

this.setState({
fieldFormatId: FieldFormat.id,
Expand Down Expand Up @@ -416,7 +425,8 @@ export class FieldEditor extends PureComponent {
renderFormat() {
const { field, fieldTypeFormats, fieldFormatId, fieldFormatParams } = this.state;
const { fieldFormatEditors } = this.props.helpers;
const defaultFormat = fieldTypeFormats[0] && fieldTypeFormats[0].resolvedTitle;
const defaultFormat = fieldTypeFormats[0]?.defaultFieldFormat.title;

const label = defaultFormat ? (
<FormattedMessage
id="common.ui.fieldEditor.defaultFormatHeader"
Expand Down

This file was deleted.

32 changes: 0 additions & 32 deletions src/legacy/ui/public/field_editor/lib/get_default_format.js

This file was deleted.

1 change: 0 additions & 1 deletion src/legacy/ui/public/field_editor/lib/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,5 +18,4 @@
*/

export { copyField } from './copy_field';
export { getDefaultFormat } from './get_default_format';
export { executeScript, isScriptValid } from './validate_script';

0 comments on commit f0e063c

Please sign in to comment.