Merge pull request #3 from socketlabs/fix/multiple-attachments

BillVolz · web-flow · commit 3d2fa7604bec · 2019-08-09T13:21:46.000-04:00
Fixes bug with multiple attachments
diff --git a/src/SocketLabs/InjectionApi/Core/InjectionRequestFactory.cs b/src/SocketLabs/InjectionApi/Core/InjectionRequestFactory.cs
@@ -115,9 +115,9 @@ internal virtual List<AttachmentJson> PopulateList(IEnumerable<IAttachment> atta
 
             var results = new List<AttachmentJson>();
 
-            var attachmentJson = new AttachmentJson();
             foreach (var attachment in attachments)
             {
+                var attachmentJson = new AttachmentJson();
                 attachmentJson.Name = attachment.Name;
                 attachmentJson.ContentType = attachment.MimeType;
                 attachmentJson.ContentId = attachment.ContentId;

Original file line number	Diff line number	Diff line change
`@@ -115,9 +115,9 @@ internal virtual List<AttachmentJson> PopulateList(IEnumerable<IAttachment> atta`
`115`	`115`
`116`	`116`	`var results = new List<AttachmentJson>();`
`117`	`117`
`118`		`- var attachmentJson = new AttachmentJson();`
`119`	`118`	`foreach (var attachment in attachments)`
`120`	`119`	`{`
	`120`	`+ var attachmentJson = new AttachmentJson();`
`121`	`121`	`attachmentJson.Name = attachment.Name;`
`122`	`122`	`attachmentJson.ContentType = attachment.MimeType;`
`123`	`123`	`attachmentJson.ContentId = attachment.ContentId;`