Resolved Adding 2 images to one table, attachment ID only appearing for last function

[nearChepstow]

I have 2 images to upload to one table, have got it to the add service and it only adds the one image, so the last attachment insert is overwriting the first one.

var attachment = model.ArticleAttachmentModel.ProcessedFiles.Where(pf => pf.FileByteArray != null).FirstOrDefault();
            var attachmentContentFile = model.ArticleContentFileAttachmentModel.ProcessedFiles.Where(pf => pf.FileByteArray != null).FirstOrDefault();

            if (attachment != null)
            {
                if (attachment.FileByteArray.IsImage())
                {
                    var newAttachment = AttachmentService.NewAttachment(db, attachment.FileByteArray, attachment.Extension, FileContext.ArticleImage, false);
                    newArticleText.Attachment = newAttachment;
                    //newArticleText.Preview_Image_Attachment_ID = newAttachment.ID;
                    newArticleText.Preview_Image_Name = attachment.FileName;
                }
            }

            if (attachmentContentFile != null)
            {
                if (attachmentContentFile.FileByteArray.IsImage())
                {
                    var newContentFileAttachment = AttachmentService.NewAttachment(db, attachmentContentFile.FileByteArray, attachmentContentFile.Extension, FileContext.ArticleContentFile, false);
                    newArticleText.Attachment = model.ConsistsOfSingleFile ? newContentFileAttachment : null;
                    //newArticleText.Main_Content_Attachment_ID = newContentFileAttachment.ID;
                }
            }

            db.SaveChanges();

            return newArticle;

 

[jmcilhinney]

the last attachment insert is overwriting the first one.

Of course it is, because that's exactly what you're telling it to do. You do this:

newArticleText.Attachment = newAttachment;

and then you do this:

newArticleText.Attachment = model.ConsistsOfSingleFile ? newContentFileAttachment : null;

If you set the same property twice, of course it's only going to have the last value you assign.

 

[nearChepstow]

Ye I can see that, .Attachment is the only option I have though. So I need to somehow get around that

 

[nearChepstow]

Ive got it thank you