From f2dfb93f093eeb7c239cba2d5b73a36b4b964900 Mon Sep 17 00:00:00 2001
From: Remi Blom-Ohlsen <remi.blom-ohlsen@puzzlepart.com>
Date: Wed, 13 Nov 2024 16:18:33 +0100
Subject: [PATCH] Bugfix for processing idea without values [skip-ci]

---
 .../IdeaModule/IdeaField/useIdeaField.tsx     | 32 +++++-----
 .../src/components/IdeaModule/IdeaModule.tsx  |  1 -
 .../Lists/Prosjektinnholdskolonner.xml        | 62 +++++++++----------
 3 files changed, 49 insertions(+), 46 deletions(-)

diff --git a/SharePointFramework/PortfolioWebParts/src/components/IdeaModule/IdeaField/useIdeaField.tsx b/SharePointFramework/PortfolioWebParts/src/components/IdeaModule/IdeaField/useIdeaField.tsx
index f21c644b8..943c06e4c 100644
--- a/SharePointFramework/PortfolioWebParts/src/components/IdeaModule/IdeaField/useIdeaField.tsx
+++ b/SharePointFramework/PortfolioWebParts/src/components/IdeaModule/IdeaField/useIdeaField.tsx
@@ -57,20 +57,24 @@ export function useIdeaField(props: IIdeaFieldProps) {
       [
         'User',
         ([user]: IPersonaProps[]) => {
-          return (
-            <Persona
-              {...user}
-              title={user.text}
-              name={user.text}
-              size='small'
-              avatar={{
-                image: {
-                  src: user.imageUrl
-                }
-              }}
-              style={{ marginTop: 6 }}
-            />
-          )
+          if (user) {
+            return (
+              <Persona
+                {...user}
+                title={user.text}
+                name={user.text}
+                size='small'
+                avatar={{
+                  image: {
+                    src: user.imageUrl
+                  }
+                }}
+                style={{ marginTop: 6 }}
+              />
+            )
+          } else {
+            return null
+          }
         }
       ],
       [
diff --git a/SharePointFramework/PortfolioWebParts/src/components/IdeaModule/IdeaModule.tsx b/SharePointFramework/PortfolioWebParts/src/components/IdeaModule/IdeaModule.tsx
index c2102c1e8..394f3aa8d 100644
--- a/SharePointFramework/PortfolioWebParts/src/components/IdeaModule/IdeaModule.tsx
+++ b/SharePointFramework/PortfolioWebParts/src/components/IdeaModule/IdeaModule.tsx
@@ -237,7 +237,6 @@ export const IdeaModule: FC<IIdeaModuleProps> = (props) => {
                           <AccordionPanel>
                             <div className={styles.idea}>
                               {state.selectedIdea.registeredFieldValues
-                                .filter((model) => !ignoreFields.includes(model.internalName))
                                 .map((model, idx) => (
                                   <IdeaField key={idx} model={model} />
                                 ))}
diff --git a/Templates/Portfolio/Objects/Lists/Prosjektinnholdskolonner.xml b/Templates/Portfolio/Objects/Lists/Prosjektinnholdskolonner.xml
index 72a7c6d4e..6c2783861 100644
--- a/Templates/Portfolio/Objects/Lists/Prosjektinnholdskolonner.xml
+++ b/Templates/Portfolio/Objects/Lists/Prosjektinnholdskolonner.xml
@@ -855,29 +855,6 @@
         </pnp:DataRow>
         <pnp:DataRow>
             <pnp:DataValue FieldName="GtSortOrder">1090</pnp:DataValue>
-            <pnp:DataValue FieldName="Title">Anbefaling</pnp:DataValue>
-            <pnp:DataValue FieldName="GtInternalName">GtIdeaRecommendation</pnp:DataValue>
-            <pnp:DataValue FieldName="GtManagedProperty">GtIdeaRecommendationOWSCHCS</pnp:DataValue>
-            <pnp:DataValue FieldName="GtFieldDataType">Tags</pnp:DataValue>
-            <pnp:DataValue FieldName="GtDataSourceCategory">Idémodul</pnp:DataValue>
-            <pnp:DataValue FieldName="GtColMinWidth">100</pnp:DataValue>
-            <pnp:DataValue FieldName="GtColMaxWidth">200</pnp:DataValue>
-            <pnp:DataValue FieldName="GtIsGroupable">True</pnp:DataValue>
-            <pnp:DataValue FieldName="GtIdeaCopyToProcess">True</pnp:DataValue>
-        </pnp:DataRow>
-        <pnp:DataRow>
-            <pnp:DataValue FieldName="GtSortOrder">1100</pnp:DataValue>
-            <pnp:DataValue FieldName="Title">Kommentar til anbefaling</pnp:DataValue>
-            <pnp:DataValue FieldName="GtInternalName">GtIdeaRecommendationComment</pnp:DataValue>
-            <pnp:DataValue FieldName="GtManagedProperty">GtIdeaRecommendationCommentOWSMTXT</pnp:DataValue>
-            <pnp:DataValue FieldName="GtFieldDataType">Note</pnp:DataValue>
-            <pnp:DataValue FieldName="GtDataSourceCategory">Idémodul</pnp:DataValue>
-            <pnp:DataValue FieldName="GtColMinWidth">200</pnp:DataValue>
-            <pnp:DataValue FieldName="GtColMaxWidth">300</pnp:DataValue>
-            <pnp:DataValue FieldName="GtIdeaCopyToProcess">True</pnp:DataValue>
-        </pnp:DataRow>
-        <pnp:DataRow>
-            <pnp:DataValue FieldName="GtSortOrder">1110</pnp:DataValue>
             <pnp:DataValue FieldName="Title">Idéeier</pnp:DataValue>
             <pnp:DataValue FieldName="GtInternalName">GtIdeaOwner</pnp:DataValue>
             <pnp:DataValue FieldName="GtManagedProperty">GtIdeaOwner</pnp:DataValue>
@@ -888,7 +865,7 @@
             <pnp:DataValue FieldName="GtIsGroupable">True</pnp:DataValue>
         </pnp:DataRow>
         <pnp:DataRow>
-            <pnp:DataValue FieldName="GtSortOrder">1120</pnp:DataValue>
+            <pnp:DataValue FieldName="GtSortOrder">1100</pnp:DataValue>
             <pnp:DataValue FieldName="Title">Overordnet løsningsbeskrivelse</pnp:DataValue>
             <pnp:DataValue FieldName="GtInternalName">GtIdeaSolutionDescription</pnp:DataValue>
             <pnp:DataValue FieldName="GtManagedProperty">GtIdeaSolutionDescriptionOWSMTXT</pnp:DataValue>
@@ -898,7 +875,7 @@
             <pnp:DataValue FieldName="GtColMaxWidth">300</pnp:DataValue>
         </pnp:DataRow>
         <pnp:DataRow>
-            <pnp:DataValue FieldName="GtSortOrder">1130</pnp:DataValue>
+            <pnp:DataValue FieldName="GtSortOrder">1110</pnp:DataValue>
             <pnp:DataValue FieldName="Title">Forventede gevinster</pnp:DataValue>
             <pnp:DataValue FieldName="GtInternalName">GtIdeaExpectedGain</pnp:DataValue>
             <pnp:DataValue FieldName="GtManagedProperty">GtIdeaExpectedGainOWSMTXT</pnp:DataValue>
@@ -908,7 +885,7 @@
             <pnp:DataValue FieldName="GtColMaxWidth">300</pnp:DataValue>
         </pnp:DataRow>
         <pnp:DataRow>
-            <pnp:DataValue FieldName="GtSortOrder">1140</pnp:DataValue>
+            <pnp:DataValue FieldName="GtSortOrder">1120</pnp:DataValue>
             <pnp:DataValue FieldName="Title">Gjennomføringsplan: Tilnærming</pnp:DataValue>
             <pnp:DataValue FieldName="GtInternalName">GtIdeaExecutionPlanApproach</pnp:DataValue>
             <pnp:DataValue FieldName="GtManagedProperty">GtIdeaExecutionPlanApproachOWSMTXT</pnp:DataValue>
@@ -918,7 +895,7 @@
             <pnp:DataValue FieldName="GtColMaxWidth">300</pnp:DataValue>
         </pnp:DataRow>
         <pnp:DataRow>
-            <pnp:DataValue FieldName="GtSortOrder">1150</pnp:DataValue>
+            <pnp:DataValue FieldName="GtSortOrder">1130</pnp:DataValue>
             <pnp:DataValue FieldName="Title">Gjennomføringsplan: Ressursbehov</pnp:DataValue>
             <pnp:DataValue FieldName="GtInternalName">GtIdeaExecutionResourceNeeds</pnp:DataValue>
             <pnp:DataValue FieldName="GtManagedProperty">GtIdeaExecutionResourceNeedsOWSMTXT</pnp:DataValue>
@@ -928,7 +905,7 @@
             <pnp:DataValue FieldName="GtColMaxWidth">300</pnp:DataValue>
         </pnp:DataRow>
         <pnp:DataRow>
-            <pnp:DataValue FieldName="GtSortOrder">1160</pnp:DataValue>
+            <pnp:DataValue FieldName="GtSortOrder">1140</pnp:DataValue>
             <pnp:DataValue FieldName="Title">Gjennomføringsplan: Suksessfaktorer</pnp:DataValue>
             <pnp:DataValue FieldName="GtInternalName">GtIdeaExecutionSuccessFactors</pnp:DataValue>
             <pnp:DataValue FieldName="GtManagedProperty">GtIdeaExecutionSuccessFactorsOWSMTXT</pnp:DataValue>
@@ -938,7 +915,7 @@
             <pnp:DataValue FieldName="GtColMaxWidth">300</pnp:DataValue>
         </pnp:DataRow>
         <pnp:DataRow>
-            <pnp:DataValue FieldName="GtSortOrder">1170</pnp:DataValue>
+            <pnp:DataValue FieldName="GtSortOrder">1150</pnp:DataValue>
             <pnp:DataValue FieldName="Title">Referanse</pnp:DataValue>
             <pnp:DataValue FieldName="GtInternalName">GtIdeaReference</pnp:DataValue>
             <pnp:DataValue FieldName="GtManagedProperty">GtIdeaReferenceOWSMTXT</pnp:DataValue>
@@ -948,7 +925,7 @@
             <pnp:DataValue FieldName="GtColMaxWidth">300</pnp:DataValue>
         </pnp:DataRow>
         <pnp:DataRow>
-            <pnp:DataValue FieldName="GtSortOrder">1180</pnp:DataValue>
+            <pnp:DataValue FieldName="GtSortOrder">1160</pnp:DataValue>
             <pnp:DataValue FieldName="Title">Andre kommentarer</pnp:DataValue>
             <pnp:DataValue FieldName="GtInternalName">GtIdeaOtherComments</pnp:DataValue>
             <pnp:DataValue FieldName="GtManagedProperty">GtIdeaOtherCommentsOWSMTXT</pnp:DataValue>
@@ -958,6 +935,29 @@
             <pnp:DataValue FieldName="GtColMaxWidth">300</pnp:DataValue>
             <pnp:DataValue FieldName="GtIdeaCopyToProcess">True</pnp:DataValue>
         </pnp:DataRow>
+        <pnp:DataRow>
+            <pnp:DataValue FieldName="GtSortOrder">1170</pnp:DataValue>
+            <pnp:DataValue FieldName="Title">Anbefaling</pnp:DataValue>
+            <pnp:DataValue FieldName="GtInternalName">GtIdeaRecommendation</pnp:DataValue>
+            <pnp:DataValue FieldName="GtManagedProperty">GtIdeaRecommendationOWSCHCS</pnp:DataValue>
+            <pnp:DataValue FieldName="GtFieldDataType">Tags</pnp:DataValue>
+            <pnp:DataValue FieldName="GtDataSourceCategory">Idémodul</pnp:DataValue>
+            <pnp:DataValue FieldName="GtColMinWidth">100</pnp:DataValue>
+            <pnp:DataValue FieldName="GtColMaxWidth">200</pnp:DataValue>
+            <pnp:DataValue FieldName="GtIsGroupable">True</pnp:DataValue>
+            <pnp:DataValue FieldName="GtIdeaCopyToProcess">True</pnp:DataValue>
+        </pnp:DataRow>
+        <pnp:DataRow>
+            <pnp:DataValue FieldName="GtSortOrder">1180</pnp:DataValue>
+            <pnp:DataValue FieldName="Title">Kommentar til anbefaling</pnp:DataValue>
+            <pnp:DataValue FieldName="GtInternalName">GtIdeaRecommendationComment</pnp:DataValue>
+            <pnp:DataValue FieldName="GtManagedProperty">GtIdeaRecommendationCommentOWSMTXT</pnp:DataValue>
+            <pnp:DataValue FieldName="GtFieldDataType">Note</pnp:DataValue>
+            <pnp:DataValue FieldName="GtDataSourceCategory">Idémodul</pnp:DataValue>
+            <pnp:DataValue FieldName="GtColMinWidth">200</pnp:DataValue>
+            <pnp:DataValue FieldName="GtColMaxWidth">300</pnp:DataValue>
+            <pnp:DataValue FieldName="GtIdeaCopyToProcess">True</pnp:DataValue>
+        </pnp:DataRow>
         <pnp:DataRow>
             <pnp:DataValue FieldName="GtSortOrder">1190</pnp:DataValue>
             <pnp:DataValue FieldName="Title">Beslutning</pnp:DataValue>
@@ -970,7 +970,7 @@
             <pnp:DataValue FieldName="GtIsGroupable">True</pnp:DataValue>
         </pnp:DataRow>
         <pnp:DataRow>
-            <pnp:DataValue FieldName="GtSortOrder">1210</pnp:DataValue>
+            <pnp:DataValue FieldName="GtSortOrder">1200</pnp:DataValue>
             <pnp:DataValue FieldName="Title">Kommentar til beslutning</pnp:DataValue>
             <pnp:DataValue FieldName="GtInternalName">GtIdeaStrategicValue</pnp:DataValue>
             <pnp:DataValue FieldName="GtManagedProperty">GtIdeaDecisionCommentOWSMTXT</pnp:DataValue>