ignore spaces for vulnerability filters (#19180)

1.Fixes #19168 Signed-off-by: AllForNothing <[email protected]>
goharbor · Aug 15, 2023 · 296e905 · 296e905
1 parent 661c762
commit 296e905
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/...on-services/vulnerability-database/vulnerability-filter/vulnerability-filter.component.ts b/...on-services/vulnerability-database/vulnerability-filter/vulnerability-filter.component.ts
@@ -97,7 +97,7 @@ export class VulnerabilityFilterComponent {
                 return;
             } else if (
                 item === OptionType.PROJECT_ID &&
-                this.valueMap[OptionType.PROJECT_ID]
+                this.valueMap[OptionType.PROJECT_ID]?.trim()
             ) {
                 result.projectId = this.valueMap[OptionType.PROJECT_ID]?.trim();
             } else if (item === OptionType.SEVERITY) {
@@ -107,14 +107,14 @@ export class VulnerabilityFilterComponent {
                     );
                 }
             } else if (item === OptionType.CVSS3) {
-                if (this.startScore || this.endScore) {
+                if (this.startScore?.trim() || this.endScore?.trim()) {
                     result.normal.push(
                         `${OptionType.CVSS3}=[${
                             this.startScore ? this.startScore?.trim() : '0.0'
                         }~${this.endScore ? this.endScore?.trim() : '10.0'}]`
                     );
                 }
-            } else if (this.valueMap[item]) {
+            } else if (this.valueMap[item]?.trim()) {
                 result.normal.push(`${item}=${this.valueMap[item]?.trim()}`);
             }
         });

diff --git a/...ervices/vulnerability-database/vulnerability-summary/vulnerability-summary.component.html b/...ervices/vulnerability-database/vulnerability-summary/vulnerability-summary.component.html
@@ -88,7 +88,7 @@ <h2>
                     {{ securitySummary?.none_cnt || 0 }}
                 </div>
             </div>
-            <div class="clr-row" [hidden]="!securitySummary">
+            <div class="clr-row" [hidden]="!securitySummary?.total_vuls">
                 <div class="placeholder">
                     <div class="pie-chart" id="pie-chart"></div>
                 </div>