Skip to content

Commit bba8ab7

Browse files
committed
Hide deleted templates from override warning in standards overview
1 parent 9d85d14 commit bba8ab7

1 file changed

Lines changed: 16 additions & 7 deletions

File tree

src/pages/tenant/manage/applied-standards.js

Lines changed: 16 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -151,6 +151,7 @@ const Page = () => {
151151
return template?.displayName || template?.templateName || template?.name || guid
152152
}
153153

154+
const templateExists = (guid) => !!guid && templateDetails.data.some((t) => t.GUID === guid)
154155
const allStandards = []
155156
if (selectedTemplate.standards) {
156157
Object.entries(selectedTemplate.standards).forEach(([standardKey, standardConfig]) => {
@@ -231,7 +232,7 @@ const Page = () => {
231232

232233
// Check if this standard is overridden by another template
233234
const tenantTemplateId = standardObject?.TemplateId
234-
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId
235+
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId && templateExists(tenantTemplateId)
235236
const overridingTemplateName = isOverridden
236237
? getTemplateDisplayName(tenantTemplateId)
237238
: null
@@ -356,7 +357,7 @@ const Page = () => {
356357

357358
// Check if this standard is overridden by another template
358359
const tenantTemplateId = standardObject?.TemplateId
359-
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId
360+
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId && templateExists(tenantTemplateId)
360361
const overridingTemplateName = isOverridden
361362
? getTemplateDisplayName(tenantTemplateId)
362363
: null
@@ -445,7 +446,7 @@ const Page = () => {
445446
const standardObject = currentTenantObj?.[standardId]
446447
const directStandardValue = standardObject?.Value
447448
const tenantTemplateId = standardObject?.TemplateId
448-
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId
449+
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId && templateExists(tenantTemplateId)
449450
const overridingTemplateName = isOverridden
450451
? getTemplateDisplayName(tenantTemplateId)
451452
: null
@@ -562,7 +563,7 @@ const Page = () => {
562563
const standardObject = currentTenantObj?.[standardId]
563564
const directStandardValue = standardObject?.Value
564565
const tenantTemplateId = standardObject?.TemplateId
565-
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId
566+
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId && templateExists(tenantTemplateId)
566567
const overridingTemplateName = isOverridden
567568
? getTemplateDisplayName(tenantTemplateId)
568569
: null
@@ -684,7 +685,7 @@ const Page = () => {
684685
const standardObject = currentTenantObj?.[standardId]
685686
const directStandardValue = standardObject?.Value
686687
const tenantTemplateId = standardObject?.TemplateId
687-
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId
688+
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId && templateExists(tenantTemplateId)
688689
const overridingTemplateName = isOverridden
689690
? getTemplateDisplayName(tenantTemplateId)
690691
: null
@@ -779,7 +780,7 @@ const Page = () => {
779780
const standardObject = currentTenantObj?.[standardId]
780781
const directStandardValue = standardObject?.Value
781782
const tenantTemplateId = standardObject?.TemplateId
782-
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId
783+
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId && templateExists(tenantTemplateId)
783784
const overridingTemplateName = isOverridden
784785
? getTemplateDisplayName(tenantTemplateId)
785786
: null
@@ -1056,7 +1057,7 @@ const Page = () => {
10561057

10571058
// Check if this standard is overridden by another template
10581059
const tenantTemplateId = standardObject?.TemplateId
1059-
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId
1060+
const isOverridden = tenantTemplateId && tenantTemplateId !== templateId && templateExists(tenantTemplateId)
10601061
const overridingTemplateName = isOverridden
10611062
? getTemplateDisplayName(tenantTemplateId)
10621063
: null
@@ -1718,6 +1719,14 @@ const Page = () => {
17181719
color={isDriftTemplate ? 'info' : 'default'}
17191720
variant="outlined"
17201721
/>
1722+
{isDriftTemplate && (
1723+
<Chip
1724+
label={'Alignment Score May Be Inaccurate For Drift Templates'}
1725+
size="small"
1726+
color={'warning'}
1727+
variant="outlined"
1728+
/>
1729+
)}
17211730
</Stack>
17221731
)}
17231732
<Menu

0 commit comments

Comments
 (0)