github-mirrors/kibana
1
0
Fork 0
mirror of https://github.com/elastic/kibana.git synced 2025-04-23 09:19:04 -04:00
Code Issues Projects Releases Packages Wiki Activity

[ML] Fix double escaping. (#30492) (#30499)

Browse source 
Because the info tooltip has been migrated to React, the escaping necessary for the previous jQuery based tooltip resulting in a double escaping and special character visible in the tooltip. This fixes it by removing the now unnecessary escaping.
This commit is contained in:
Walter Rafelsberger 2019-02-08 14:29:43 +01:00 • committed by GitHub
parent ab34ca30c5
commit 6bb1b10cee
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

7
x-pack/plugins/ml/public/explorer/explorer_charts/explorer_chart_config_builder.js
View file

@ -15,7 +15,6 @@ import _ from 'lodash';
import { parseInterval } from 'ui/utils/parse_interval';
import { buildConfigFromDetector } from '../../util/chart_config_builder';
import { mlEscape } from '../../util/string_utils';
import { mlJobService } from '../../services/job_service';
// Builds the chart configuration for the provided anomaly record, returning
@ -78,7 +77,7 @@ export function buildConfig(record) {
// Build the tooltip data for the chart info icon, showing further details on what is being plotted.
let functionLabel = config.metricFunction;
if (config.metricFieldName !== undefined) {
functionLabel += ` ${mlEscape(config.metricFieldName)}`;
functionLabel += ` ${config.metricFieldName}`;
}
config.infoTooltip = {
@ -86,8 +85,8 @@ export function buildConfig(record) {
aggregationInterval: config.interval,
chartFunction: functionLabel,
entityFields: config.entityFields.map((f) => ({
fieldName: mlEscape(f.fieldName),
fieldValue: mlEscape(f.fieldValue),
fieldName: f.fieldName,
fieldValue: f.fieldValue,
}))
};