Updates to capture the value correctly

samsmithnz · Nov 12, 2023 · 37d27d4 · 37d27d4
1 parent 5be21d2
commit 37d27d4
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 4 deletions.
diff --git a/src/RepoGovernance.Service/Controllers/SummaryItemsController.cs b/src/RepoGovernance.Service/Controllers/SummaryItemsController.cs
@@ -51,7 +51,8 @@ public async Task<int> UpdateSummaryItemNuGetPackageStats(NuGetPayload nugetPayl
                 string? repo = nugetPayload?.Repo;
                 string? owner = nugetPayload?.Owner;
                 string? user = nugetPayload?.User;
-                string? jsonPayload = nugetPayload?.JsonPayload;
+                //There is some weirdness when the json is embedded in this object and then the object is serialized a second time - it returns an array of strings.
+                string? jsonPayload = nugetPayload?.JsonPayloadString;
                 string? payloadType = nugetPayload?.PayloadType;
 
                 if (repo == null || owner == null || user == null || jsonPayload == null || payloadType == null)

diff --git a/src/RepoGovernance.Service/Models/NuGetPayload.cs b/src/RepoGovernance.Service/Models/NuGetPayload.cs
@@ -1,8 +1,10 @@
-namespace RepoGovernance.Service.Models
+using System.Text;
+
+namespace RepoGovernance.Service.Models
 {
     public class NuGetPayload
     {
-        public NuGetPayload(string user, string owner, string repo, string jsonPayload, string payloadType)
+        public NuGetPayload(string user, string owner, string repo, string[] jsonPayload, string payloadType)
         {
             User = user;
             Owner = owner;
@@ -13,7 +15,29 @@ public NuGetPayload(string user, string owner, string repo, string jsonPayload,
         public string? User { get; set; }
         public string? Owner { get; set; }
         public string? Repo { get; set; }
-        public string? JsonPayload { get; set; }
+        //There is some weirdness when the json is embedded in this object and then the object is serialized a second time - it returns an array of strings.
+        public string[]? JsonPayload { get; set; }
+        public string JsonPayloadString
+        {
+            get
+            {
+                if (JsonPayload == null)
+                {
+                    return string.Empty;
+                }
+                return UsingLoopStringBuilder(JsonPayload);
+            }
+        }
         public string? PayloadType { get; set; }
+
+        private string UsingLoopStringBuilder(string[] array)
+        {
+            StringBuilder result = new();
+            foreach (string item in array)
+            {
+                result.Append(item);
+            }
+            return result.ToString();
+        }
     }
 }