Refactoring: optimize condition for latest widget timewindow; fix types
This commit is contained in:
parent
fb61aa447d
commit
442c45524e
@ -296,9 +296,8 @@ export class DashboardUtilsService {
|
|||||||
}
|
}
|
||||||
widgetConfig.datasources = this.validateAndUpdateDatasources(widgetConfig.datasources);
|
widgetConfig.datasources = this.validateAndUpdateDatasources(widgetConfig.datasources);
|
||||||
if (type === widgetType.latest) {
|
if (type === widgetType.latest) {
|
||||||
const onlyHistoryTimewindow = datasourcesHasOnlyComparisonAggregation(widgetConfig.datasources);
|
if (datasourcesHasAggregation(widgetConfig.datasources)) {
|
||||||
const aggregationEnabledForKeys = datasourcesHasAggregation(widgetConfig.datasources);
|
const onlyHistoryTimewindow = datasourcesHasOnlyComparisonAggregation(widgetConfig.datasources);
|
||||||
if (aggregationEnabledForKeys) {
|
|
||||||
widgetConfig.timewindow = initModelFromDefaultTimewindow(widgetConfig.timewindow, true,
|
widgetConfig.timewindow = initModelFromDefaultTimewindow(widgetConfig.timewindow, true,
|
||||||
onlyHistoryTimewindow, this.timeService, false);
|
onlyHistoryTimewindow, this.timeService, false);
|
||||||
}
|
}
|
||||||
@ -356,7 +355,7 @@ export class DashboardUtilsService {
|
|||||||
return widgetConfig;
|
return widgetConfig;
|
||||||
}
|
}
|
||||||
|
|
||||||
public removeTimewindowConfigIfUnused(widgetConfig: WidgetConfig, type: widgetType) {
|
private removeTimewindowConfigIfUnused(widgetConfig: WidgetConfig, type: widgetType) {
|
||||||
const widgetHasTimewindow = widgetTypeHasTimewindow(type) || (type === widgetType.latest && datasourcesHasAggregation(widgetConfig.datasources));
|
const widgetHasTimewindow = widgetTypeHasTimewindow(type) || (type === widgetType.latest && datasourcesHasAggregation(widgetConfig.datasources));
|
||||||
if (!widgetHasTimewindow || widgetConfig.useDashboardTimewindow) {
|
if (!widgetHasTimewindow || widgetConfig.useDashboardTimewindow) {
|
||||||
delete widgetConfig.displayTimewindow;
|
delete widgetConfig.displayTimewindow;
|
||||||
|
|||||||
@ -197,7 +197,7 @@ export function deleteNullProperties(obj: any) {
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
export function deleteFalseProperties(obj: any) {
|
export function deleteFalseProperties(obj: Record<string, any>): void {
|
||||||
if (isUndefinedOrNull(obj)) {
|
if (isUndefinedOrNull(obj)) {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user