From 6e6fd6966e2ce3f7e669cfb10e2a2eaf6de42fa4 Mon Sep 17 00:00:00 2001
From: Laurent Bonnet <laurent.bonnet@filigran.io>
Date: Fri, 13 Sep 2024 15:05:12 +0200
Subject: [PATCH] [frontend] make TAI personas always visible to be able to add
 some

---
 .../src/components/FieldOrEmpty.tsx           |  2 +-
 .../StixCyberObservableCreation.jsx           |  2 +
 .../ThreatActorGroupLocation.jsx              | 96 +++++++++----------
 .../ThreatActorIndividualDetailsChips.tsx     | 73 +++++++-------
 .../ThreatActorIndividualLocation.jsx         |  2 +-
 5 files changed, 89 insertions(+), 86 deletions(-)

diff --git a/opencti-platform/opencti-front/src/components/FieldOrEmpty.tsx b/opencti-platform/opencti-front/src/components/FieldOrEmpty.tsx
index 632009c9481ff..1dd899f96659d 100644
--- a/opencti-platform/opencti-front/src/components/FieldOrEmpty.tsx
+++ b/opencti-platform/opencti-front/src/components/FieldOrEmpty.tsx
@@ -6,7 +6,7 @@ interface FieldOrEmptyProps<T> {
   children: React.ReactNode;
 }
 
-const FieldOrEmpty = <T = never>({ source, children }: FieldOrEmptyProps<T>) => {
+const FieldOrEmpty = <T = unknown>({ source, children }: FieldOrEmptyProps<T>) => {
   return <>{isNotEmptyField(source) ? children : '-'}</>; // render the children if source is defined
 };
 export default FieldOrEmpty;
diff --git a/opencti-platform/opencti-front/src/private/components/observations/stix_cyber_observables/StixCyberObservableCreation.jsx b/opencti-platform/opencti-front/src/private/components/observations/stix_cyber_observables/StixCyberObservableCreation.jsx
index 0cd3e0f90e589..c4c84bbc9f9c5 100644
--- a/opencti-platform/opencti-front/src/private/components/observations/stix_cyber_observables/StixCyberObservableCreation.jsx
+++ b/opencti-platform/opencti-front/src/private/components/observations/stix_cyber_observables/StixCyberObservableCreation.jsx
@@ -270,6 +270,7 @@ const StixCyberObservableCreation = ({
   inputValue,
   paginationKey,
   paginationOptions,
+  controlledDialStyles = {},
   defaultCreatedBy = null,
   defaultMarkingDefinitions = null,
   isFromBulkRelation,
@@ -901,6 +902,7 @@ const StixCyberObservableCreation = ({
               entityType={type ?? 'Observable'}
               onOpen={handleOpen}
               onClose={() => {}}
+              style={controlledDialStyles}
             />
           : <Fab
               onClick={handleOpen}
diff --git a/opencti-platform/opencti-front/src/private/components/threats/threat_actors_group/ThreatActorGroupLocation.jsx b/opencti-platform/opencti-front/src/private/components/threats/threat_actors_group/ThreatActorGroupLocation.jsx
index 80c9194b76a07..86897be3f4c09 100644
--- a/opencti-platform/opencti-front/src/private/components/threats/threat_actors_group/ThreatActorGroupLocation.jsx
+++ b/opencti-platform/opencti-front/src/private/components/threats/threat_actors_group/ThreatActorGroupLocation.jsx
@@ -21,6 +21,7 @@ import Security from '../../../../utils/Security';
 import { KNOWLEDGE_KNUPDATE } from '../../../../utils/hooks/useGranted';
 import AddLocationsThreatActorGroup from './AddLocationsThreatActorGroup';
 import { addLocationsThreatActorGroupMutationRelationDelete } from './AddLocationsThreatActorGroupLines';
+import FieldOrEmpty from '../../../../components/FieldOrEmpty';
 
 class ThreatActorGroupLocationsComponent extends Component {
   removeLocation(locationEdge) {
@@ -60,62 +61,59 @@ class ThreatActorGroupLocationsComponent extends Component {
           />
         </Security>
         <div className="clearfix" />
-        <List style={{ marginTop: -10 }}>
-          {threatActorGroup.locations.edges.length === 0 && (
-            <ListItem dense={true} divider={true} button={false}>
-              <ListItemText primary="-" />
-            </ListItem>
-          )}
-          {threatActorGroup.locations.edges.map((locationEdge) => {
-            const { types } = locationEdge;
-            const location = locationEdge.node;
-            const link = resolveLink(location.entity_type);
-            const flag = location.entity_type === 'Country'
+        <FieldOrEmpty source={threatActorGroup.locations.edges}>
+          <List style={{ marginTop: -10 }}>
+            {threatActorGroup.locations.edges.map((locationEdge) => {
+              const { types } = locationEdge;
+              const location = locationEdge.node;
+              const link = resolveLink(location.entity_type);
+              const flag = location.entity_type === 'Country'
               && R.head(
                 (location.x_opencti_aliases ?? []).filter(
                   (n) => n?.length === 2,
                 ),
               );
-            return (
-              <ListItem
-                key={location.id}
-                dense={true}
-                divider={true}
-                button={true}
-                component={Link}
-                to={`${link}/${location.id}`}
-              >
-                <ListItemIcon>
+              return (
+                <ListItem
+                  key={location.id}
+                  dense={true}
+                  divider={true}
+                  button={true}
+                  component={Link}
+                  to={`${link}/${location.id}`}
+                >
                   <ListItemIcon>
-                    {flag ? (
-                      <img
-                        style={{ width: 20 }}
-                        src={`${APP_BASE_PATH}/static/flags/4x3/${flag.toLowerCase()}.svg`}
-                        alt={location.name}
-                      />
-                    ) : (
-                      <ItemIcon type={location.entity_type} />
-                    )}
+                    <ListItemIcon>
+                      {flag ? (
+                        <img
+                          style={{ width: 20 }}
+                          src={`${APP_BASE_PATH}/static/flags/4x3/${flag.toLowerCase()}.svg`}
+                          alt={location.name}
+                        />
+                      ) : (
+                        <ItemIcon type={location.entity_type} />
+                      )}
+                    </ListItemIcon>
                   </ListItemIcon>
-                </ListItemIcon>
-                <ListItemText primary={location.name} />
-                {types.includes('manual') ? (
-                  <ListItemSecondaryAction>
-                    <Security needs={[KNOWLEDGE_KNUPDATE]}>
-                      <IconButton
-                        aria-label="Remove"
-                        onClick={() => this.removeLocation(locationEdge)}
-                        size="large"
-                      >
-                        <LinkOff />
-                      </IconButton>
-                    </Security>
-                  </ListItemSecondaryAction>
-                ) : <AutoFix fontSize="small" style={{ marginRight: 13 }}/>}
-              </ListItem>
-            );
-          })}
-        </List>
+                  <ListItemText primary={location.name} />
+                  {types.includes('manual') ? (
+                    <ListItemSecondaryAction>
+                      <Security needs={[KNOWLEDGE_KNUPDATE]}>
+                        <IconButton
+                          aria-label="Remove"
+                          onClick={() => this.removeLocation(locationEdge)}
+                          size="large"
+                        >
+                          <LinkOff />
+                        </IconButton>
+                      </Security>
+                    </ListItemSecondaryAction>
+                  ) : <AutoFix fontSize="small" style={{ marginRight: 13 }}/>}
+                </ListItem>
+              );
+            })}
+          </List>
+        </FieldOrEmpty>
       </>
     );
   }
diff --git a/opencti-platform/opencti-front/src/private/components/threats/threat_actors_individual/ThreatActorIndividualDetailsChips.tsx b/opencti-platform/opencti-front/src/private/components/threats/threat_actors_individual/ThreatActorIndividualDetailsChips.tsx
index 5e349d8956452..25e7f4edfc81c 100644
--- a/opencti-platform/opencti-front/src/private/components/threats/threat_actors_individual/ThreatActorIndividualDetailsChips.tsx
+++ b/opencti-platform/opencti-front/src/private/components/threats/threat_actors_individual/ThreatActorIndividualDetailsChips.tsx
@@ -5,6 +5,7 @@ import { KNOWLEDGE_KNUPDATE } from 'src/utils/hooks/useGranted';
 import { ThreatActorIndividualDetails_ThreatActorIndividual$data } from './__generated__/ThreatActorIndividualDetails_ThreatActorIndividual.graphql';
 import AddPersonaThreatActorIndividual from './AddPersonasThreatActorIndividual';
 import AddIndividualsThreatActorIndividual from './AddIndividualsThreatActorIndividual';
+import FieldOrEmpty from '../../../../components/FieldOrEmpty';
 
 type SupportedTypes = 'known-as' | 'impersonates';
 type SupportedFields = 'persona_name' | 'name';
@@ -59,43 +60,45 @@ ThreatActorIndividualDetailsChipsProps
     setNodes(getRelationshipsOfType(relType));
   }, [data]);
 
-  // Hide if there are no valid relations
-
-  return (<div style={{ marginBottom: '20px' }}>
-    <div style={{
-      display: 'flex',
-      flexDirection: 'row',
-    }}
-    >
-
-      <Typography
-        variant="h3"
-        gutterBottom={true}
+  return (
+    <div style={{ marginBottom: '20px' }}>
+      <div style={{
+        display: 'flex',
+        flexDirection: 'row',
+      }}
       >
-        {title}
-      </Typography>
-      <Security
-        needs={[KNOWLEDGE_KNUPDATE]}
-        placeholder={<div style={{ height: 29 }} />}
-      >
-        <AddComponent threatActorIndividual={data} />
-      </Security>
-    </div>
-    <div className='clearfix' />
-    {
-      nodes.map(({ id, to }) => (
-        <Button
-          key={id}
-          variant='outlined'
-          size='small'
-          href={`${path}/${to?.id}`}
-          style={{ margin: '0 5px 5px 0' }}
+
+        <Typography
+          variant="h3"
+          gutterBottom={true}
         >
-          {to?.[field]}
-        </Button>
-      ))
-    }
-  </div>);
+          {title}
+        </Typography>
+        <Security
+          needs={[KNOWLEDGE_KNUPDATE]}
+          placeholder={<div style={{ height: 29 }} />}
+        >
+          <AddComponent threatActorIndividual={data} />
+        </Security>
+      </div>
+      <div className='clearfix' />
+      <FieldOrEmpty source={nodes}>
+        {
+        nodes.map(({ id, to }) => (
+          <Button
+            key={id}
+            variant='outlined'
+            size='small'
+            href={`${path}/${to?.id}`}
+            style={{ margin: '0 5px 5px 0' }}
+          >
+            {to?.[field]}
+          </Button>
+        ))
+      }
+      </FieldOrEmpty>
+    </div>
+  );
 };
 
 export default ThreatActorIndividualDetailsChips;
diff --git a/opencti-platform/opencti-front/src/private/components/threats/threat_actors_individual/ThreatActorIndividualLocation.jsx b/opencti-platform/opencti-front/src/private/components/threats/threat_actors_individual/ThreatActorIndividualLocation.jsx
index 38841e9587a8f..563ef891864b9 100644
--- a/opencti-platform/opencti-front/src/private/components/threats/threat_actors_individual/ThreatActorIndividualLocation.jsx
+++ b/opencti-platform/opencti-front/src/private/components/threats/threat_actors_individual/ThreatActorIndividualLocation.jsx
@@ -63,7 +63,7 @@ class ThreatActorIndividualLocationsComponent extends Component {
           />
         </Security>
         <div className="clearfix" />
-        <FieldOrEmpty source={threatActorIndividual.locations}>
+        <FieldOrEmpty source={threatActorIndividual.locations.edges}>
           <List style={{ marginTop: -10 }}>
             {threatActorIndividual.locations.edges.map((locationEdge) => {
               const { types } = locationEdge;