ensure all files present in filters

Signed-off-by: Colin Grant <colin.grant@ericsson.com>
eclipse-theia · Jun 23, 2020 · f8b67c8 · f8b67c8
1 parent 52a43ce
commit f8b67c8
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/packages/filesystem/src/browser/file-dialog/file-dialog-tree-filters-renderer.tsx b/packages/filesystem/src/browser/file-dialog/file-dialog-tree-filters-renderer.tsx
@@ -36,26 +36,24 @@ export class FileDialogTreeFilters {
 
 export class FileDialogTreeFiltersRenderer extends ReactRenderer {
 
+    readonly appliedFilters: FileDialogTreeFilters;
+
     constructor(
-        readonly filters: FileDialogTreeFilters,
+        readonly suppliedFilters: FileDialogTreeFilters,
         readonly fileDialogTree: FileDialogTree
     ) {
         super();
+        this.appliedFilters = { 'All Files': [], ...suppliedFilters, };
     }
 
     protected readonly handleFilterChanged = (e: React.ChangeEvent<HTMLSelectElement>) => this.onFilterChanged(e);
 
     protected doRender(): React.ReactNode {
-        if (!this.filters) {
+        if (!this.appliedFilters) {
             return undefined;
         }
 
-        const fileTypes = ['All Files'];
-        Object.keys(this.filters).forEach(element => {
-            fileTypes.push(element);
-        });
-
-        const options = fileTypes.map(value => this.renderLocation(value));
+        const options = Object.keys(this.appliedFilters).map(value => this.renderLocation(value));
         return <select className={'theia-select ' + FILE_TREE_FILTERS_LIST_CLASS} onChange={this.handleFilterChanged}>{...options}</select>;
     }
 
@@ -67,7 +65,7 @@ export class FileDialogTreeFiltersRenderer extends ReactRenderer {
         const locationList = this.locationList;
         if (locationList) {
             const value = locationList.value;
-            const filters = this.filters[value];
+            const filters = this.appliedFilters[value];
             this.fileDialogTree.setFilter(filters);
         }