From bbd3452e4c9950e5d040e3ebcefaa9b7645df005 Mon Sep 17 00:00:00 2001
From: Ryan Mathis <rmathis@unicon.net>
Date: Thu, 15 Aug 2019 10:24:44 -0700
Subject: [PATCH] SHIBUI-1385 Fixed filter list for older versions

---
 .../filter-configuration-list-item.component.html   |  4 ++--
 .../filter-configuration-list-item.component.ts     |  1 +
 .../filter-configuration-list.component.html        |  1 +
 .../container/version-options.component.html        |  2 +-
 .../container/version-options.component.ts          |  5 +++--
 ui/src/app/metadata/configuration/reducer/index.ts  | 13 +++++++++++++
 6 files changed, 21 insertions(+), 5 deletions(-)

diff --git a/ui/src/app/metadata/configuration/component/filter-configuration-list-item.component.html b/ui/src/app/metadata/configuration/component/filter-configuration-list-item.component.html
index b8df62a11..cf1cfe3be 100644
--- a/ui/src/app/metadata/configuration/component/filter-configuration-list-item.component.html
+++ b/ui/src/app/metadata/configuration/component/filter-configuration-list-item.component.html
@@ -1,6 +1,6 @@
 <div class="d-flex justify-content-start align-items-center">
     <span class="mr-2">{{ index + 1 }}</span>
-    <div class="d-flex justify-content-between">
+    <div class="d-flex justify-content-between" *ngIf="editable">
         <button class="btn btn-link btn-sm" (click)="onUpdateOrderUp.emit(filter)" [disabled]="isFirst">
             <i class="fa fa-arrow-circle-up fa-2x sr-hidden"></i>
             <span class="sr-only" translate="action.move-up">Move Up</span>
@@ -19,7 +19,7 @@
 </div>
 <div *ngIf="open">
     <hr class="my-2" />
-    <div class="d-flex justify-content-end mb-2">
+    <div class="d-flex justify-content-end mb-2" *ngIf="editable">
         <div class="d-flex justify-content-between">
             <a class="btn btn-link"
                 [routerLink]="['../../', 'filter', filter.resourceId, 'edit']">
diff --git a/ui/src/app/metadata/configuration/component/filter-configuration-list-item.component.ts b/ui/src/app/metadata/configuration/component/filter-configuration-list-item.component.ts
index ede42a55a..9e5cb7630 100644
--- a/ui/src/app/metadata/configuration/component/filter-configuration-list-item.component.ts
+++ b/ui/src/app/metadata/configuration/component/filter-configuration-list-item.component.ts
@@ -16,6 +16,7 @@ export class FilterConfigurationListItemComponent implements OnChanges {
     @Input() index: number;
     @Input() isFirst: boolean;
     @Input() isLast: boolean;
+    @Input() editable: boolean;
 
     @Output() onUpdateOrderUp: EventEmitter<MetadataFilter> = new EventEmitter();
     @Output() onUpdateOrderDown: EventEmitter<MetadataFilter> = new EventEmitter();
diff --git a/ui/src/app/metadata/configuration/component/filter-configuration-list.component.html b/ui/src/app/metadata/configuration/component/filter-configuration-list.component.html
index c49a7738c..03d89d651 100644
--- a/ui/src/app/metadata/configuration/component/filter-configuration-list.component.html
+++ b/ui/src/app/metadata/configuration/component/filter-configuration-list.component.html
@@ -6,6 +6,7 @@
             [index]="i"
             [isFirst]="isFirst"
             [isLast]="isLast"
+            [editable]="editable"
             (onUpdateOrderDown)="onUpdateOrderDown.emit($event)"
             (onUpdateOrderUp)="onUpdateOrderUp.emit($event)"
             (onRemove)="onRemove.emit($event)"></filter-configuration-list-item>
diff --git a/ui/src/app/metadata/configuration/container/version-options.component.html b/ui/src/app/metadata/configuration/container/version-options.component.html
index caecfb8a3..5bc60ca3f 100644
--- a/ui/src/app/metadata/configuration/container/version-options.component.html
+++ b/ui/src/app/metadata/configuration/container/version-options.component.html
@@ -34,7 +34,7 @@ <h2 class="title h4 m-0 ml-2 flex-grow-1">
         </div>
         <filter-configuration-list
             [editable]="false"
-            [filters]="filters"></filter-configuration-list>
+            [filters]="filters$ | async"></filter-configuration-list>
     </div>
     <button class="btn btn-link" (click)="onScrollTo('header')">
         <i class="fa fa-chevron-up sr-hidden"></i>&nbsp;
diff --git a/ui/src/app/metadata/configuration/container/version-options.component.ts b/ui/src/app/metadata/configuration/container/version-options.component.ts
index 54143f1be..6202761c9 100644
--- a/ui/src/app/metadata/configuration/container/version-options.component.ts
+++ b/ui/src/app/metadata/configuration/container/version-options.component.ts
@@ -10,7 +10,8 @@ import {
     getConfigurationSections,
     getSelectedVersion,
     getConfigurationModelEnabled,
-    getConfigurationModelType
+    getConfigurationModelType,
+    getVersionModelFilters
 } from '../reducer';
 import { MetadataConfiguration } from '../model/metadata-configuration';
 import { MetadataVersion } from '../model/version';
@@ -35,9 +36,9 @@ export class VersionOptionsComponent implements OnDestroy {
     isEnabled$: Observable<boolean> = this.store.select(getConfigurationModelEnabled);
     model$: Observable<Metadata> = this.store.select(getVersionModel);
     type$: Observable<string> = this.store.select(getConfigurationModelType);
+    filters$: Observable<any[]> = this.store.select(getVersionModelFilters);
     id: string;
     kind: string;
-    filters = [];
 
     constructor(
         protected store: Store<ConfigurationState>,
diff --git a/ui/src/app/metadata/configuration/reducer/index.ts b/ui/src/app/metadata/configuration/reducer/index.ts
index 5a3b103ea..43638ba72 100644
--- a/ui/src/app/metadata/configuration/reducer/index.ts
+++ b/ui/src/app/metadata/configuration/reducer/index.ts
@@ -151,6 +151,13 @@ export const getComparisonConfigurations = createSelector(
 export const getComparisonConfigurationCount = createSelector(getComparisonConfigurations, (config) => config ? config.dates.length : 0);
 
 // Version Restoration
+export const filterPluginTypes = ['RequiredValidUntil', 'SignatureValidation', 'EntityRoleWhiteList'];
+export const isAdditionalFilter = (type) => filterPluginTypes.indexOf(type) === -1;
+
+export const getVersionModelFiltersFn =
+    (model, kind) => kind === 'provider' ?
+    model.metadataFilters.filter(filter => isAdditionalFilter(filter['@type'])) :
+    null;
 
 export const getVersionState = createSelector(getState, getVersionStateFn);
 export const getVersionModel = createSelector(getVersionState, fromVersion.getVersionModel);
@@ -162,6 +169,12 @@ export const getVersionConfigurationSections = createSelector(
     getConfigurationSectionsFn
 );
 
+export const getVersionModelFilters = createSelector(
+    getVersionModel,
+    getConfigurationModelKind,
+    getVersionModelFiltersFn
+);
+
 // Mixed states
 
 export const getConfigurationModelFn = (kind, version, provider, resolver) => {