Github user merrimanr commented on a diff in the pull request:
https://github.com/apache/metron/pull/1010#discussion_r189060226
--- Diff:
metron-interface/metron-alerts/src/app/alerts/alerts-list/table-view/table-view.component.ts
---
@@ -77,12 +80,24 @@ export class TableViewComponent implements OnChanges {
searchService: SearchService,
metronDialogBox: MetronDialogBox,
updateService: UpdateService,
- metaAlertService: MetaAlertService) {
+ metaAlertService: MetaAlertService,
+ globalConfigService: GlobalConfigService) {
this.router = router;
this.searchService = searchService;
this.metronDialogBox = metronDialogBox;
this.updateService = updateService;
this.metaAlertService = metaAlertService;
+ this.globalConfigService = globalConfigService;
+ }
+
+ ngOnInit() {
+ this.globalConfigService.get().subscribe((config: {}) => {
+ this.globalConfig = config;
+ if (this.globalConfig['source.type.field'] === 'source.type' &&
!this.alertsColumnsToDisplay['source.type']) {
--- End diff --
I don't think this logic is correct. What happens if you have
"source.type" selected as a table column? I went in a updated by local storage
directly and ended up with 2 "source.type" columns.
---
